Merge pull request #4 from SCT-Engineering/crashy

hints for s3
SCT-Engineering · Oct 10, 2024 · 72c4c74 · 72c4c74
2 parents 5bb3429 + 027b52b
commit 72c4c74
Showing 1 changed file with 12 additions and 12 deletions.
diff --git a/main.tf b/main.tf
@@ -44,14 +44,14 @@ resource "helm_release" "tempo" {
   values           = [file("${path.module}/tempo_values.yaml")]
 
 
-  # set {
-  #   name  = "tempo.storage.trace.backend"
-  #   value = "s3"
-  # }
-  # set {
-  #   name  = "tempo.storage.trace.s3.insecure"
-  #   value = "false"
-  # }
+  set {
+    name  = "tempo.storage.trace.backend"
+    value = "s3"
+  }
+  set {
+    name  = "tempo.storage.trace.s3.insecure"
+    value = "false"
+  }
   # set {
   #   name  = "replicas"
   #   value = "3"
@@ -71,10 +71,10 @@ resource "helm_release" "tempo" {
     name  = "tempo.storage.trace.s3.bucket"
     value = module.tempo-s3.s3_bucket_id
   }
-  # set {
-  #   name  = "tempo.storage.trace.s3.endpoint"
-  #   value = "s3.${var.region}.amazonaws.com"
-  # }
+  set {
+    name  = "tempo.storage.trace.s3.endpoint"
+    value = "s3.${var.region}.amazonaws.com"
+  }
   set {
     name  = "tempo.metricsGenerator.remoteWriteUrl"
     value = "http://${var.prometheus_svc}.${var.prometheus_namespace}:${var.prometheus_port}/api/v1/write"