Skip to content

Comments

fix(vm): patch kvvm.spec.template.metadata on VM metadata changes#1971

Merged
hardcoretime merged 4 commits intomainfrom
fix/vm/sync-kvvm-spec-labels
Feb 13, 2026
Merged

fix(vm): patch kvvm.spec.template.metadata on VM metadata changes#1971
hardcoretime merged 4 commits intomainfrom
fix/vm/sync-kvvm-spec-labels

Conversation

@hardcoretime
Copy link
Contributor

@hardcoretime hardcoretime commented Feb 9, 2026

Description

KubeVirt virtual machine spec template metadata labels and annotations are patched by the SyncMetadataHandler when a Deckhouse virtual machine updates its labels and annotations.

Why do we need it, and what problem does it solve?

This change is required to resolve the problem where KubeVirt virtual machine instances and their pods may restore their labels and annotations in some cases, even after these have been deleted from the Deckhouse virtual machine metadata.

What is the expected result?

The SyncMetadataHandler should properly synchronize labels and annotations.

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: vm
type: fix
summary: "Labels and annotations now work properly on virtual machines."

@hardcoretime hardcoretime added this to the v1.6.0 milestone Feb 9, 2026
@hardcoretime hardcoretime force-pushed the fix/vm/sync-kvvm-spec-labels branch 2 times, most recently from 3cad89b to 392fe0d Compare February 10, 2026 12:38
Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime force-pushed the fix/vm/sync-kvvm-spec-labels branch from 392fe0d to 7bf2d0d Compare February 10, 2026 12:57
@hardcoretime hardcoretime changed the title fix(vm): sync kvvm spec labels fix(vm): patch kvvm.spec.template.metadata on VM metadata changes Feb 10, 2026
@hardcoretime hardcoretime marked this pull request as ready for review February 10, 2026 14:32
@hardcoretime hardcoretime force-pushed the fix/vm/sync-kvvm-spec-labels branch from c83a9d5 to 4aa53b3 Compare February 11, 2026 14:26
Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime force-pushed the fix/vm/sync-kvvm-spec-labels branch from 4aa53b3 to ab7b941 Compare February 11, 2026 15:13
Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime force-pushed the fix/vm/sync-kvvm-spec-labels branch from 0c40717 to 2b5e436 Compare February 11, 2026 23:07
@hardcoretime hardcoretime marked this pull request as draft February 12, 2026 13:46
Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime force-pushed the fix/vm/sync-kvvm-spec-labels branch from e663fde to c5c49d9 Compare February 13, 2026 00:02
@hardcoretime hardcoretime marked this pull request as ready for review February 13, 2026 00:21
@hardcoretime hardcoretime requested a review from fl64 as a code owner February 13, 2026 00:21
@hardcoretime hardcoretime merged commit 697f05d into main Feb 13, 2026
29 of 30 checks passed
@hardcoretime hardcoretime deleted the fix/vm/sync-kvvm-spec-labels branch February 13, 2026 07:44
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