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

Support Start/Shutdown Orders #392

Closed
yoshikakbudto opened this issue Jun 28, 2023 · 2 comments · Fixed by #445
Closed

Support Start/Shutdown Orders #392

yoshikakbudto opened this issue Jun 28, 2023 · 2 comments · Fixed by #445
Labels
✨ enhancement New feature or request

Comments

@yoshikakbudto
Copy link

yoshikakbudto commented Jun 28, 2023

Running Proxmox 7.4

I use pulumi package that is using under the hood this provider's latest version to configure the Proxmox cluster.

And i've noticed that the startup clause in qemu.conf either not supported or treated wrong.
To be specific: here is how the parameter startup looks like in qemu config file:

# grep "startup" /etc/pve/qemu-server/100.conf
startup: down=400

It was configured from a Proxmox UI here: VM/CT => Options => "Start/Shutdown Order".

Then i do a pulumi refresh that should match the real cluster state with the current state file, and got the error from provider:

* error retrieving VM: failed to decode HTTP GET response (path: nodes/pve-b2/qemu/100/config) - Reason: json: cannot unmarshal string into Go struct field GetResponseData.data.startup of type vms.CustomStartupOrder
@bpg
Copy link
Owner

bpg commented Jun 29, 2023

Hey @yoshikakbudto! 👋🏼

Thanks for the report. The startup order is not fully supported at the moment. It was added to the original PVE API client a while back, but was not actually exposed in the TF VM resource. I believe the API has changed a bit since then, and become incompatible with the client, so it throws an error you see.

@bpg
Copy link
Owner

bpg commented Jul 20, 2023

The error was reported separately in #421, and then fixed in #428, this ticket is to implement the full startup order support in the VM resource.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants