update: remove enable-compression config temporarily (#1785)

alibaba · Oct 8, 2024 · 6526c34 · 6526c34
1 parent bf7de40
commit 6526c34
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/core/prometheus/schedulers/ScrapeConfig.cpp b/core/prometheus/schedulers/ScrapeConfig.cpp
@@ -43,9 +43,9 @@ bool ScrapeConfig::Init(const Json::Value& scrapeConfig) {
 
     if (scrapeConfig.isMember(prometheus::ENABLE_COMPRESSION)
         && scrapeConfig[prometheus::ENABLE_COMPRESSION].isBool()) {
-        InitEnableCompression(scrapeConfig[prometheus::ENABLE_COMPRESSION].asBool());
+        // InitEnableCompression(scrapeConfig[prometheus::ENABLE_COMPRESSION].asBool());
     } else {
-        InitEnableCompression(true);
+        // InitEnableCompression(true);
     }
 
     // basic auth, authorization, oauth2

diff --git a/core/unittest/prometheus/ScrapeConfigUnittest.cpp b/core/unittest/prometheus/ScrapeConfigUnittest.cpp
@@ -111,7 +111,7 @@ void ScrapeConfigUnittest::TestInit() {
                       "application/openmetrics-text;version=0.0.1;q=0.2,*/*;q=0.1");
 
     // disable compression
-    APSARA_TEST_EQUAL(scrapeConfig.mRequestHeaders["Accept-Encoding"], "identity");
+    // APSARA_TEST_EQUAL(scrapeConfig.mRequestHeaders["Accept-Encoding"], "identity");
 
     // basic auth
     APSARA_TEST_EQUAL(scrapeConfig.mRequestHeaders["Authorization"], "Basic dGVzdF91c2VyOnRlc3RfcGFzc3dvcmQ=");
@@ -383,7 +383,7 @@ void ScrapeConfigUnittest::TestEnableCompression() {
     APSARA_TEST_TRUE(ParseJsonTable(configStr, config, errorMsg));
     scrapeConfig.mRequestHeaders.clear();
     APSARA_TEST_TRUE(scrapeConfig.Init(config));
-    APSARA_TEST_EQUAL("gzip", scrapeConfig.mRequestHeaders["Accept-Encoding"]);
+    // APSARA_TEST_EQUAL("gzip", scrapeConfig.mRequestHeaders["Accept-Encoding"]);
 
     // disable
     configStr = R"JSON({
@@ -397,7 +397,7 @@ void ScrapeConfigUnittest::TestEnableCompression() {
     APSARA_TEST_TRUE(ParseJsonTable(configStr, config, errorMsg));
     scrapeConfig.mRequestHeaders.clear();
     APSARA_TEST_TRUE(scrapeConfig.Init(config));
-    APSARA_TEST_EQUAL("identity", scrapeConfig.mRequestHeaders["Accept-Encoding"]);
+    // APSARA_TEST_EQUAL("identity", scrapeConfig.mRequestHeaders["Accept-Encoding"]);
 
     // enable
     configStr = R"JSON({
@@ -411,7 +411,7 @@ void ScrapeConfigUnittest::TestEnableCompression() {
     APSARA_TEST_TRUE(ParseJsonTable(configStr, config, errorMsg));
     scrapeConfig.mRequestHeaders.clear();
     APSARA_TEST_TRUE(scrapeConfig.Init(config));
-    APSARA_TEST_EQUAL("gzip", scrapeConfig.mRequestHeaders["Accept-Encoding"]);
+    // APSARA_TEST_EQUAL("gzip", scrapeConfig.mRequestHeaders["Accept-Encoding"]);
 }
 
 UNIT_TEST_CASE(ScrapeConfigUnittest, TestInit);