Merge pull request #31 from mila-iqia/feat/change_update

feat: Allow to update VMs with new values
mila-iqia · Mar 28, 2023 · eddf7f3 · eddf7f3
2 parents d45171f + 2a500cc
commit eddf7f3
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/roles/vm/README.md b/roles/vm/README.md
@@ -34,3 +34,7 @@ By default, the proxmox modules in `community.general` do not validate SSL
 certs. To enable certificate validation, define the variable:
 
  - `proxmox_api_validate_certs: true`
+
+To enable update of VMs with new values:
+
+ - `proxmox_kvm_update: true`
diff --git a/roles/vm/defaults/main.yml b/roles/vm/defaults/main.yml
@@ -5,3 +5,6 @@
 # certs. Make this a default for use with other modules like
 # `ansible.builtin.uri` (used for API calls)
 proxmox_api_validate_certs: false
+
+# Do not update the VM when proxmox_kvm runs
+proxmox_kvm_update: false
diff --git a/roles/vm/tasks/main.yml b/roles/vm/tasks/main.yml
@@ -17,7 +17,12 @@
     scsi: "{{ item.scsi | default(omit) }}"
     scsihw: "{{ item.scsihw | default(omit) }}"
     tags: "{{ item.tags | default(omit) }}"
-    update: "{{ item.refresh | default(omit) }}"
+    update: "
+      {% if 'update' in item %}
+      {{ item['update'] }}
+      {% else %}
+      {{ proxmox_kvm_update }}
+      {% endif %}"
     validate_certs: "{{ proxmox_api_validate_certs }}"
     virtio: "{{ item.virtio | default(omit) }}"
     vmid: "{{ item.vmid }}"