Merge branch 'main' into update-otel

Author: xiang17

agent/agent-bootstrap/gradle.lockfile

@@ -1,8 +1,8 @@
 # This is a Gradle generated file for dependency locking.
 # Manual edits can break the build and are not advised.
 # This file is expected to be part of source control.
-com.azure:azure-sdk-bom:1.3.0=runtimeClasspath
-com.fasterxml.jackson:jackson-bom:2.20.0=runtimeClasspath
+com.azure:azure-sdk-bom:1.3.2=runtimeClasspath
+com.fasterxml.jackson:jackson-bom:2.20.1=runtimeClasspath
 io.netty:netty-bom:4.2.7.Final=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom-alpha:2.20.0-alpha=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom:2.20.0=runtimeClasspath

agent/agent-bootstrap/src/main/java/com/microsoft/applicationinsights/agent/bootstrap/BytecodeUtil.java

@@ -18,7 +18,7 @@
 import javax.annotation.Nullable;
 
 // supporting all properties of event, metric, remote dependency and page view telemetry
-@SuppressWarnings("TooManyParameters")
+@SuppressWarnings("TooManyParameters") // parameter count justified by method complexity
 public class BytecodeUtil {
 
   private static BytecodeUtilDelegate delegate;
@@ -324,7 +324,7 @@ public static void onEnter() {
     startNanosHolder.set(System.nanoTime());
   }
 
-  @SuppressWarnings("SystemOut")
+  @SuppressWarnings("SystemOut") // legitimate use of System.out for logging/output
   public static void onExit() {
     Long startNanos = startNanosHolder.get();
     if (startNanos == null) {

agent/agent-for-testing/gradle.lockfile

@@ -1,8 +1,8 @@
 # This is a Gradle generated file for dependency locking.
 # Manual edits can break the build and are not advised.
 # This file is expected to be part of source control.
-com.azure:azure-sdk-bom:1.3.0=runtimeClasspath
-com.fasterxml.jackson:jackson-bom:2.20.0=runtimeClasspath
+com.azure:azure-sdk-bom:1.3.2=runtimeClasspath
+com.fasterxml.jackson:jackson-bom:2.20.1=runtimeClasspath
 io.netty:netty-bom:4.2.7.Final=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom-alpha:2.20.0-alpha=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom:2.20.0=runtimeClasspath

agent/agent-gc-monitor/gc-monitor-api/gradle.lockfile

@@ -1,8 +1,8 @@
 # This is a Gradle generated file for dependency locking.
 # Manual edits can break the build and are not advised.
 # This file is expected to be part of source control.
-com.azure:azure-sdk-bom:1.3.0=runtimeClasspath
-com.fasterxml.jackson:jackson-bom:2.20.0=runtimeClasspath
+com.azure:azure-sdk-bom:1.3.2=runtimeClasspath
+com.fasterxml.jackson:jackson-bom:2.20.1=runtimeClasspath
 com.google.errorprone:error_prone_annotations:2.43.0=runtimeClasspath
 io.netty:netty-bom:4.2.7.Final=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom-alpha:2.20.0-alpha=runtimeClasspath

agent/agent-gc-monitor/gc-monitor-api/src/main/java/com/microsoft/gcmonitor/MemoryManagers.java

@@ -27,7 +27,8 @@ public enum MemoryManagers {
 
   ZGC(GarbageCollectors.Zgc.class);
 
-  @SuppressWarnings("ImmutableEnumChecker")
+  @SuppressWarnings(
+      "ImmutableEnumChecker") // mutable enum state is intentional and properly synchronized
   private final Class<? extends GarbageCollector>[] managers;
 
   public static MemoryManagers of(MemoryManagement manager) {

agent/agent-gc-monitor/gc-monitor-api/src/main/java/com/microsoft/gcmonitor/memorypools/MemoryPools.java

@@ -300,7 +300,10 @@ public Shenandoah(Set<GarbageCollector> managers) {
     }
   }
 
-  @SuppressWarnings({"checkstyle:AbbreviationAsWordInName", "MemberName"})
+  @SuppressWarnings({
+    "checkstyle:AbbreviationAsWordInName",
+    "MemberName"
+  }) // CGroup is the standard abbreviation for Control Group
   public static class ZHeap extends MemoryPool {
     public static final String POOL_NAME = "ZHeap";
 

agent/agent-gc-monitor/gc-monitor-core/gradle.lockfile

@@ -1,8 +1,8 @@
 # This is a Gradle generated file for dependency locking.
 # Manual edits can break the build and are not advised.
 # This file is expected to be part of source control.
-com.azure:azure-sdk-bom:1.3.0=runtimeClasspath
-com.fasterxml.jackson:jackson-bom:2.20.0=runtimeClasspath
+com.azure:azure-sdk-bom:1.3.2=runtimeClasspath
+com.fasterxml.jackson:jackson-bom:2.20.1=runtimeClasspath
 com.google.errorprone:error_prone_annotations:2.43.0=runtimeClasspath
 io.netty:netty-bom:4.2.7.Final=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom-alpha:2.20.0-alpha=runtimeClasspath

agent/agent-gc-monitor/gc-monitor-core/src/main/java/com/microsoft/gcmonitor/collectors/GcCollectionSample.java

@@ -63,7 +63,7 @@ public GcCollectionSample(
 
   private static Map<MemoryPool, MemoryUsage> groupMemoryUsageByPoolName(
       Object map, MemoryManagement memoryManagement) {
-    @SuppressWarnings("unchecked")
+    @SuppressWarnings("unchecked") // safe unchecked cast - type verified by runtime context
     Map<List<String>, CompositeData> byName = (Map<List<String>, CompositeData>) map;
     Map<MemoryPool, MemoryUsage> byIdentifier = new HashMap<>();
     try {

agent/agent-gc-monitor/gc-monitor-tests/gradle.lockfile

@@ -1,8 +1,8 @@
 # This is a Gradle generated file for dependency locking.
 # Manual edits can break the build and are not advised.
 # This file is expected to be part of source control.
-com.azure:azure-sdk-bom:1.3.0=runtimeClasspath
-com.fasterxml.jackson:jackson-bom:2.20.0=runtimeClasspath
+com.azure:azure-sdk-bom:1.3.2=runtimeClasspath
+com.fasterxml.jackson:jackson-bom:2.20.1=runtimeClasspath
 io.netty:netty-bom:4.2.7.Final=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom-alpha:2.20.0-alpha=runtimeClasspath
 io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom:2.20.0=runtimeClasspath

agent/agent-gc-monitor/gc-monitor-tests/src/test/java/com/microsoft/gcmonitortests/GcEventGenerator.java

@@ -25,7 +25,7 @@ public static void main(String[] args) throws InterruptedException {
     new GcEventGenerator().run();
   }
 
-  @SuppressWarnings("SystemOut")
+  @SuppressWarnings("SystemOut") // legitimate use of System.out for logging/output
   private void run() throws InterruptedException {
     System.out.println("Hit return to start");