Disable MockTelemetryPlugin by default for integ tests (#12877) (#12901)

opensearch-trigger-bot[bot] · github-actions[bot] · web-flow · commit bdcf14e6bf29 · 2024-03-25T14:07:35.000-04:00
With this plugin enabled we see significant increase in memory usage. This is due to a static map that collects Span entries per write/replication request. Tests with higher counts of independent writes are at risk of running oom or experiencing slow down / warnings related to this. Disabled the plugin by default until this is resolved given it is still under FeatureFlag. (cherry picked from commit 52ae79d) Signed-off-by: Marc Handalian <marc.handalian@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
diff --git a/test/framework/src/main/java/org/opensearch/test/OpenSearchIntegTestCase.java b/test/framework/src/main/java/org/opensearch/test/OpenSearchIntegTestCase.java
@@ -2112,7 +2112,8 @@ protected boolean addMockGeoShapeFieldMapper() {
      * @return boolean.
      */
     protected boolean addMockTelemetryPlugin() {
-        return true;
+        // setting to false until https://github.com/opensearch-project/OpenSearch/issues/12615 is resolved
+        return false;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -2112,7 +2112,8 @@ protected boolean addMockGeoShapeFieldMapper() {`
`2112`	`2112`	`* @return boolean.`
`2113`	`2113`	`*/`
`2114`	`2114`	`protected boolean addMockTelemetryPlugin() {`
`2115`		`- return true;`
	`2115`	`+ // setting to false until https://github.com/opensearch-project/OpenSearch/issues/12615 is resolved`
	`2116`	`+ return false;`
`2116`	`2117`	`}`
`2117`	`2118`
`2118`	`2119`	`/**`