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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename configuration properties otel.instrumentation.* to match package naming convention #2444
Comments
Think long term we don't want any properties read in library instrumentation, programmatic makes sense. In which case the current names seem ok. |
Hmm, don't we want to move all the sanitization classes to instrumentation-api? Anyway, I think that I like the current names: you don't have to think if it's javaagent or library instrumentation, the naming is hierarchical and logical: otel > instrumentation > library > enabled |
I think we have agreement now on the following:
|
The package naming in this repo is all good (don't want you to worry that I'm opening that can of worms 馃槃):
io.opentelemetry.javaagent
- javaagentio.opentelemetry.javaagent.instrumentation
- javaagent instrumentationio.opentelemetry.instrumentation
- library instrumentationShould we follow similar naming for configuration properties:
otel.javaagent.*
- javaagent propertiesotel.javaagent.instrumentation.*
- javaagent instrumentation propertiesotel.instrumentation.*
- library instrumentation properties (if we even need any? e.g. library instrumentation could possibly be programmatic config only)The things that would need to be renamed if we want to make this change are:
otel.instrumentation.default-enabled
-->otel.javaagent.instrumentation.default-enabled
otel.instrumentation.[id].enabled
-->otel.javaagent.instrumentation.[id].enabled
otel.instrumentation.[id].*
-->otel.javaagent.instrumentation.[id].*
(instrumentation-specific properties)otel.instrumentation.db-statement-sanitizer.enabled
-->otel.javaagent.db-statement-sanitizer.enabled
(or maybeotel.javaagent.instrumentation.db-statement-sanitizer.enabled
, but that looks likeotel.javaagent.instrumentation.[id].enabled
)The text was updated successfully, but these errors were encountered: