Merge jammy to noble#590
Merged
Merged
Conversation
Refactor module loading
We've observed many issues over the years with ubuntu's apt servers, including lots of temporary 503's. This commit will 1) Use an in-gcp mirror for debootstrap, which should be fast (it doesn't run on ubuntu's infrastructure at all) 2) Add retries for all apt-get invocations during the build of both the os-image, and the iaas-specific builds - details below Acquire::Retries=10 If a download fails (including transient HTTP errors like 503), apt retries that fetch up to 10 times before giving up on that URL. Without this, one bad response often aborts the whole apt-get run. Acquire::Retries::Delay=true Between retries, apt waits with increasing delay (backoff) instead of hammering the server immediately. On Jammy this is the “delay between retries” behavior (apt ≥ 2.0). Cumulatively with Retries=10, you get many attempts spread over time, which helps when snapshot.ubuntu.com is briefly overloaded rather than hard-down. Acquire::http::Timeout=120 and Acquire::https::Timeout=120 Each individual HTTP/HTTPS connection apt opens to a mirror can block for at most 120 seconds before apt treats it as stuck and fails that attempt (which can then trigger a retry if retries remain). ai-assisted=yes [TNZ-88995]
Add apt retries, optional mirror for debootstrap
🐛 Fix ytt comment
Turns out, the internal GCE mirror is designed only for actual vms running in GCP, and either didn't play well with the Concourse container's egress or doesn't server jammy. Fixes debootstrap --arch=amd64 jammy /mnt/stemcells/null/null/ubuntu/work/work/chroot http://gce.archive.ubuntu.com/ubuntu I: Retrieving InRelease I: Retrieving Release E: Failed getting release file http://gce.archive.ubuntu.com/ubuntu/dists/jammy/Release
…irror Use kernel.org mirror for debootstrap
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
aramprice
approved these changes
May 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note: the pipeline has not yet been flown