Skip to content

Commit f7314ab

Browse files
committed
cope with not having permission to set availability zone
1 parent ac55b7a commit f7314ab

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

environments/skeleton/{{cookiecutter.environment}}/terraform/compute/nodes.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ resource "openstack_compute_instance_v2" "compute" {
5050
k3s_server = var.k3s_server
5151
}
5252

53-
availability_zone = var.match_ironic_node ? "${var.availability_zone}::${var.baremetal_nodes[each.key]}" : var.availability_zone
53+
availability_zone = var.match_ironic_node ? "${var.availability_zone}::${var.baremetal_nodes[each.key]}" : null
5454

5555
user_data = <<-EOF
5656
#cloud-config

environments/skeleton/{{cookiecutter.environment}}/terraform/compute/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ variable "match_ironic_node" {
8585
}
8686

8787
variable availability_zone {
88-
description = "Name of availability zone. NB using ZONE:HOST or ZONE::NODE is not supported if setting match_ironic_node"
88+
description = "Name of availability zone - ignored unless match_ironic_node is true"
8989
type = string
9090
default = "nova"
9191
}

environments/skeleton/{{cookiecutter.environment}}/terraform/nodes.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ resource "openstack_compute_instance_v2" "login" {
148148
k3s_server = [for n in openstack_compute_instance_v2.control["control"].network: n.fixed_ip_v4 if n.access_network][0]
149149
}
150150

151-
availability_zone = each.value.match_ironic_node ? "${each.value.availability_zone}::${data.external.baremetal_nodes.result[each.key]}" : each.value.availability_zone
151+
availability_zone = each.value.match_ironic_node ? "${each.value.availability_zone}::${data.external.baremetal_nodes.result[each.key]}" : null
152152

153153
user_data = <<-EOF
154154
#cloud-config

environments/skeleton/{{cookiecutter.environment}}/terraform/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,8 @@ variable "login_nodes" {
3636
Required:
3737
flavor: String flavor name
3838
Optional:
39-
availability_zone: String, name of availability zone. NB using ZONE:HOST or ZONE::NODE is not supported if setting match_ironic_node
4039
match_ironic_node: Bool, whether to launch instances on the Ironic node of the same name as this cluster node
40+
availability_zone: String, name of availability zone - ignored unless match_ironic_node is true
4141
EOF
4242
}
4343

0 commit comments

Comments
 (0)