New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename more properties #2486
Rename more properties #2486
Conversation
@@ -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.chunk.new-trace=true' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strong +1 to new-trace always much more grokkable than talking about root spans.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add experimental
to this property?
@@ -22,9 +22,9 @@ | |||
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"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think about otel.javaagent.debug.thread-propagation.enabled
instead? We could also set its default value to Config.isAgentDebugEnabled()
Make
otel.exporter.jar
andotel.initializer.jar
experimental because we are planning to replace withotel.javaagent.extensions
soon (#2442):otel.exporter.jar
->otel.javaagent.experimental.exporter.jar
otel.initializer.jar
->otel.javaagent.experimental.initializer.jar
Put all instrumentation settings that are not specific to one
[id]
underotel.instrumentation.common.*
:otel.instrumentation.default-enabled
->otel.instrumentation.common.default-enabled
Follow existing convention of using
.enabled
:otel.instrumentation.kafka.client-propagation
->otel.instrumentation.kafka.client-propagation.enabled
Rename to better name(?):
otel.instrumentation.spring-batch.chunk.root-span
->otel.instrumentation.spring-batch.chunk.new-trace
I think these are experimental(?):
otel.javaagent.runtime.context.field.injection
->otel.javaagent.experimental.field-injection.enabled
otel.threadPropagationDebugger
->otel.javaagent.experimental.thread-propagation-debugger.enabled
Use
otel.javaagent.testing.*
for settings used during testing only:internal.testing.disable.global.library.ignores
->otel.javaagent.testing.additional-library-ignores.enabled
(and flipped)otel.internal.failOnContextLeak
->otel.javaagent.testing.fail-on-context-leak
otel.internal.enableTransformSafeLogging
->otel.javaagent.testing.transform-safe-logging.enabled
Note to self: I owe two follow-up PRs for this: