update readme [skip ci]

kaiohenricunha · Jan 25, 2024 · cc27114 · cc27114
2 parents bd95fe8 + 58603c9
commit cc27114
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 12 deletions.
diff --git a/.github/workflows/destroy-workflow.yaml b/.github/workflows/destroy-workflow.yaml
@@ -62,13 +62,6 @@ jobs:
         cd environments/lab
         terraform destroy -target=helm_release.aws_load_balancer_controller -auto-approve
 
-    # Destroy Karpenter and its dependencies first
-    - name: Terraform Destroy Karpenter
-      if: contains(github.event.inputs.terraform_components, 'karpenter')
-      run: |
-        cd environments/lab
-        terraform destroy -target=module.eks_fargate_karpenter.module.karpenter -auto-approve
-
     - name: Terraform Destroy RDS
       if: contains(github.event.inputs.terraform_components, 'rds')
       run: |
@@ -81,6 +74,13 @@ jobs:
         cd environments/lab
         terraform destroy -target=module.security_group -auto-approve
 
+    # Destroy Karpenter and its dependencies first
+    - name: Terraform Destroy Karpenter
+      if: contains(github.event.inputs.terraform_components, 'karpenter')
+      run: |
+        cd environments/lab
+        terraform destroy -target=module.eks_fargate_karpenter.module.karpenter -auto-approve
+
     - name: Terraform Destroy EKS
       if: contains(github.event.inputs.terraform_components, 'eks')
       run: |

diff --git a/infra/eks-fargate-karpenter/main.tf b/infra/eks-fargate-karpenter/main.tf
@@ -113,11 +113,11 @@ resource "helm_release" "karpenter" {
   create_namespace = true
 
   name                = "karpenter"
-  repository          = "oci://public.ecr.aws/karpenter/karpenter"
+  repository          = "oci://public.ecr.aws/karpenter"
   repository_username = data.aws_ecrpublic_authorization_token.token.user_name
   repository_password = data.aws_ecrpublic_authorization_token.token.password
   chart               = "karpenter"
-  version             = "v0.33.0"
+  version             = "v0.32.0"
 
   values = [
     <<-EOT

diff --git a/stack/metabase/metabase-hpa.yaml b/stack/metabase/metabase-hpa.yaml
@@ -18,10 +18,11 @@ spec:
   - type: prometheus
     metadata:
       serverAddress: http://prometheus-operated.monitoring:9090
-      metricName: istio_requests_total
+      metricName: requests_per_second
+      # (rate) will give us the number of requests per second within a 2 minute window. sum() will add add data from all pods.
       query: |
-        sum(istio_requests_total{destination_workload="metabase"})
-      threshold: "1000"
+        sum(rate(istio_requests_total{destination_workload="metabase"}[2m]))
+      threshold: "100"
   - type: memory
     metricType: Utilization
     metadata: