Skip to content

chore(vm): add system migration policy#2236

Merged
LopatinDmitr merged 3 commits into
mainfrom
feat/vm/add-system-migration-policy
Apr 20, 2026
Merged

chore(vm): add system migration policy#2236
LopatinDmitr merged 3 commits into
mainfrom
feat/vm/add-system-migration-policy

Conversation

@LopatinDmitr
Copy link
Copy Markdown
Contributor

@LopatinDmitr LopatinDmitr commented Apr 18, 2026

Description

Add a system-level live migration policy override sourced from ModuleConfig/virtualization annotation virtualization.deckhouse.io/system-migration-policy.

The controller now reads this annotation at startup and, when valid, applies it globally in live migration policy calculation.

What is the expected result?

  1. Set annotation on ModuleConfig/virtualization:
    virtualization.deckhouse.io/system-migration-policy: <valid policy>.
  2. Restart/rollout virtualization-controller.
  3. Run VMOP migration/eviction.
  4. Confirm effective migration configuration follows the system policy override (VM spec and VMOP force do not override it).
  5. If annotation is missing or invalid, behavior remains unchanged.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: core
type: chore
summary: "Add a system live migration policy override via ModuleConfig annotation for VMOP/live migration policy calculation."
impact_level: low

@LopatinDmitr LopatinDmitr self-assigned this Apr 18, 2026
@LopatinDmitr LopatinDmitr added this to the v1.8.0 milestone Apr 18, 2026
@LopatinDmitr LopatinDmitr force-pushed the feat/vm/add-system-migration-policy branch 2 times, most recently from 5f206e0 to 3840050 Compare April 18, 2026 11:52
Signed-off-by: Dmitry Lopatin <dmitry.lopatin@flant.com>
@LopatinDmitr LopatinDmitr force-pushed the feat/vm/add-system-migration-policy branch from 3840050 to db10b35 Compare April 18, 2026 11:54
@LopatinDmitr LopatinDmitr requested a review from diafour April 18, 2026 12:16
Comment thread images/virtualization-artifact/cmd/virtualization-controller/main.go Outdated
Comment thread images/virtualization-artifact/cmd/virtualization-controller/main.go Outdated
@LopatinDmitr LopatinDmitr requested a review from diafour April 20, 2026 08:45
Signed-off-by: Dmitry Lopatin <dmitry.lopatin@flant.com>
@LopatinDmitr LopatinDmitr force-pushed the feat/vm/add-system-migration-policy branch from 5df31cb to ee449a1 Compare April 20, 2026 09:55
@LopatinDmitr LopatinDmitr changed the title feat(vm): add system migration policy chore(vm): add system migration policy Apr 20, 2026
Comment thread images/virtualization-artifact/pkg/livemigration/system_migration_policy.go Outdated
…ion_policy.go

Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com>
Signed-off-by: Dmitry Lopatin <93423466+LopatinDmitr@users.noreply.github.com>
@LopatinDmitr LopatinDmitr merged commit 7ef2217 into main Apr 20, 2026
27 of 28 checks passed
@LopatinDmitr LopatinDmitr deleted the feat/vm/add-system-migration-policy branch April 20, 2026 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants