Skip to content

fix(build): watchdog must not qm start a template#5

Open
gabbelitoV2 wants to merge 2 commits into
ConvoyPanel:mainfrom
gabbelitoV2:fix/watchdog-skip-template
Open

fix(build): watchdog must not qm start a template#5
gabbelitoV2 wants to merge 2 commits into
ConvoyPanel:mainfrom
gabbelitoV2:fix/watchdog-skip-template

Conversation

@gabbelitoV2

Copy link
Copy Markdown
Contributor

The build-VM watchdog restarts the VM when it has reported status stopped for 20s. A template also reports stopped, so once packer finishes and converts the VM to a template the watchdog calls qm start and gets the error you cannot start a vm if it is a template, repeatedly, until it burns through its restart limit. The build still passes (the error is swallowed) but the log is alarming and the watchdog wastes time. Check the template flag before restarting and exit cleanly instead.

The build-VM watchdog restarts the VM when it has reported status stopped for 20s. A template also reports stopped, so once packer finishes and converts the VM to a template the watchdog calls qm start and gets the error you cannot start a vm if it is a template, repeatedly, until it burns through its restart limit. The build still passes (the error is swallowed) but the log is alarming and the watchdog wastes time. Check the template flag before restarting and exit cleanly instead.
…ding down

The watchdog exits the moment the communicator port (SSH 22 / WinRM 5985) first responds. On Windows this races with Setup: WinRM opens briefly during OOBE, the watchdog exits, then Windows reboots or powers off for a later phase. With the watchdog gone the powered-off VM is never restarted and Packer waits out its full 4h timeout with no route to host, finishing with no artifact. Require the port to stay up for several consecutive checks (~30s) before standing down; a transient blip resets the counter and the watchdog keeps restarting the VM on power-off until the port is stably up (Packer connected and provisioning). Fixes the intermittent Windows builds that produced no template.
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.

1 participant