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
Automated cherry pick of #102882: devicemanager: checkpoint: support pre-1.20 data #106295
Automated cherry pick of #102882: devicemanager: checkpoint: support pre-1.20 data #106295
Conversation
/kind bug |
/hold |
/triage accepted |
0bd427e
to
6d02f2d
Compare
verified the (reduced version of) e2e tests locally on BM - the serial lane doesn't have multi-NUMA, SRIOV-enabled machines. |
/pull |
/help |
/hold cancel |
FYI @endocrimes @klueska @swatisehgal Unfortunately this isn't a trivial rebase, some changes had to be done especially to have some e2e test coverage. |
So the "core" code changes cherry-picked cleanly, but you needed to patch up the e2e tests with some extra commits? or did you have to also tweak the first commit in this chain to resolve some conflicts? |
the core code changes got in relatively cleanly. The lion's share of the changes and extra work went into the e2e tests. I'd still be kindly requesting/happy with a deep review, because the 1.20 branch is in deep maintenance mode and I'd love to contribute positively to the stability of the branch. Note: I'm aware I'm asking quite a lot here, still I believe this backport is worth because it silently breaks the devicemanager flow on upgrades. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
this is a slightly riskier backport but imo worth it to de-risk upgrades for folks.
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: endocrimes, fromanirh, klueska 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 |
Thank you for providing context around the changes and the extra commit @fromanirh |
/test pull-kubernetes-bazel-test |
…tke/v1.20.6' (merge request !909) tke/v1.20.6-device-manager-support-statefile-v1 Issue:kubernetes#102880 Cherry Pick:kubernetes#106295 device manager 读取state file时同时适配v1和v2两种格式
Cherry pick of #102882 on release-1.20.
#102882: devicemanager: checkpoint: support pre-1.20 data
For details on the cherry pick process, see the cherry pick requests page.