Skip to content

feat(vmsnapshot): add hotplugged image manifests to snapshot #1198

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

Merged

Conversation

hardcoretime
Copy link
Contributor

@hardcoretime hardcoretime commented Jun 25, 2025

Description

A virtual machine block device attachment that has a reference to a virtual image must also be snapshotted.

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

A virtual machine block device attachment must be restored together with the virtual machine.

What is the expected result?

All virtual machine block device attachments from the virtual machine snapshot will be restored.

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: vmsnapshot
type: feature
summary: "All virtual machine block device attachment manifests are stored in the virtual machine snapshot when it is created."

Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime added this to the v0.22.0 milestone Jun 25, 2025
@hardcoretime hardcoretime added the e2e/run Run e2e test on cluster of PR author label Jun 25, 2025
@deckhouse-BOaTswain
Copy link
Contributor

deckhouse-BOaTswain commented Jun 25, 2025

Workflow has started.
Follow the progress here: Workflow Run

The target step completed with status: success.

@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Jun 25, 2025
@hardcoretime hardcoretime merged commit db151cb into main Jun 25, 2025
72 of 75 checks passed
@hardcoretime hardcoretime deleted the feat/vmsnapshot/add-hotplugged-image-manifests-to-snapshot branch June 25, 2025 14:25
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.

3 participants