Skip to content

Commit b5cd4da

Browse files
committed
fix: add missing Keycloak variables to destroy workflow
1 parent 001ed68 commit b5cd4da

File tree

2 files changed

+41
-9
lines changed

2 files changed

+41
-9
lines changed

lgtm-stack/terraform/grafana.tf

Lines changed: 29 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,6 @@ resource "grafana_data_source" "tempo" {
223223
#
224224
# REQUIRES: Grafana OSS with accesscontrol feature flag enabled
225225
# (set GF_FEATURE_TOGGLES_ENABLE: accesscontrol in grafana-values.yaml)
226-
# OR Grafana Enterprise.
227226

228227
resource "grafana_data_source_permission" "loki" {
229228
for_each = toset(var.tenants)
@@ -235,8 +234,8 @@ resource "grafana_data_source_permission" "loki" {
235234
}
236235

237236
depends_on = [
238-
helm_release.grafana,
239-
null_resource.wait_for_grafana
237+
grafana_data_source.loki,
238+
grafana_team.tenants
240239
]
241240
}
242241

@@ -250,8 +249,8 @@ resource "grafana_data_source_permission" "mimir" {
250249
}
251250

252251
depends_on = [
253-
helm_release.grafana,
254-
null_resource.wait_for_grafana
252+
grafana_data_source.mimir,
253+
grafana_team.tenants
255254
]
256255
}
257256

@@ -265,8 +264,8 @@ resource "grafana_data_source_permission" "prometheus" {
265264
}
266265

267266
depends_on = [
268-
helm_release.grafana,
269-
null_resource.wait_for_grafana
267+
grafana_data_source.prometheus,
268+
grafana_team.tenants
270269
]
271270
}
272271

@@ -280,8 +279,8 @@ resource "grafana_data_source_permission" "tempo" {
280279
}
281280

282281
depends_on = [
283-
helm_release.grafana,
284-
null_resource.wait_for_grafana
282+
grafana_data_source.tempo,
283+
grafana_team.tenants
285284
]
286285
}
287286

@@ -301,6 +300,27 @@ resource "grafana_folder" "tenants" {
301300
]
302301
}
303302

303+
# ---- Global Data Sources (Cluster Admins) -----------------------
304+
# These data sources point to the "default" tenant where
305+
# infrastructure metrics and logs are stored.
306+
# Restricted to the grafana-admins team.
307+
308+
resource "grafana_data_source" "global_loki" {
309+
name = "Global-Loki"
310+
type = "loki"
311+
url = "http://monitoring-loki-gateway:80"
312+
http_headers = { "X-Scope-OrgID" = "default" }
313+
depends_on = [helm_release.grafana, null_resource.wait_for_grafana]
314+
}
315+
316+
resource "grafana_data_source" "global_mimir" {
317+
name = "Global-Mimir"
318+
type = "prometheus"
319+
url = "http://monitoring-mimir-nginx:80/prometheus"
320+
http_headers = { "X-Scope-OrgID" = "default" }
321+
depends_on = [helm_release.grafana, null_resource.wait_for_grafana]
322+
}
323+
304324
resource "grafana_folder_permission" "tenants" {
305325
for_each = toset(var.tenants)
306326

lgtm-stack/terraform/main.tf

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -221,6 +221,12 @@ module "cert_manager" {
221221
issuer_namespace = var.namespace
222222
ingress_class_name = var.ingress_class_name
223223

224+
# Pass GKE credentials for module-internal providers
225+
gke_endpoint = var.gke_endpoint
226+
gke_ca_certificate = var.gke_ca_certificate
227+
project_id = var.project_id
228+
region = var.region
229+
224230
# Ensure namespace exists before issuer creation (handled inside module)
225231
}
226232

@@ -233,6 +239,12 @@ module "ingress_nginx" {
233239
release_name = var.nginx_ingress_release_name
234240
namespace = var.nginx_ingress_namespace
235241
ingress_class_name = var.ingress_class_name
242+
243+
# Pass GKE credentials for module-internal providers
244+
gke_endpoint = var.gke_endpoint
245+
gke_ca_certificate = var.gke_ca_certificate
246+
project_id = var.project_id
247+
region = var.region
236248
}
237249

238250
# Loki

0 commit comments

Comments
 (0)