Skip to content

fix(core): bump kubevirt to prevent hotplug mount leaks on failed migration cleanup#2385

Merged
yaroslavborbat merged 1 commit into
mainfrom
fix/mount-leaks
May 21, 2026
Merged

fix(core): bump kubevirt to prevent hotplug mount leaks on failed migration cleanup#2385
yaroslavborbat merged 1 commit into
mainfrom
fix/mount-leaks

Conversation

@yaroslavborbat
Copy link
Copy Markdown
Member

@yaroslavborbat yaroslavborbat commented May 21, 2026

Description

Bump bundled KubeVirt to a version that includes an upstream fix for hotplug volume cleanup during migration cleanup.

The upstream change switches cleanup logic from Unmount() to UnmountAll() for this path.

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

Failed migration with hotplug volumes can leave stale mounts on the target node.

The reason is that Unmount() removes only volumes absent in the VMI spec, while migration cleanup must unmount all hotplug volumes. This causes mount leaks and may affect subsequent operations on the node.

What is the expected result?

  1. Run migration for a VMI with hotplug volumes and reproduce a failed migration.
  2. Wait for cleanup on the target node.
  3. Verify that no stale hotplug mounts remain after cleanup.
  4. Verify repeated migration attempts are not affected by leftover mounts.

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: fix
summary: "Prevent hotplug volume mount leaks on failed migration cleanup by bumping KubeVirt."

@yaroslavborbat yaroslavborbat added this to the v1.9.0 milestone May 21, 2026
@yaroslavborbat yaroslavborbat changed the title fix(core): bump kubevirt to include hotplug mount leak fix fix(core): bump kubevirt to prevent hotplug mount leaks on failed migration cleanup May 21, 2026
@yaroslavborbat yaroslavborbat marked this pull request as ready for review May 21, 2026 14:58
@yaroslavborbat yaroslavborbat requested a review from Isteb4k as a code owner May 21, 2026 14:58
Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com>
@yaroslavborbat yaroslavborbat merged commit 433d415 into main May 21, 2026
22 of 23 checks passed
@yaroslavborbat yaroslavborbat deleted the fix/mount-leaks branch May 21, 2026 15:59
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