-
Notifications
You must be signed in to change notification settings - Fork 2.7k
compat: remove deprecated VirtualSize #26213
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
base: main
Are you sure you want to change the base?
compat: remove deprecated VirtualSize #26213
Conversation
Since compat version 1.43 the VirtualSize field in the GET /images/{name}/json, GET /images/json, and GET /system/df responses is deprecated and will no longer be included in API v1.44. Use the Size field instead, which contains the same information. Signed-off-by: Nicola Sella <nsella@redhat.com>
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-sigs/prow repository. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: inknos The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What happens if a client talks in an older API version? Should there be some version check around this instead? |
I think the answer depends on our policy for deprecations. What is it? And is the libpod deprecation policy the same as for compat? |
I think the policy for compat is match whatever docker is doing. If you can run the latest docker and use API v1.40 (or anything below the removal) and it still returns that field then we need to do as well. For libpod we have a strict semver commitment so any breaking changes such as remove response field should wait for the next major version. |
A friendly reminder that this PR had no activity for 30 days. |
Since compat version 1.43 the VirtualSize field in the GET /images/{name}/json, GET /images/json, and
GET /system/df responses is deprecated and will no longer be included in API v1.44. Use the Size field instead, which contains the same information.
Does this PR introduce a user-facing change?