-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[v3.2] Update vendors of container projects #10415
Conversation
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mheon, rhatdan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I'm not sure what's going on here. |
Last PR before 3.2.0-RC2 Signed-off-by: Matthew Heon <[email protected]>
And now |
Ah, we never reset base branch in Cirrus |
Signed-off-by: Matthew Heon <[email protected]>
@cevich PTAL at the DEST_BRANCH change |
/lgtm |
Update containers common to the latest HEAD. Some bug fixes in libimage forced us to have a clearer separation between ordinary images and manifest lists. Hence, when looking up manifest lists without recursing into any of their instances, we need to use `LookupManifestList()`. Also account for some other changes in c/common (e.g., the changed order in the security labels). Further vendor the latest HEAD from Buildah which is required to get the bud tests to pass. Signed-off-by: Valentin Rothberg <[email protected]> <MH: Stripped out vendor bits - just left remaining changes> Signed-off-by: Matthew Heon <[email protected]>
LGTM |
/lgtm |
/hold cancel |
Update our c/common, c/storage, and Buildah to match the main branch.