add ability to set Proxmox HA options (#23)

Author: glitchcrab

.gitignore

@@ -0,0 +1,2 @@
+.terraform.lock.hcl
+.terraform/

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+
+- Ability to set Proxmox HA options.
+
 ## [1.3.0] - 2022-12-03
 
 ### Added

README.md

@@ -11,24 +11,23 @@ This module is an opinionated take on creating a VM in Proxmox; not all possible
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
 | boot | Boot order for the instance. | `string` | `"cdn"` | no |
-| cicustom | Path(s) to cloud-init config files (ignored when pxe_boot is true). | `string` | `""` | no |
-| citemplate_storage | Name of the storage containing the cloud-init snippets (ignored when pxe_boot is true). | `string` | `"local"` | no |
-| clone | Name of the template to clone (ignored when pxe_boot is true). | `string` | `""` | no |
-| cloudinit_cdrom_storage | Name of the storage to create the cloud-init image in. | `string` | `"local-lvm"` | no |
+| cicustom | Path(s) to cloud-init config files (ignored when pxe_boot is true). | `string` | `null` | no |
+| citemplate_storage | Name of the storage containing the cloud-init snippets (ignored when pxe_boot is true). | `string` | `null` | no |
+| clone | Name of the template to clone (ignored when pxe_boot is true). | `string` | `null` | no |
+| cloudinit_cdrom_storage | Name of the storage to create the cloud-init image in (e.g. local-lvm). | `string` | `null` | no |
 | cores | Number of cores to allocate. | `number` | n/a | yes |
 | disks | List of objects representing additional disks. | <pre>list(object({<br>    type    = string<br>    storage = string<br>    size    = string<br>  }))</pre> | `null` | no |
-| full_clone | Create a full clone; if false, a linked clone will be created (ignored when pxe_boot is true). | `bool` | `false` | no |
-| instance_domain | Domain name to use. | `string` | n/a | yes |
+| full_clone | Create a full clone; if false, a linked clone will be created (ignored when pxe_boot is true). | `bool` | `null` | no |
+| hagroup | The HA group identifier the resource belongs to. | `string` | `null` | no |
+| hastate | Requested HA state for the resource. | `string` | `null` | no |
 | memory | Amount of memory to allocate. | `number` | n/a | yes |
-| nameserver | Nameserver to use in the instance. | `string` | n/a | yes |
 | network_interfaces | List of objects representing instance interface configuration. | <pre>list(object({<br>    model   = string<br>    bridge  = string<br>    tag     = number<br>    macaddr = string<br>  }))</pre> | n/a | yes |
-| os_type | Type of OS for preprovisioning. | `string` | `""` | no |
+| os_type | Type of OS for preprovisioning. | `string` | `null` | no |
 | pve_instance_description | Description of the instance. | `string` | n/a | yes |
 | pve_instance_name | Name of the instance. | `string` | n/a | yes |
 | pxe_boot | Set PXE boot mode | `bool` | `false` | no |
 | qemu_agent | Enable QEMU guest agent (must be installed in the template). Set to `1` to enable or `0` to disable. | `number` | `0` | no |
 | resource_pool | Name of the resource pool the assign the instance to. | `string` | n/a | yes |
-| searchdomain | DNS searchdomain to use. | `string` | n/a | yes |
 | snippet_dir | Name of the snippet subdirectory. | `string` | `"snippets"` | no |
 | snippet_file_base | Path to the directory containing the snippet_dir. | `string` | `"/var/lib/vz"` | no |
 | sockets | Number of sockets to allocate. | `number` | n/a | yes |

instance.tf

@@ -13,6 +13,9 @@ resource "proxmox_vm_qemu" "proxmox_instance" {
   target_node = var.target_node
   pool        = var.resource_pool
 
+  hastate = var.hastate
+  hagroup = var.hagroup
+
   cores   = var.cores
   sockets = var.sockets
   memory  = var.memory

variables.tf

@@ -41,6 +41,29 @@ variable "resource_pool" {
   type        = string
 }
 
+variable "hastate" {
+  default     = null
+  description = "Requested HA state for the resource."
+  type        = string
+  validation {
+    condition = anytrue([
+      var.hastate == "disabled",
+      var.hastate == "enabled",
+      var.hastate == "ignored",
+      var.hastate == "started",
+      var.hastate == "stopped",
+      var.hastate == null
+    ])
+    error_message = "Must be one of 'disabled', 'enabled', 'ignored', 'started', 'stopped', or null (default)."
+  }
+}
+
+variable "hagroup" {
+  default     = null
+  description = "The HA group identifier the resource belongs to."
+  type        = string
+}
+
 variable "cores" {
   description = "Number of cores to allocate."
   type        = number