From f921543e4b7593071c1ebdd9778ac94cc44e2bcc Mon Sep 17 00:00:00 2001
From: Moad Zardab <mzardab@redhat.com>
Date: Mon, 16 Dec 2024 14:10:02 +0400
Subject: [PATCH] Revert "Adding compactor concurrency flags (#710)"

This reverts commit c13eb8bf9cf045c0bbdabc502f60931c9a7b4a26.
---
 jsonnetfile.lock.json                                       | 2 +-
 resources/services/observatorium-metrics-template.yaml      | 6 ------
 .../observatorium-metrics-template-overwrites.libsonnet     | 6 +-----
 services/observatorium-metrics-template.jsonnet             | 2 --
 4 files changed, 2 insertions(+), 14 deletions(-)

diff --git a/jsonnetfile.lock.json b/jsonnetfile.lock.json
index cc15931ead..b58d384d8b 100644
--- a/jsonnetfile.lock.json
+++ b/jsonnetfile.lock.json
@@ -230,7 +230,7 @@
           "subdir": "configuration"
         }
       },
-      "version": "e59b139d57c7d5f60dbc43c061257d5139032e4b",
+      "version": "e743a6abc673774e60bf37b94e6454e542db46f0",
       "sum": "AuUPEXXfd/hxMbKWdURRKaCTBPcTJD4cs9XApBIZbiA="
     },
     {
diff --git a/resources/services/observatorium-metrics-template.yaml b/resources/services/observatorium-metrics-template.yaml
index dcef4a3334..3ff10b770f 100644
--- a/resources/services/observatorium-metrics-template.yaml
+++ b/resources/services/observatorium-metrics-template.yaml
@@ -144,8 +144,6 @@ objects:
           - --downsample.concurrency=1
           - --deduplication.replica-label=replica
           - --debug.max-compaction-level=3
-          - --compact.concurrency=${THANOS_COMPACTOR_COMPACT_CONCURRENCY}
-          - --downsample.concurrency=${THANOS_COMPACTOR_DOWNSAMPLE_CONCURRENCY}
           - ${THANOS_COMPACTOR_RETENTION_DISABLE_DOWNSAMPLING}
           env:
           - name: OBJSTORE_CONFIG
@@ -4455,10 +4453,6 @@ parameters:
   value: 50Gi
 - name: THANOS_COMPACTOR_REPLICAS
   value: "1"
-- name: THANOS_COMPACTOR_COMPACT_CONCURRENCY
-  value: "1"
-- name: THANOS_COMPACTOR_DOWNSAMPLE_CONCURRENCY
-  value: "1"
 - name: THANOS_CONFIG_SECRET
   value: thanos-objectstorage
 - name: THANOS_IMAGE_TAG
diff --git a/services/observatorium-metrics-template-overwrites.libsonnet b/services/observatorium-metrics-template-overwrites.libsonnet
index b03e2c6388..b623bd1f87 100644
--- a/services/observatorium-metrics-template-overwrites.libsonnet
+++ b/services/observatorium-metrics-template-overwrites.libsonnet
@@ -92,11 +92,7 @@ local thanosRuleSyncer = import './sidecars/thanos-rule-syncer.libsonnet';
                   // Temporary workaround on high cardinality blocks for 2w.
                   // Since we have only 2w retention, there is no point in having 2w blocks.
                   // See: https://issues.redhat.com/browse/OBS-437
-                  args+: [
-                    '--debug.max-compaction-level=3',
-                    '--compact.concurrency=${THANOS_COMPACTOR_COMPACT_CONCURRENCY}',
-                    '--downsample.concurrency=${THANOS_COMPACTOR_DOWNSAMPLE_CONCURRENCY}',
-                  ] + disableDownsamplingFlag,
+                  args+: ['--debug.max-compaction-level=3'] + disableDownsamplingFlag,
                 } else c
                 for c in super.containers
               ],
diff --git a/services/observatorium-metrics-template.jsonnet b/services/observatorium-metrics-template.jsonnet
index 2153bad8ca..b0b588f7d3 100644
--- a/services/observatorium-metrics-template.jsonnet
+++ b/services/observatorium-metrics-template.jsonnet
@@ -76,8 +76,6 @@ local obs = import 'observatorium.libsonnet';
     { name: 'THANOS_COMPACTOR_MEMORY_REQUEST', value: '1Gi' },
     { name: 'THANOS_COMPACTOR_PVC_REQUEST', value: '50Gi' },
     { name: 'THANOS_COMPACTOR_REPLICAS', value: '1' },
-    { name: 'THANOS_COMPACTOR_COMPACT_CONCURRENCY', value: '1' },
-    { name: 'THANOS_COMPACTOR_DOWNSAMPLE_CONCURRENCY', value: '1' },
     { name: 'THANOS_CONFIG_SECRET', value: 'thanos-objectstorage' },
     { name: 'THANOS_IMAGE_TAG', value: 'v0.36.0' },
     { name: 'THANOS_IMAGE', value: 'quay.io/thanos/thanos' },