diff --git a/gradle/instrumentation.gradle b/gradle/instrumentation.gradle index 0b6913f80cff..66ab1d4b6a1d 100644 --- a/gradle/instrumentation.gradle +++ b/gradle/instrumentation.gradle @@ -111,13 +111,13 @@ evaluationDependsOn(":testing:agent-for-testing") tasks.withType(Test).configureEach { jvmArgs "-Dotel.javaagent.debug=true" jvmArgs "-javaagent:${project(":testing:agent-for-testing").tasks.shadowJar.archiveFile.get().asFile.absolutePath}" - jvmArgs "-Dotel.initializer.jar=${shadowJar.archiveFile.get().asFile.absolutePath}" - jvmArgs "-Dinternal.testing.disable.global.library.ignores=true" + jvmArgs "-Dotel.javaagent.experimental.initializer.jar=${shadowJar.archiveFile.get().asFile.absolutePath}" + jvmArgs "-Dotel.javaagent.testing.additional-library-ignores.enabled=false" // always run with the thread propagation debugger to help track down sporadic test failures - jvmArgs "-Dotel.threadPropagationDebugger=true" - jvmArgs "-Dotel.internal.failOnContextLeak=true" + jvmArgs "-Dotel.javaagent.experimental.thread-propagation-debugger.enabled=true" + jvmArgs "-Dotel.javaagent.testing.fail-on-context-leak=true" // prevent sporadic gradle deadlocks, see SafeLogger for more details - jvmArgs "-Dotel.internal.enableTransformSafeLogging=true" + jvmArgs "-Dotel.javaagent.testing.transform-safe-logging.enabled=true" dependsOn shadowJar dependsOn ":testing:agent-for-testing:shadowJar" diff --git a/instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/context/ContextPropagationDebug.java b/instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/context/ContextPropagationDebug.java index a53ce9f30f52..8b8f2da2d68e 100644 --- a/instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/context/ContextPropagationDebug.java +++ b/instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/context/ContextPropagationDebug.java @@ -22,9 +22,9 @@ public final class ContextPropagationDebug { ContextKey.named("thread-propagation-locations"); private static final boolean THREAD_PROPAGATION_DEBUGGER = - Boolean.getBoolean("otel.threadPropagationDebugger"); + Boolean.getBoolean("otel.javaagent.experimental.thread-propagation-debugger.enabled"); private static final boolean FAIL_ON_CONTEXT_LEAK = - Boolean.getBoolean("otel.internal.failOnContextLeak"); + Boolean.getBoolean("otel.javaagent.testing.fail-on-context-leak"); public static boolean isThreadPropagationDebuggerEnabled() { return THREAD_PROPAGATION_DEBUGGER; diff --git a/instrumentation/apache-camel-2.20/javaagent/src/test/groovy/io/opentelemetry/javaagent/instrumentation/apachecamel/SqsCamelTest.groovy b/instrumentation/apache-camel-2.20/javaagent/src/test/groovy/io/opentelemetry/javaagent/instrumentation/apachecamel/SqsCamelTest.groovy index 3300ce1981cf..a790ddfc35a5 100644 --- a/instrumentation/apache-camel-2.20/javaagent/src/test/groovy/io/opentelemetry/javaagent/instrumentation/apachecamel/SqsCamelTest.groovy +++ b/instrumentation/apache-camel-2.20/javaagent/src/test/groovy/io/opentelemetry/javaagent/instrumentation/apachecamel/SqsCamelTest.groovy @@ -36,19 +36,15 @@ class SqsCamelTest extends AgentInstrumentationSpecification { /** * Temporarily using emq instead of localstack till the latter supports AWS trace propagation * - sqs = new LocalStackContainer(DockerImageName.parse("localstack/localstack:latest")) - .withServices(LocalStackContainer.Service.SQS) - sqs.start() - sqsPort = sqs.getMappedPort(4566) + sqs = new LocalStackContainer(DockerImageName.parse("localstack/localstack:latest")) + .withServices(LocalStackContainer.Service.SQS) + sqs.start() + sqsPort = sqs.getMappedPort(4566) - def app = new SpringApplication(SqsConfig) - app.addInitializers(new ApplicationContextInitializer() { - @Override - void initialize(AbstractApplicationContext applicationContext) { - applicationContext.getBeanFactory().registerSingleton("localStack", sqs) - } - }) - server = app.run()**/ + def app = new SpringApplication(SqsConfig) + app.addInitializers(new ApplicationContextInitializer() {@Override + void initialize(AbstractApplicationContext applicationContext) {applicationContext.getBeanFactory().registerSingleton("localStack", sqs)}}) + server = app.run()**/ sqsPort = PortUtils.randomOpenPort() sqs = SQSRestServerBuilder.withPort(sqsPort).withInterface("localhost").start() @@ -105,7 +101,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "SendMessageRequest" + "aws.operation" "SendMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -124,7 +120,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -156,7 +152,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -177,7 +173,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "DeleteMessageRequest" + "aws.operation" "DeleteMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -198,7 +194,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -219,7 +215,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -240,7 +236,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -275,7 +271,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "SendMessageRequest" + "aws.operation" "SendMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -294,7 +290,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -326,7 +322,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -347,7 +343,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "DeleteMessageRequest" + "aws.operation" "DeleteMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -368,7 +364,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -389,7 +385,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -410,7 +406,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -449,7 +445,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "CreateQueueRequest" + "aws.operation" "CreateQueue" "aws.queue.name" "sqsCamelSeparateQueueTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -488,7 +484,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "SendMessageRequest" + "aws.operation" "SendMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelSeparateQueueTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -507,7 +503,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelSeparateQueueTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" @@ -533,7 +529,7 @@ class SqsCamelTest extends AgentInstrumentationSpecification { attributes { "aws.agent" "java-aws-sdk" "aws.endpoint" "http://localhost:$sqsPort" - "aws.operation" "ReceiveMessageRequest" + "aws.operation" "ReceiveMessage" "aws.queue.url" "http://localhost:$sqsPort/queue/sqsCamelSeparateQueueTest" "aws.service" "AmazonSQS" "http.flavor" "1.1" diff --git a/instrumentation/kafka-clients-0.11/javaagent/kafka-clients-0.11-javaagent.gradle b/instrumentation/kafka-clients-0.11/javaagent/kafka-clients-0.11-javaagent.gradle index b88a7ceef209..c89d14f4e5ed 100644 --- a/instrumentation/kafka-clients-0.11/javaagent/kafka-clients-0.11-javaagent.gradle +++ b/instrumentation/kafka-clients-0.11/javaagent/kafka-clients-0.11-javaagent.gradle @@ -42,5 +42,5 @@ test.finalizedBy(tasks.register("testPropagationDisabled", Test) { filter { includeTestsMatching 'KafkaClientPropagationDisabledTest' } - jvmArgs "-Dotel.instrumentation.kafka.client-propagation=false" + jvmArgs "-Dotel.instrumentation.kafka.client-propagation.enabled=false" }) diff --git a/instrumentation/kafka-clients-0.11/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/kafkaclients/KafkaClientsConfig.java b/instrumentation/kafka-clients-0.11/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/kafkaclients/KafkaClientsConfig.java index 3c11ee7b8047..202e8b6e9d69 100644 --- a/instrumentation/kafka-clients-0.11/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/kafkaclients/KafkaClientsConfig.java +++ b/instrumentation/kafka-clients-0.11/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/kafkaclients/KafkaClientsConfig.java @@ -10,7 +10,8 @@ public final class KafkaClientsConfig { public static boolean isPropagationEnabled() { - return Config.get().getBooleanProperty("otel.instrumentation.kafka.client-propagation", true); + return Config.get() + .getBooleanProperty("otel.instrumentation.kafka.client-propagation.enabled", true); } public static boolean captureExperimentalSpanAttributes() { diff --git a/instrumentation/kafka-clients-0.11/javaagent/src/test/groovy/KafkaClientBaseTest.groovy b/instrumentation/kafka-clients-0.11/javaagent/src/test/groovy/KafkaClientBaseTest.groovy index 4069735991ad..c52a5832bb37 100644 --- a/instrumentation/kafka-clients-0.11/javaagent/src/test/groovy/KafkaClientBaseTest.groovy +++ b/instrumentation/kafka-clients-0.11/javaagent/src/test/groovy/KafkaClientBaseTest.groovy @@ -23,7 +23,7 @@ abstract class KafkaClientBaseTest extends AgentInstrumentationSpecification { protected static final SHARED_TOPIC = "shared.topic" private static final boolean propagationEnabled = Boolean.parseBoolean( - System.getProperty("otel.instrumentation.kafka.client-propagation", "true")) + System.getProperty("otel.instrumentation.kafka.client-propagation.enabled", "true")) @Rule KafkaEmbedded embeddedKafka = new KafkaEmbedded(1, true, SHARED_TOPIC) diff --git a/instrumentation/spring/spring-batch-3.0/javaagent/spring-batch-3.0-javaagent.gradle b/instrumentation/spring/spring-batch-3.0/javaagent/spring-batch-3.0-javaagent.gradle index b329fe590076..f2fcdeef35e5 100644 --- a/instrumentation/spring/spring-batch-3.0/javaagent/spring-batch-3.0-javaagent.gradle +++ b/instrumentation/spring/spring-batch-3.0/javaagent/spring-batch-3.0-javaagent.gradle @@ -30,7 +30,7 @@ test.finalizedBy(tasks.register("testChunkRootSpan", Test) { filter { includeTestsMatching '*ChunkRootSpanTest' } - jvmArgs '-Dotel.instrumentation.spring-batch.chunk.root-span=true' + jvmArgs '-Dotel.instrumentation.spring-batch.experimental.chunk.new-trace=true' }).finalizedBy(tasks.register("testItemLevelSpan", Test) { filter { includeTestsMatching '*ItemLevelSpanTest' diff --git a/instrumentation/spring/spring-batch-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/spring/batch/SpringBatchInstrumentationConfig.java b/instrumentation/spring/spring-batch-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/spring/batch/SpringBatchInstrumentationConfig.java index 4ccb9ab9c20e..060c1a363326 100644 --- a/instrumentation/spring/spring-batch-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/spring/batch/SpringBatchInstrumentationConfig.java +++ b/instrumentation/spring/spring-batch-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/spring/batch/SpringBatchInstrumentationConfig.java @@ -20,7 +20,8 @@ public final class SpringBatchInstrumentationConfig { Config.get().isInstrumentationPropertyEnabled(instrumentationNames(), "item.enabled", false); private static final boolean CREATE_ROOT_SPAN_FOR_CHUNK = Config.get() - .isInstrumentationPropertyEnabled(instrumentationNames(), "chunk.root-span", false); + .isInstrumentationPropertyEnabled( + instrumentationNames(), "experimental.chunk.new-trace", false); public static List instrumentationNames() { return INSTRUMENTATION_NAMES; diff --git a/javaagent-api/src/main/java/io/opentelemetry/javaagent/instrumentation/api/concurrent/ExecutorInstrumentationUtils.java b/javaagent-api/src/main/java/io/opentelemetry/javaagent/instrumentation/api/concurrent/ExecutorInstrumentationUtils.java index 6ff1df139924..4aac417704de 100644 --- a/javaagent-api/src/main/java/io/opentelemetry/javaagent/instrumentation/api/concurrent/ExecutorInstrumentationUtils.java +++ b/javaagent-api/src/main/java/io/opentelemetry/javaagent/instrumentation/api/concurrent/ExecutorInstrumentationUtils.java @@ -29,7 +29,7 @@ public static boolean shouldAttachStateToTask(Object task) { Class enclosingClass = taskClass.getEnclosingClass(); // not much point in propagating root context - // plus it causes failures under otel.internal.failOnContextLeak=true + // plus it causes failures under otel.javaagent.testing.fail-on-context-leak=true return Context.current() != Context.root() // TODO Workaround for // https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/787 diff --git a/javaagent-bootstrap/src/main/java/io/opentelemetry/javaagent/bootstrap/AgentClassLoader.java b/javaagent-bootstrap/src/main/java/io/opentelemetry/javaagent/bootstrap/AgentClassLoader.java index e37b247b4df1..9323bbcca0e2 100644 --- a/javaagent-bootstrap/src/main/java/io/opentelemetry/javaagent/bootstrap/AgentClassLoader.java +++ b/javaagent-bootstrap/src/main/java/io/opentelemetry/javaagent/bootstrap/AgentClassLoader.java @@ -27,7 +27,7 @@ public class AgentClassLoader extends URLClassLoader { } private static final String AGENT_INITIALIZER_JAR = - System.getProperty("otel.initializer.jar", ""); + System.getProperty("otel.javaagent.experimental.initializer.jar", ""); // Calling java.lang.instrument.Instrumentation#appendToBootstrapClassLoaderSearch // adds a jar to the bootstrap class lookup, but not to the resource lookup. diff --git a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentInstaller.java b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentInstaller.java index 3e15fa68d488..7e5698bc4c1c 100644 --- a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentInstaller.java +++ b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentInstaller.java @@ -56,8 +56,8 @@ public class AgentInstaller { // We set this system property when running the agent with unit tests to allow verifying that we // don't ignore libraries that we actually attempt to instrument. It means either the list is // wrong or a type matcher is. - private static final String DISABLE_GLOBAL_LIBRARY_IGNORES_FOR_TEST = - "internal.testing.disable.global.library.ignores"; + private static final String ADDITIONAL_LIBRARY_IGNORES_ENABLED = + "otel.javaagent.testing.additional-library-ignores.enabled"; private static final Map> CLASS_LOAD_CALLBACKS = new HashMap<>(); private static volatile Instrumentation INSTRUMENTATION; @@ -127,7 +127,7 @@ public static ResettableClassFileTransformer installBytebuddyAgent( ignoredAgentBuilder = ignoredAgentBuilder.or( globalIgnoresMatcher( - Config.get().getBooleanProperty(DISABLE_GLOBAL_LIBRARY_IGNORES_FOR_TEST, false), + Config.get().getBooleanProperty(ADDITIONAL_LIBRARY_IGNORES_ENABLED, true), ignoreMatcherProvider)); ignoredAgentBuilder = ignoredAgentBuilder.or(matchesConfiguredExcludes()); diff --git a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentTracerProviderConfigurer.java b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentTracerProviderConfigurer.java index a20dfe83f04f..c8dcc46e4145 100644 --- a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentTracerProviderConfigurer.java +++ b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/AgentTracerProviderConfigurer.java @@ -34,7 +34,7 @@ public class AgentTracerProviderConfigurer implements SdkTracerProviderConfigurer { private static final Logger log = LoggerFactory.getLogger(AgentTracerProviderConfigurer.class); - static final String EXPORTER_JAR_CONFIG = "otel.exporter.jar"; + static final String EXPORTER_JAR_CONFIG = "otel.javaagent.experimental.exporter.jar"; @Override public void configure(SdkTracerProviderBuilder sdkTracerProviderBuilder) { diff --git a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/InstrumentationModule.java b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/InstrumentationModule.java index e7cbd311a36b..567b9122282f 100644 --- a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/InstrumentationModule.java +++ b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/InstrumentationModule.java @@ -330,6 +330,6 @@ protected Map contextStore() { } protected boolean defaultEnabled() { - return Config.get().getBooleanProperty("otel.instrumentation.default-enabled", true); + return Config.get().getBooleanProperty("otel.instrumentation.common.default-enabled", true); } } diff --git a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/TransformSafeLogger.java b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/TransformSafeLogger.java index 083470fd7f4c..b1f98608098b 100644 --- a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/TransformSafeLogger.java +++ b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/TransformSafeLogger.java @@ -25,7 +25,7 @@ public class TransformSafeLogger { private static final boolean ENABLE_TRANSFORM_SAFE_LOGGING = - Boolean.getBoolean("otel.internal.enableTransformSafeLogging"); + Boolean.getBoolean("otel.javaagent.testing.transform-safe-logging.enabled"); private static final BlockingQueue logMessageQueue; diff --git a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/context/FieldBackedProvider.java b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/context/FieldBackedProvider.java index e3a26ba3d4fe..8896ab78a15b 100644 --- a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/context/FieldBackedProvider.java +++ b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/context/FieldBackedProvider.java @@ -104,7 +104,7 @@ public class FieldBackedProvider implements InstrumentationContextProvider { } private static final boolean FIELD_INJECTION_ENABLED = - Config.get().getBooleanProperty("otel.javaagent.runtime.context.field.injection", true); + Config.get().getBooleanProperty("otel.javaagent.experimental.field-injection.enabled", true); private final Class instrumenterClass; private final ByteBuddy byteBuddy; diff --git a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/matcher/GlobalIgnoresMatcher.java b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/matcher/GlobalIgnoresMatcher.java index 5e3ff40684ec..846835377418 100644 --- a/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/matcher/GlobalIgnoresMatcher.java +++ b/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/matcher/GlobalIgnoresMatcher.java @@ -33,18 +33,18 @@ public class GlobalIgnoresMatcher Pattern.compile("com\\.mchange\\.v2\\.c3p0\\..*Proxy"); public static ElementMatcher.Junction globalIgnoresMatcher( - boolean skipAdditionalLibraryMatcher, IgnoreMatcherProvider ignoreMatcherProviders) { - return new GlobalIgnoresMatcher<>(skipAdditionalLibraryMatcher, ignoreMatcherProviders); + boolean additionalLibraryMatcher, IgnoreMatcherProvider ignoreMatcherProviders) { + return new GlobalIgnoresMatcher<>(additionalLibraryMatcher, ignoreMatcherProviders); } private final ElementMatcher additionalLibraryIgnoreMatcher = AdditionalLibraryIgnoresMatcher.additionalLibraryIgnoresMatcher(); - private final boolean skipAdditionalLibraryMatcher; + private final boolean additionalLibraryMatcher; private final IgnoreMatcherProvider ignoreMatcherProvider; private GlobalIgnoresMatcher( - boolean skipAdditionalLibraryMatcher, IgnoreMatcherProvider ignoreMatcherProvider) { - this.skipAdditionalLibraryMatcher = skipAdditionalLibraryMatcher; + boolean additionalLibraryMatcher, IgnoreMatcherProvider ignoreMatcherProvider) { + this.additionalLibraryMatcher = additionalLibraryMatcher; this.ignoreMatcherProvider = ignoreMatcherProvider; } @@ -194,7 +194,7 @@ public boolean matches(T target) { return true; } - if (!skipAdditionalLibraryMatcher && additionalLibraryIgnoreMatcher.matches(target)) { + if (additionalLibraryMatcher && additionalLibraryIgnoreMatcher.matches(target)) { return true; } diff --git a/javaagent-tooling/src/test/groovy/io/opentelemetry/javaagent/tooling/InstrumentationModuleTest.groovy b/javaagent-tooling/src/test/groovy/io/opentelemetry/javaagent/tooling/InstrumentationModuleTest.groovy index 5d2dfac1ad9b..fee1b93c0620 100644 --- a/javaagent-tooling/src/test/groovy/io/opentelemetry/javaagent/tooling/InstrumentationModuleTest.groovy +++ b/javaagent-tooling/src/test/groovy/io/opentelemetry/javaagent/tooling/InstrumentationModuleTest.groovy @@ -86,7 +86,7 @@ class InstrumentationModuleTest extends Specification { def "configure default sys prop as #value"() { setup: Config.INSTANCE = new ConfigBuilder().readProperties([ - "otel.instrumentation.default-enabled": String.valueOf(value) + "otel.instrumentation.common.default-enabled": String.valueOf(value) ]).build() def target = new TestInstrumentationModule(["test"]) target.instrument(new AgentBuilder.Default()) @@ -108,7 +108,7 @@ class InstrumentationModuleTest extends Specification { def "configure sys prop enabled for #value when default is disabled"() { setup: Config.INSTANCE = new ConfigBuilder().readProperties([ - "otel.instrumentation.default-enabled" : "false", + "otel.instrumentation.common.default-enabled" : "false", ("otel.instrumentation." + value + ".enabled"): "true" ]).build() diff --git a/testing-common/integration-tests/integration-tests.gradle b/testing-common/integration-tests/integration-tests.gradle index e6ae35ab8c25..e1a44bfebfa3 100644 --- a/testing-common/integration-tests/integration-tests.gradle +++ b/testing-common/integration-tests/integration-tests.gradle @@ -40,6 +40,6 @@ test.finalizedBy(tasks.register("testFieldInjectionDisabled", Test) { filter { includeTestsMatching 'context.FieldInjectionDisabledTest' } - jvmArgs '-Dotel.javaagent.runtime.context.field.injection=false' + jvmArgs '-Dotel.javaagent.experimental.field-injection.enabled=false' jvmArgs '-Dotel.instrumentation.context-test-instrumentation.enabled=true' }) diff --git a/testing-common/integration-tests/src/test/groovy/context/FieldInjectionDisabledTest.groovy b/testing-common/integration-tests/src/test/groovy/context/FieldInjectionDisabledTest.groovy index 76347939517a..ba9cfe29670d 100644 --- a/testing-common/integration-tests/src/test/groovy/context/FieldInjectionDisabledTest.groovy +++ b/testing-common/integration-tests/src/test/groovy/context/FieldInjectionDisabledTest.groovy @@ -11,7 +11,7 @@ import java.lang.reflect.Field import library.DisabledKeyClass // this test is run using: -// -Dotel.javaagent.runtime.context.field.injection=false +// -Dotel.javaagent.experimental.field-injection.enabled=false // -Dotel.instrumentation.context-test-instrumentation.enabled=true // (see integration-tests.gradle) class FieldInjectionDisabledTest extends AgentInstrumentationSpecification {