fixed tests, revert unintended changes

Author: milldr

src/notifications.tf

@@ -219,22 +219,22 @@ locals {
       if key != "ssm_path_prefix" && key != "webhook"
     },
     {
-      for key, value in coalesce(local.notifications_notifiers.webhook, {}) :
+      for key, value in lookup(local.notifications_notifiers, "webhook", {}) :
       format("webhook_%s", key) =>
       { for param_name, param_value in value : param_name => param_value if param_value != null }
     }
   )
 
   ## Get paths to read configs for each notifier service
-  notifications_notifiers_ssm_path = merge(
+  notifications_notifiers_ssm_path = local.enabled ? merge(
     {
       for key, value in local.notifications_notifiers_variables :
       key => format("%s/%s/", local.notifications_notifiers.ssm_path_prefix, key)
     },
     {
       common = format("%s/common/", local.notifications_notifiers.ssm_path_prefix)
     },
-  )
+  ) : {}
 
   ## Read SSM secrets into object for each notifier service
   notifications_notifiers_ssm_configs = {

src/provider-helm.tf

@@ -133,9 +133,8 @@ locals {
     "--profile", var.kube_exec_auth_aws_profile
   ] : []
 
-  kube_exec_auth_role_arn = coalesce(var.kube_exec_auth_role_arn, module.iam_roles.terraform_role_arn)
   exec_role = local.kube_exec_auth_enabled && var.kube_exec_auth_role_arn_enabled ? [
-    "--role-arn", local.kube_exec_auth_role_arn
+    "--role-arn", coalesce(var.kube_exec_auth_role_arn, module.iam_roles.terraform_role_arn)
   ] : []
 
   # Provide dummy configuration for the case where the EKS cluster is not available.

src/remote-state.tf

@@ -1,6 +1,6 @@
 module "eks" {
   source  = "cloudposse/stack-config/yaml//modules/remote-state"
-  version = "1.5.0"
+  version = "1.8.0"
 
   component = var.eks_component_name
 
@@ -9,7 +9,7 @@ module "eks" {
 
 module "dns_gbl_delegated" {
   source  = "cloudposse/stack-config/yaml//modules/remote-state"
-  version = "1.5.0"
+  version = "1.8.0"
 
   environment = "gbl"
   component   = "dns-delegated"
@@ -20,7 +20,7 @@ module "dns_gbl_delegated" {
 module "saml_sso_providers" {
   for_each = local.enabled ? var.saml_sso_providers : {}
   source   = "cloudposse/stack-config/yaml//modules/remote-state"
-  version  = "1.5.0"
+  version  = "1.8.0"
 
   component   = each.value.component
   environment = each.value.environment
@@ -32,7 +32,7 @@ module "argocd_repo" {
   for_each = local.enabled ? var.argocd_repositories : {}
 
   source  = "cloudposse/stack-config/yaml//modules/remote-state"
-  version = "1.5.0"
+  version = "1.8.0"
 
   component   = each.key
   environment = each.value.environment