-
Notifications
You must be signed in to change notification settings - Fork 2.8k
DO NOT MERGE: Test vendoring with precise pulled-image IDs #24462
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
Conversation
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
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 <pholzing@redhat.com>
Added in containers/buildah#5783 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
containers/common#2227 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
> go mod edit -replace github.com/containers/image/v5=github.com/mtrmac/image/v5@copy-resolve-destination Signed-off-by: Miloslav Trmač <mitr@redhat.com>
[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č <mitr@redhat.com>
f55c51d
to
5e0e083
Compare
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
“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?