-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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 #120550: DRA: call plugins for claims even if exist in cache #121748
Automated cherry pick of #120550: DRA: call plugins for claims even if exist in cache #121748
Conversation
Today, DRA manager does not call plugin NodePrepareResource for claims that it previously successfully handled, that is, if claims are present in cache (checkpoint) even if node rebooted. After node reboots, it is required to call DRA plugin for resource claims so that plugins may prepare them again in case the resources dont persist reboot. To achieve that, once kubelet is started, we call DRA plugins for claims once if a pod sandbox is required to be created during PodSync. Signed-off-by: adrianc <adrianc@nvidia.com>
adjust existing tests and add new test flows to cover new DRA manager behaviour Signed-off-by: adrianc <adrianc@nvidia.com>
going over cherry-picks i see that alpha feature related bugs are not considered critical, so if this does not qualify for backport feel free to close this one. Reason to add it:
|
/test pull-kubernetes-integration |
/triage accepted |
LGTM label has been added. Git tree hash: 24592c7e926847b10141cfc75f98df854b4b0892
|
/approve |
/cc kubernetes/release-managers |
@bart0sh is there anything else required ? |
@adrianchiris We're waiting for a Release managers approval as far as I understand. |
How big is the risk that we introduce a regression here? It feels that we changed a non-trivial behavior, right? |
Adding my POV: the gist of the change: note: dra plugins are defined as idempotent, so callling more than once with same parameters should yield same result. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adrianchiris, klueska, saschagrunert 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 |
Cherry pick of #120550 on release-1.28.
#120550: DRA: call plugins for claims even if exist in cache
For details on the cherry pick process, see the cherry pick requests page.