./gradlew spotlessApply

Author: xiang17

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/init/AiConfigCustomizer.java

@@ -118,9 +118,12 @@ public Map<String, String> apply(ConfigProperties otelConfig) {
 
     String metricsExporter = otelConfig.getString("otel.metrics.exporter");
     String aksNamespaceId = System.getenv("AKS_ARM_NAMESPACE_ID");
-    String metricsToLogAnalyticsEnabled = otelConfig.getString("applicationinsights.metrics.to.loganalytics.enabled");
+    String metricsToLogAnalyticsEnabled =
+        otelConfig.getString("applicationinsights.metrics.to.loganalytics.enabled");
     if (isAksAttach(aksNamespaceId)) {
-      properties.put("otel.metrics.exporter", updateMetricsExporter(metricsExporter, metricsToLogAnalyticsEnabled));
+      properties.put(
+          "otel.metrics.exporter",
+          updateMetricsExporter(metricsExporter, metricsToLogAnalyticsEnabled));
     } else if (metricsExporter == null) {
       // this overrides the default "otlp" so the exporter can be configured later
       properties.put("otel.metrics.exporter", "none");
@@ -358,7 +361,8 @@ static boolean isAksAttach(String aksNamespaceId) {
   static String updateMetricsExporter(String metricsExporter, String metricsToLogAnalyticsEnabled) {
     String azureMonitorName = AzureMonitorExporterProviderKeys.EXPORTER_NAME;
     if (metricsExporter == null || metricsExporter.isEmpty()) {
-      if (metricsToLogAnalyticsEnabled == null || Boolean.parseBoolean(metricsToLogAnalyticsEnabled)) {
+      if (metricsToLogAnalyticsEnabled == null
+          || Boolean.parseBoolean(metricsToLogAnalyticsEnabled)) {
         return azureMonitorName + ",otlp";
       } else {
         return azureMonitorName;

agent/agent-tooling/src/test/java/com/microsoft/applicationinsights/agent/internal/init/AiConfigCustomizerTest.java

@@ -6,7 +6,6 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import java.util.stream.Stream;
-
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
@@ -28,44 +27,45 @@ void updateMetricsExporter_ExporterUnset() {
     assertThat(AiConfigCustomizer.updateMetricsExporter(null, null))
         .isEqualTo("azure_monitor,otlp");
 
-    assertThat(AiConfigCustomizer.updateMetricsExporter("", null))
-        .isEqualTo("azure_monitor,otlp");
-    
+    assertThat(AiConfigCustomizer.updateMetricsExporter("", null)).isEqualTo("azure_monitor,otlp");
+
     assertThat(AiConfigCustomizer.updateMetricsExporter(null, "true"))
         .isEqualTo("azure_monitor,otlp");
-    
+
     assertThat(AiConfigCustomizer.updateMetricsExporter(null, "True"))
         .isEqualTo("azure_monitor,otlp");
   }
 
   static Stream<Arguments> stringPairs() {
     return Stream.of(
-        Arguments.of("none",               "none"),
-        Arguments.of("azure_monitor",      "azure_monitor,otlp"),
+        Arguments.of("none", "none"),
+        Arguments.of("azure_monitor", "azure_monitor,otlp"),
         Arguments.of("azure_monitor,otlp", "azure_monitor,otlp"),
-        Arguments.of("otlp",               "otlp"));
-    }
-  
+        Arguments.of("otlp", "otlp"));
+  }
+
   @ParameterizedTest
   @MethodSource("stringPairs")
-  void updateMetricsExporter_ExporterSet_AMLE_Unset(String metricsExporter, String expectAzureMonitor) {
+  void updateMetricsExporter_ExporterSet_AMLE_Unset(
+      String metricsExporter, String expectAzureMonitor) {
     assertThat(AiConfigCustomizer.updateMetricsExporter(metricsExporter, null))
         .isEqualTo(expectAzureMonitor);
     assertThat(AiConfigCustomizer.updateMetricsExporter(metricsExporter, ""))
         .isEqualTo(expectAzureMonitor);
   }
-  
+
   static Stream<Arguments> stringPairs2() {
     return Stream.of(
-        Arguments.of("none",                "azure_monitor,otlp"),
-        Arguments.of("azure_monitor",       "azure_monitor,otlp"),
-        Arguments.of("azure_monitor,otlp",  "azure_monitor,otlp"),
-        Arguments.of("otlp",                "otlp,azure_monitor"));
-    }
-  
+        Arguments.of("none", "azure_monitor,otlp"),
+        Arguments.of("azure_monitor", "azure_monitor,otlp"),
+        Arguments.of("azure_monitor,otlp", "azure_monitor,otlp"),
+        Arguments.of("otlp", "otlp,azure_monitor"));
+  }
+
   @ParameterizedTest
   @MethodSource("stringPairs2")
-  void updateMetricsExporter_ExporterSet_AMLE_True(String metricsExporter, String expectAzureMonitor) {
+  void updateMetricsExporter_ExporterSet_AMLE_True(
+      String metricsExporter, String expectAzureMonitor) {
     assertThat(AiConfigCustomizer.updateMetricsExporter(metricsExporter, "true"))
         .isEqualTo(expectAzureMonitor);
     assertThat(AiConfigCustomizer.updateMetricsExporter(metricsExporter, "True"))
@@ -74,15 +74,16 @@ void updateMetricsExporter_ExporterSet_AMLE_True(String metricsExporter, String
 
   static Stream<Arguments> stringPairs3() {
     return Stream.of(
-        Arguments.of("none",                "none"),
-        Arguments.of("azure_monitor",       "azure_monitor"),
-        Arguments.of("azure_monitor,otlp",  "azure_monitor"),
-        Arguments.of("otlp",                "otlp"));
-    }
-  
+        Arguments.of("none", "none"),
+        Arguments.of("azure_monitor", "azure_monitor"),
+        Arguments.of("azure_monitor,otlp", "azure_monitor"),
+        Arguments.of("otlp", "otlp"));
+  }
+
   @ParameterizedTest
   @MethodSource("stringPairs3")
-  void updateMetricsExporter_ExporterSet_AMLE_False(String metricsExporter, String expectAzureMonitor) {
+  void updateMetricsExporter_ExporterSet_AMLE_False(
+      String metricsExporter, String expectAzureMonitor) {
     assertThat(AiConfigCustomizer.updateMetricsExporter(metricsExporter, "false"))
         .isEqualTo(expectAzureMonitor);
     assertThat(AiConfigCustomizer.updateMetricsExporter(metricsExporter, "False"))