fix(lxc): add missing onboot param on container clone create

[rafsaf]

Contributor's Note

• I have added / updated documentation in /docs for any user-facing features or additions.
• I have added / updated templates in /example for any new or updated resources / data sources.
• I have ran make example to verify that the change works as expected.

Proof of Work

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Closes #794

[bpg]

It looks like PVE does not have start attribute in the update request, see PUT method in the API doc. When running examples I'm getting:

│ Error: error updating container: received an HTTP 400 response - Reason: Parameter verification failed. (start: property is not defined in schema and the schema does not allow additional properties)
│ 
│   with proxmox_virtual_environment_container.example,
│   on resource_virtual_environment_container.tf line 59, in resource "proxmox_virtual_environment_container" "example":
│   59: resource "proxmox_virtual_environment_container" "example" {
│ 

Apparently the UpdateRequestBody type definition is wrong, "start on boot" for update request is called onboot 🤦🏼

[rafsaf]

I looked at the API for a moment and didn't check it as I was pretty sure this is correct, I will try to fix it with proper testing later

[rafsaf]

ok, onboot was correct there but start was unnecessary there

now ok

[bpg]

Thanks for the fix @rafsaf!

LGTM! 🚀