Skip to content

Add test that reboots a guest during live migration#88

Draft
tpressure wants to merge 1 commit intocyberus-technology:mainfrom
tpressure:reboot_during_live_migration
Draft

Add test that reboots a guest during live migration#88
tpressure wants to merge 1 commit intocyberus-technology:mainfrom
tpressure:reboot_during_live_migration

Conversation

@tpressure
Copy link
Contributor

@tpressure tpressure requested a review from phip1611 December 9, 2025 13:08
@tpressure tpressure marked this pull request as draft December 9, 2025 13:08
@tpressure tpressure force-pushed the reboot_during_live_migration branch from b2bd476 to 9a4dfcf Compare December 9, 2025 13:09
@tpressure tpressure force-pushed the reboot_during_live_migration branch 2 times, most recently from 7c75cca to 81749f5 Compare December 9, 2025 13:12
@tpressure tpressure force-pushed the reboot_during_live_migration branch from 81749f5 to 7fe64cd Compare December 9, 2025 13:13
@phip1611 phip1611 added the ga-gate Our next milestone (currently General Availability (GA)). label Jan 21, 2026
Copy link
Member

@phip1611 phip1611 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase and adapt the code to our latest standards

controllerVM.succeed("virsh define /etc/domain-chv.xml")
controllerVM.succeed("virsh start testvm")

assert wait_for_ssh(controllerVM)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obsolete, we now throw exceptions (#121)

Suggested change
assert wait_for_ssh(controllerVM)
wait_for_ssh(controllerVM)


assert wait_for_ssh(controllerVM)

status, _ = ssh(controllerVM, "screen -dmS stress stress -m 4 --vm-bytes 400M")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obsolete, we now throw exceptions (#121)

Suggested change
status, _ = ssh(controllerVM, "screen -dmS stress stress -m 4 --vm-bytes 400M")
ssh(controllerVM, "screen -dmS stress stress -m 4 --vm-bytes 400M")

status, _ = ssh(controllerVM, "screen -dmS stress stress -m 4 --vm-bytes 400M")
assert status == 0

status, _ = ssh(controllerVM, "screen -dmS nohup sh -c 'sleep 10 && reboot'")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
status, _ = ssh(controllerVM, "screen -dmS nohup sh -c 'sleep 10 && reboot'")
ssh(controllerVM, "screen -dmS nohup sh -c 'sleep 10 && reboot'")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ga-gate Our next milestone (currently General Availability (GA)).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants