-
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
Update c/storage after https://github.com/containers/storage/pull/1436 #16581
Conversation
@mtrmac: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
7eb2e46
to
5857ec2
Compare
5857ec2
to
d1d1db3
Compare
Now includes updates to avoid newly deprecated interfaces. |
d1d1db3
to
7425816
Compare
Current test failures match #16578 , so I guess this is blocked on that work. |
#16578 should fix this PR, you will need to rebase. |
7425816
to
509afd7
Compare
The c/storage PR was merged, and requires Podman updates from this PR to avoid now-deprecated interfaces. So, leaving this PR open, and ready for merge at some point. Merging it before the eventual vendor dance, and running the new code base in the meantime, might also improve our confidence in the c/storage changes a bit. |
... and update to remove the now-deprecated Locker interface. Signed-off-by: Miloslav Trmač <mitr@redhat.com>
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mtrmac, 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 |
No description provided.