Skip to content

Commit 409aa5b

Browse files
fix/istio-base-override-values (#145)
Co-authored-by: CloudDrove CI <84795582+clouddrove-ci@users.noreply.github.com>
1 parent 354fa21 commit 409aa5b

File tree

3 files changed

+5
-12
lines changed

3 files changed

+5
-12
lines changed

addons/istio-ingress/locals.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
locals {
22
istio_base = {
3-
helm_config = {
3+
default_helm_config = {
44
name = try(var.istio_ingress_extra_configs.istiobase_release_name, "base")
55
chart = "base"
66
repository = "https://istio-release.storage.googleapis.com/charts"
@@ -63,11 +63,11 @@ locals {
6363

6464
istiod_helm_config = merge(
6565
local.istiod.default_helm_config,
66-
var.istio_ingress_extra_configs.istiod
66+
try(var.istio_ingress_extra_configs.istiod, {})
6767
)
6868

6969
istio_base_helm_config = merge(
70-
local.istio_base.helm_config,
71-
var.istio_ingress_extra_configs.istio_base
70+
local.istio_base.default_helm_config,
71+
try(var.istio_ingress_extra_configs.istio_base, {})
7272
)
7373
}

addons/istio-ingress/main.tf

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ module "istio_base" {
33
count = try(var.istio_ingress_extra_configs.install_istiobase, true) ? 1 : 0
44

55
manage_via_gitops = var.manage_via_gitops
6-
helm_config = local.istio_base.helm_config
6+
helm_config = local.istio_base_helm_config
77
addon_context = var.addon_context
88

99
}
@@ -40,4 +40,3 @@ resource "kubectl_manifest" "istio_gateway_manifest" {
4040
depends_on = [kubectl_manifest.istio_ingress_manifest]
4141
yaml_body = file(var.istio_manifests.istio_gateway_manifest_file_path[count.index])
4242
}
43-

addons/istio-ingress/variables.tf

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,3 @@ variable "istio_ingress_extra_configs" {
3737
type = any
3838
default = {}
3939
}
40-
41-
variable "istiod_helm_config" {
42-
description = "Path to override-values.yaml for Istiod Helm Chart"
43-
type = any
44-
default = null
45-
}

0 commit comments

Comments
 (0)