-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Bump Compat API maximum version to v1.41 #14219
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mheon 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 |
LGTM |
/lgtm |
@mheon Did you review changes in the API to see if we need to make changes in the compat handlers? |
Docker bumped their API, so we should do the same. Fixes containers#14204 Signed-off-by: Matthew Heon <[email protected]>
Rebased and repushed. Looks like this was completely forgotten about, but I'm pretty sure we still want it. |
@mheon I thought we were waiting on https://issues.redhat.com/browse/RUN-1538 before making this change. |
I thought we were going to merge, and make the changes at a later date? |
One more flake, restarting |
Friendly ping |
I think this is time to merge. |
/hold cancel Let's go ahead and merge, we can deal with the diffs at a later point |
Docker bumped their API, so we should do the same.
Fixes #14204
Does this PR introduce a user-facing change?