-
Notifications
You must be signed in to change notification settings - Fork 847
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid running apt-get more than once per day #2281
base: develop
Are you sure you want to change the base?
Avoid running apt-get more than once per day #2281
Conversation
I was thinking more updating the packages than the sources, but this is needed too. We'll need to account for utilities as well and the VVV version though, the date isn't enough |
I figure we store an SHA of a string of the version/utilities/etc |
Wondering if we can extend this also to other utilities or command like npm or wp-cli download. |
I'll think about this, i'm sure we can do this kind of thing for everything. |
@@ -12,6 +12,7 @@ | |||
. "/srv/config/bash_aliases" | |||
|
|||
# cleanup | |||
mkdir -p /vvv |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
imo this should go in /vagrant
and the move to /vvv
should be a separate issue where we symlink it to /vagrant
, that way we aren't spreading things across multiple folders
I'll turn this into a draft, since I don't have time to work on this on the next couple days. |
I think the todo list of things here should be:
So we should compare an SHA hash of:
We can then provide finder control in the future by allowing the We should also look into appending the VVV commit hash to the version in the vagrant file so that people on At that point we're probably good to extend this to the entire main provisioner |
Additionally, only succesful provisions should update the hash so that it doesn't stay stuck in a broken state |
any updates @msaggiorato |
This attempts to speed up provision by addressing #2272