Skip to content
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

Update Apt automatically #582

Closed
BronsonQuick opened this issue Sep 20, 2018 · 7 comments · Fixed by #732
Closed

Update Apt automatically #582

BronsonQuick opened this issue Sep 20, 2018 · 7 comments · Fixed by #732
Assignees

Comments

@BronsonQuick
Copy link
Member

As a developer, my apt packages should update frequently otherwise I run the risk of bumping a PHP version in my configuration file and I can get a failed provision.

I have had code for this here: https://github.com/Chassis/Chassis/pull/476/files#diff-556e699d648d31f209eec94844939770R15 it's important that we ship this for developers.

@johnbillion
Copy link
Contributor

I just ran into this exact issue when attempting to install an extension on an existing box. The extension's Debian package returned a 404. Turns out I had to run sudo apt-get update.

@rmccue
Copy link
Contributor

rmccue commented Aug 20, 2019

IMO, best way to fix this is to add into the preprovision.sh. Rather than just checking whether the did-update file exists, we should write a timestamp into it, and check that isn't older than ~2 weeks (or something; if there's a better period to use, let's do that).

@BronsonQuick
Copy link
Member Author

Hah it's interesting that you suggested the time stamp. I was actually thinking I should touch a file and add a timestamp for my auto updates branch so it only runs once a week. I'll think about making that approach reusable here as well!

@rmccue
Copy link
Contributor

rmccue commented Aug 20, 2019

👍 Only potential issue might be that the apt updates run inside the machine, whereas the others have to run outside the machine (in case there's private repos or anything).

@BronsonQuick
Copy link
Member Author

Ahh yeah. Good point! 🍆

@BronsonQuick
Copy link
Member Author

@rmccue
Copy link
Contributor

rmccue commented Oct 9, 2019

This was alas not fixed by #476, and now that we have a basebox, the issue is popping up on initial provisions too. #720 will fix that temporarily, but IMO we should consider the time-based fix instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants