-
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
DO NOT MERGE: Test vendoring with precise pulled-image IDs #24462
Conversation
Signed-off-by: Paul Holzinger <[email protected]>
Regression test for containers#23550. Setting the TZDIR env should make no difference for the local timezone as this is not a real timezone name that is resolved from that directory. Signed-off-by: Paul Holzinger <[email protected]>
Added in containers/buildah#5783 Signed-off-by: Paul Holzinger <[email protected]>
containers/common#2227 Signed-off-by: Paul Holzinger <[email protected]>
> go mod edit -replace github.com/containers/image/v5=github.com/mtrmac/image/v5@copy-resolve-destination Signed-off-by: Miloslav Trmač <[email protected]>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mtrmac 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 |
> go mod edit -replace github.com/containers/common=github.com/mtrmac/common@precise-pull-id Signed-off-by: Miloslav Trmač <[email protected]>
f55c51d
to
5e0e083
Compare
Signed-off-by: Miloslav Trmač <[email protected]>
Signed-off-by: Miloslav Trmač <[email protected]>
“TESTING: Drop the new composefs path” added to confirm a guess at the cause of #24463 . |
The c/image and c/common PRs have merged, and seem to be succeeding in #24447 , closing this PR in favor of ongoing work in the other one. |
This is #24447 + containers/image#2609 + containers/common#2209 .
Does this PR introduce a user-facing change?