Closed lyarwood closed 3 months ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
/cc @vladikr
FWIW I took a look at the code and I think this should be pretty straight forward if we apply instance types and preferences before doing the comparisons. Any thoughts?
https://github.com/kubevirt/kubevirt/pull/11455 WIP code here that seems to work.
@lyarwood Thanks! I think it makes a lot of sense to do it now.
@vladikr would you mind taking another look at this now it's marked as ready to review?
The PoC code is under https://github.com/kubevirt/kubevirt/pull/11455
@vladikr if you have time this week could you take another look at this? The implementation PR is also about to land https://github.com/kubevirt/kubevirt/pull/11455 if you also have time.
/approve Thanks, very nice! And apologies for the delay.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: vladikr
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What this PR does / why we need it:
The VM Rollout Strategy feature introduced in KubeVirt 1.2 now allows for specific changes made to a running VM to propagate to the VMI without a restart. This design proposal covers extending this support to cover changes made to the referenced instance type or preference of a running VM.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Fixes #Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR. Approvers are expected to review this list.
Release note: