terraform: upgrade proxmox provider to last release
Closed, MigratedEdits Locked
Actions

Assigned To

Authored By

	vsellier
	Dec 14 2021, 6:28 PM

Description

The proxmox provider is now "supported" and regularly released. [1]

We should upgrade our configuration to migrate to the last version

a quick test show there will be some adaptation to do:

~/src/swh/swh-sysadmin-provisioning/proxmox/terraform/staging master* ❯ terraform plan                                                                                                                                     18:22:32
╷
│ Error: Unsupported argument
│ 
│   on ../modules/node/main.tf line 60, in resource "proxmox_vm_qemu" "node":
│   60:       id           = disk.value["id"]
│ 
│ An argument named "id" is not expected here. Did you mean "ssd"?
....

[1] https://registry.terraform.io/providers/Telmate/proxmox/latest/docs

Revisions and Commits

rSPRE sysadm-provisioning
	D6840	rSPRE5cf38552a3b5 terraform: update the proxmox provider

Related Objects

Mentioned In: T3795: update terraform recipe to configure the vm's cpu type to default and support onboot option

Event Timeline

vsellier changed the task status from Open to Work in Progress.Dec 14 2021, 6:28 PM

vsellier triaged this task as Normal priority.

vsellier created this task.

vsellier mentioned this in T3795: update terraform recipe to configure the vm's cpu type to default and support onboot option.Dec 14 2021, 6:30 PM

vsellier updated the task description. (Show Details)Dec 14 2021, 6:32 PM

After some adapations, the syntax is now good.

One of the hardest problem to find due to the lack of context was this error when a terraform command is executed:

│ Error: number is required
│ 
│   with module.webapp1.proxmox_vm_qemu.node,
│   on ../modules/node/main.tf line 1, in resource "proxmox_vm_qemu" "node":
│    1: resource "proxmox_vm_qemu" "node" {
│

It was due to a configuration format change : https://github.com/Telmate/terraform-provider-proxmox/commit/facf1c0432f4a8b5e553544e8f9d16e9c7687426

The boolean values need to be converted to int
This sed do the job:

sed  -i -e 's/"backup": false/"backup": 0/' -e 's/"iothread": false/"iothread": 0/' -e 's/"replicate": false/"replicate": 0/' -e 's/"ssd": false/"ssd": 0/' terraform.tfstate

The states can now be updated to avoid any change detection:

# module.worker18.proxmox_vm_qemu.node will be updated in-place
~ resource "proxmox_vm_qemu" "node" {
    + additional_wait           = 0
    ~ clone_wait                = 15 -> 0
    + guest_agent_ready_timeout = 100
      id                        = "uffizi/qemu/141"
      name                      = "worker18"
    + oncreate                  = true
      # (37 unchanged attributes hidden)


      # (2 unchanged blocks hidden)
  }

vsellier added a revision: D6840: terraform: update the proxmox provider.Dec 15 2021, 12:55 PM

vsellier added a commit: rSPRE5cf38552a3b5: terraform: update the proxmox provider.Dec 15 2021, 3:20 PM

proxmox provider updated to v2.9.3

This task has been migrated to GitLab.

terraform: upgrade proxmox provider to last releaseClosed, MigratedEdits LockedActions

Description

Revisions and Commits

Related Objects

Event Timeline

terraform: upgrade proxmox provider to last release
Closed, MigratedEdits Locked
Actions