-
-
Notifications
You must be signed in to change notification settings - Fork 130
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
fix(docs): document qemu-guest-agent behavior #670
Conversation
Modify example to start with disabled guest agent. Signed-off-by: Oto Petřík <oto.petrik@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome as usual, @otopetrik, thanks for clarifying all of this!
@@ -23,7 +23,8 @@ resource "proxmox_virtual_environment_vm" "ubuntu_vm" { | |||
vm_id = 4321 | |||
|
|||
agent { | |||
enabled = true | |||
# read 'Qemu guest agent' section, change to true only when ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏼
"Monitor" tab in Proxmox GUI can be used to send low-level commands to `qemu`. | ||
See the [documentation](https://www.qemu.org/docs/master/system/monitor.html). | ||
Commands `system_powerdown` and `quit` have proven useful in shutting down VMs | ||
with `agent.enabled = true` and no agent running. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really helpful!
@all-contributors please add @otopetrik for documentation |
I've put up a pull request to add @otopetrik! 🎉 |
Modify example to start with disabled guest agent.
Contributor's Note
/docs
for any user-facing features or additions./example
for any new or updated resources / data sources.make example
to verify that the change works as expected.Proof of Work
#669 and #449 show that
agent.enabled = true
behavior is not obvious and should be documented. It should not be the default in the example, but users are still expected to change it to true once they understand the details and configure everything correctly.Community Note
Closes #669 | Relates #0000