Skip to content

chore(vmop): do not clone labels on vm#2009

Merged
eofff merged 6 commits intomainfrom
chore/vmop/do-not-restore-labels-on-cloned-vm
Feb 20, 2026
Merged

chore(vmop): do not clone labels on vm#2009
eofff merged 6 commits intomainfrom
chore/vmop/do-not-restore-labels-on-cloned-vm

Conversation

@eofff
Copy link
Contributor

@eofff eofff commented Feb 18, 2026

Description

Do not clone labels on VM by restore virtual machine operation.

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

Cloning labels may cause issues with incoming traffic routed by services.

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: vmop
type: chore
summary: Do not clone labels on VM by restore virtual machine operation.

@eofff eofff self-assigned this Feb 18, 2026
@eofff eofff added the e2e/run Run e2e test on cluster of PR author label Feb 18, 2026
@deckhouse-BOaTswain
Copy link
Contributor

deckhouse-BOaTswain commented Feb 18, 2026

Workflow has started.
Follow the progress here: Workflow Run

The target step completed with status: failure.

@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Feb 18, 2026
@eofff eofff changed the title do not clone labels chore(vmop): do not clone labels on vm Feb 19, 2026
@eofff eofff added this to the v1.6.0 milestone Feb 19, 2026
@eofff eofff requested a review from hardcoretime February 19, 2026 07:49
@eofff eofff marked this pull request as ready for review February 19, 2026 07:49
@hardcoretime
Copy link
Contributor

Labels should also be removed from the vmrestore logic.

@eofff eofff force-pushed the chore/vmop/do-not-restore-labels-on-cloned-vm branch from 5053665 to 3931b15 Compare February 19, 2026 12:02
Valeriy Khorunzhin and others added 4 commits February 19, 2026 16:21
Signed-off-by: Valeriy Khorunzhin <valeriy.khorunzhin@flant.com>
…/restorers/vm_restorer.go

Co-authored-by: Roman Sysoev <36233932+hardcoretime@users.noreply.github.com>
Signed-off-by: Valeriy Khorunzhin <b1kapb999@gmail.com>
Signed-off-by: Valeriy Khorunzhin <valeriy.khorunzhin@flant.com>
Signed-off-by: Valeriy Khorunzhin <valeriy.khorunzhin@flant.com>
@eofff eofff force-pushed the chore/vmop/do-not-restore-labels-on-cloned-vm branch from a16553e to d016e30 Compare February 19, 2026 13:21
@eofff eofff requested a review from hardcoretime February 19, 2026 15:19
hardcoretime
hardcoretime previously approved these changes Feb 19, 2026
Signed-off-by: Valeriy Khorunzhin <valeriy.khorunzhin@flant.com>
Signed-off-by: Vladislav Panfilov <vladislav.panfilov@flant.com>
@eofff eofff merged commit e586260 into main Feb 20, 2026
27 of 28 checks passed
@eofff eofff deleted the chore/vmop/do-not-restore-labels-on-cloned-vm branch February 20, 2026 12:15
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.

4 participants

Comments