Skip to content

Commit 98b246f

Browse files
robo-caphyder
authored andcommitted
Support cloud-init update for cluster networks and instancepools. Fix IAM prerequisite dependency. Fix workers output
1 parent 3bdf976 commit 98b246f

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

module-iam.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@ module "iam_cluster_prerequisites" {
7777
operator_volume_kms_key_id = var.operator_volume_kms_key_id
7878
worker_volume_kms_key_id = var.worker_volume_kms_key_id
7979

80-
autoscaler_compartments = local.autoscaler_compartments
81-
worker_compartments = local.worker_compartments
80+
autoscaler_compartments = []
81+
worker_compartments = []
8282

8383
providers = {
8484
oci.home = oci.home

modules/workers/instanceconfig.tf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,6 @@ resource "oci_core_instance_configuration" "workers" {
161161
create_before_destroy = true
162162
ignore_changes = [
163163
defined_tags, freeform_tags, display_name,
164-
instance_details[0].launch_details[0].metadata,
165164
instance_details[0].launch_details[0].defined_tags,
166165
instance_details[0].launch_details[0].freeform_tags,
167166
instance_details[0].launch_details[0].create_vnic_details[0].defined_tags,

modules/workers/locals.tf

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ locals {
157157
{
158158
"oke.oraclecloud.com/tf.module" = "terraform-oci-oke"
159159
"oke.oraclecloud.com/tf.state_id" = var.state_id
160-
"oke.oraclecloud.com/tf.workspace" = terraform.workspace
160+
# "oke.oraclecloud.com/tf.workspace" = terraform.workspace
161161
"oke.oraclecloud.com/pool.name" = pool_name
162162
"oke.oraclecloud.com/pool.mode" = pool.mode
163163
"oke.oraclecloud.com/cluster_autoscaler" = pool.allow_autoscaler ? "allowed" : "disabled"
@@ -232,11 +232,11 @@ locals {
232232
}
233233

234234
# Maps of worker pool OCI resources by pool name enriched with desired/custom parameters for various modes
235-
worker_node_pools = { for k, v in merge(oci_containerengine_node_pool.tfscaled_workers, oci_containerengine_node_pool.autoscaled_workers) : k => merge(v, lookup(local.worker_pools_final, k, {})) }
236-
worker_virtual_node_pools = { for k, v in oci_containerengine_virtual_node_pool.workers : k => merge(v, lookup(local.worker_pools_final, k, {})) }
237-
worker_instance_pools = { for k, v in merge(oci_core_instance_pool.tfscaled_workers, oci_core_instance_pool.autoscaled_workers) : k => merge(v, lookup(local.worker_pools_final, k, {})) }
238-
worker_cluster_networks = { for k, v in oci_core_cluster_network.workers : k => merge(v, lookup(local.worker_pools_final, k, {})) }
239-
worker_instances = { for k, v in oci_core_instance.workers : k => merge(v, lookup(local.worker_pools_final, k, {})) }
235+
worker_node_pools = { for k, v in merge(oci_containerengine_node_pool.tfscaled_workers, oci_containerengine_node_pool.autoscaled_workers) : k => merge(lookup(local.worker_pools_final, k, {}), v) }
236+
worker_virtual_node_pools = { for k, v in oci_containerengine_virtual_node_pool.workers : k => merge(lookup(local.worker_pools_final, k, {}), v) }
237+
worker_instance_pools = { for k, v in merge(oci_core_instance_pool.tfscaled_workers, oci_core_instance_pool.autoscaled_workers) : k => merge(lookup(local.worker_pools_final, k, {}), v) }
238+
worker_cluster_networks = { for k, v in oci_core_cluster_network.workers : k => merge(lookup(local.worker_pools_final, k, {}), v) }
239+
worker_instances = { for k, v in oci_core_instance.workers : k => merge(lookup(local.worker_pools_final, k, {}), v) }
240240

241241
# Combined map of outputs by pool name for all modes excluding 'instance' (output separately)
242242
worker_pools_output = merge(

0 commit comments

Comments
 (0)