Skip to content

fix(kubevirt): Improve error handling in migration monitor during and after live migration#1040

Closed
yaroslavborbat wants to merge 0 commit into
mainfrom
fix/kubevirt/fix-migration-monitor-after-completion
Closed

fix(kubevirt): Improve error handling in migration monitor during and after live migration#1040
yaroslavborbat wants to merge 0 commit into
mainfrom
fix/kubevirt/fix-migration-monitor-after-completion

Conversation

@yaroslavborbat
Copy link
Copy Markdown
Member

@yaroslavborbat yaroslavborbat commented May 12, 2025

Description

The migration monitor (migrationMonitor) continues to make calls to GetJobStats() even after a successful migration has completed.
This leads to errors like:

"Failed to get domain job info": virError(Code=42, Domain=10, Message='Domain not found...')

These errors occur because the domain has already been successfully migrated and cleaned up on the source node.

Additionally, during migration, there are cosmetic errors such as:

"Requested operation is not valid: domain is not running"

This PR resolves the issue by properly handling domain lifecycle events during and after migration.

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

What is the expected result?

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: Improve error handling in migration monitor during and after live migration
impact_level: low

@yaroslavborbat yaroslavborbat added this to the v0.19.0 milestone May 12, 2025
@yaroslavborbat yaroslavborbat marked this pull request as ready for review May 13, 2025 08:25
@yaroslavborbat yaroslavborbat force-pushed the fix/kubevirt/fix-migration-monitor-after-completion branch 2 times, most recently from 15b4bbe to 0710a2e Compare May 13, 2025 08:27
@Isteb4k Isteb4k modified the milestones: v0.19.0, v0.20.0 May 29, 2025
@yaroslavborbat yaroslavborbat marked this pull request as draft June 3, 2025 16:13
@yaroslavborbat yaroslavborbat force-pushed the fix/kubevirt/fix-migration-monitor-after-completion branch from 0710a2e to 8ae1c93 Compare June 9, 2025 07:41
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