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

chore(ci): add image pull with retry #508

Merged
merged 3 commits into from
Feb 22, 2024
Merged

Conversation

bsherman
Copy link
Contributor

This is another in a series of PRs which have been proposed for #502 .

My thought is to identify areas in build_image which have spurious failures and fix them if reasonable within our scripting, or otherwise address issues, like this one.

The idea here is we fix the spurious failures pulling our (very large) base images, by pulling them to the build runner before using the buildah action.

@bsherman bsherman marked this pull request as ready for review February 22, 2024 05:22
@bsherman bsherman requested a review from castrojo as a code owner February 22, 2024 05:22
@bsherman bsherman requested a review from a team February 22, 2024 05:22
@bsherman
Copy link
Contributor Author

This still won't catch all issues, and it doesn't even try to pre-pull akmods or config image layers ... but we have other efforts in flight to address those concerns by moving to a dnf/yum repo instead.

@bsherman bsherman requested a review from a team February 22, 2024 16:30
@bsherman bsherman enabled auto-merge February 22, 2024 16:30
@bsherman
Copy link
Contributor Author

The failure on this currently is a kernel version incompatibility between a kmod and image kernel. Shouldn't block merging of this.

@bsherman bsherman disabled auto-merge February 22, 2024 17:13
@bsherman bsherman merged commit 27f4c63 into main Feb 22, 2024
22 of 24 checks passed
@bsherman bsherman deleted the add-retry-to-image-pull branch February 22, 2024 17:15
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.

3 participants