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

Use Ubuntu latest for the buildbox #27497

Merged
merged 1 commit into from
Jun 7, 2023
Merged

Conversation

jakule
Copy link
Contributor

@jakule jakule commented Jun 6, 2023

After moving the Connect to a separate Docker image #27175 we're able to use the latest ubuntu LTS on our build image. We're not using this image to produce any releases (only CI runs), so updating the image will have no effect on our releases.

I also remove the TODO for v13.

Contributes to #26856

@rosstimothy
Copy link
Contributor

Do we really want latest though? What kind of impact will that have on reproducible builds?

Copy link
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

I suppose I could be convinced to go for ":latest", but the main worry is that it'll do a release-sized jump from time to time. I'd be happy with the more recent LTS; it's already a 4-years jump compared to a few weeks ago.

build.assets/Dockerfile Outdated Show resolved Hide resolved
After moving the Connect to a separate Docker image #27175 we're able to use the latest ubuntu LTS on our build image. We're not using this image to produce any releases (only CI runs), so updating the image will have no effect on our releases.
@jakule jakule force-pushed the jakule/buildbox-use-latest branch from 73467a2 to 2ee21ba Compare June 6, 2023 22:09
@jakule
Copy link
Contributor Author

jakule commented Jun 6, 2023

I reverted latest to 22.04. I'll try to keep updating this image from time to time 🤷‍♂️

@jakule
Copy link
Contributor Author

jakule commented Jun 6, 2023

What kind of impact will that have on reproducible builds?

No impact. This image is not used for any builds now.

@jakule jakule requested a review from codingllama June 6, 2023 22:12
@codingllama
Copy link
Contributor

Thanks for doing this. Just the fact that we can update it, if the need arises or if it becomes convenient to do so, is excellent.

@jakule jakule added this pull request to the merge queue Jun 7, 2023
Merged via the queue into master with commit 5d53c91 Jun 7, 2023
20 checks passed
@jakule jakule deleted the jakule/buildbox-use-latest branch June 7, 2023 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants