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

Bump base image in docker push workflow to 20.04 #5432

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

jeffwidman
Copy link
Member

We've been running on 20.04 since #4394, I just didn't realize this actions workflow also pinned the version. So this bumps to our current base image.

Unfortunately, this will likely be outdated in the future again, once #5030 lands... and likely to be an ongoing issue of drift whenever we bump Ubuntu versions.

I wish there was an easy way to extract this value from the Dockerfile, but everything I come up with seems quite hacky/brittle.

We've been running on `20.04` since #4394, I just didn't realize this actions workflow also pinned the version. So this bumps to our current base image.

Unfortunately, this will likely be outdated in the future again, once #5030 lands... and likely to be an ongoing issue.

I wish there was an easy way to extract this value from the Dockerfile, but everything I come up with seems quite hacky/brittle.
@jeffwidman jeffwidman requested a review from a team as a code owner July 27, 2022 05:36
Copy link
Member

@jurre jurre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could do a similarly hacky thing that we do for bundler which is to write a little script that does all of them in one go (bin/update-bundler.rb), and maybe a workflow that checks that we didn't forget, but it's not great

@jeffwidman jeffwidman merged commit 3852316 into main Jul 28, 2022
@jeffwidman jeffwidman deleted the bump-ubuntu-base-image-actions-workflow branch July 28, 2022 14:27
@jeffwidman
Copy link
Member Author

True I didn't even think of that as an option. A workflow to verify would be great, but again, tough to keep it up to date. I guess for now just ignore it unless something pops to mind suddenly.

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 this pull request may close these issues.

2 participants