Startup/Shutdown options require the up_delay and down_delay to be set up

[yoshikakbudto]

Startup/Shutdown options require the up_delay and down_delay to be set up

[muhlba91]

can you please verify, and open an issue for this in the upstream provider? (https://github.com/bpg/terraform-provider-proxmox)

this resource takes the terraform provider and maps its definition to pulumi resources. if this mapping leads to such behaviour it‘s either an upstream requirement/issue or is autogenerated errornously through the pulumi generator. so far, alls of these minor mapping issues have been upstream issues though.

[yoshikakbudto]

I'v tried to configure a vm resource directly with the upstream provider, the vm creates with no issues:
the upstream provider version is 0.28.0

resource "proxmox_virtual_environment_vm" "test_node" {
  startup {
    order      = "3"
  //  up_delay  and  down_delay will have a default value
  }
}

terraform apply:

...
      + startup {
          + down_delay = -1
          + order      = 3
          + up_delay   = -1
        }

and the resulting VM has values as expected:

If i configure vm resource with pulumi with the running pulumi versions:
pulumi==3.78.0
pulumi-proxmoxve==5.10.0

proxmox.vm.VirtualMachine(
   startup=proxmox.vm.VirtualMachineStartupArgs(order=3),
...

the pulumi up throws error:

    error: proxmoxve:VM/virtualMachine:VirtualMachine resource 'test-vm' has a problem: expected up_delay to be at least (0), got -1. Examine values at 'test-vm.startup.upDelay'.
    error: proxmoxve:VM/virtualMachine:VirtualMachine resource 'test-vm' has a problem: expected down_delay to be at least (0), got -1. Examine values at 'test-vm.startup.downDelay'.

[muhlba91]

i have tested the startup order with the newest version, and the default values are the same as in Terraform: -1.

i have tested this with the newest Proxmox version. maybe there's a difference in Proxmox between versions?

btw. because of the way Pulumi computes the diff, it will take the plan generated by Terraform.
hence, you need to either set these values to the same (-1) or add them to the ignoreDifferences parameter if you don't want to have an "update" every time.

[muhlba91]

closing as it is fixed upstream.