Skip to content
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

Pass additional KVM environment parameters into guest on cold boot and in ensure #17

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

sysvinit
Copy link
Member

@sysvinit sysvinit commented Oct 16, 2024

Add a mechanism similar to the qemu binary generation signalling (seeding in the tmp volume at cold boot and then periodically write into a file in the guest's /run at runtime) in order to signal additional parameters to the agent. Use this mechanism to signal the CPU flags and the RBD pool into the guest, so the guest can schedule a cold reboot to allow us to make the necessary changes after the qemu process has exited and before it's restarted.

@sysvinit sysvinit requested a review from ctheune October 16, 2024 07:54
Copy link
Member

@ctheune ctheune left a comment

Choose a reason for hiding this comment

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

Tiny bit of test coverage would be nice. :)

@sysvinit sysvinit force-pushed the PL-131857-signal-cold-reboot-changes branch from b2574f7 to ef278b0 Compare October 16, 2024 10:43
In addition to the boot-time qemu binary generation, also separately
seed the CPU flags and the RBD pool so the guest can detect changes at
runtime.
@sysvinit sysvinit force-pushed the PL-131857-signal-cold-reboot-changes branch from ef278b0 to b3c4598 Compare October 16, 2024 12:28
@sysvinit sysvinit changed the title Seed additional guest ENC parameters into VM at cold boot time Pass additional KVM environment parameters into guest on cold boot and in ensure Oct 16, 2024
Allow the guest to detect whether the KVM environment has changed at
runtime in combination with the boot-time seed data.
@sysvinit sysvinit force-pushed the PL-131857-signal-cold-reboot-changes branch from b3c4598 to 9e42515 Compare October 16, 2024 14:28
@ctheune ctheune marked this pull request as ready for review October 16, 2024 14:28
@ctheune ctheune merged commit 25d44d2 into master Oct 16, 2024
1 check passed
@sysvinit sysvinit deleted the PL-131857-signal-cold-reboot-changes branch October 18, 2024 09:00
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