-
Notifications
You must be signed in to change notification settings - Fork 856
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
Document available configuration #342
Comments
We need these docs about config parameters. My understanding is 1255 and 479 has been released in 0.4.0. Docs doesn't reflect these parameters. If yes, should we update the docs? |
Of course we should update the docs. This issue is exactly about that :) Do you care to submit a PR and kick it off? |
I split out a more targeted issue specifically to document the standard OpenTelemetry env var / system properties that we support #666. |
#666 has been closed for a while, and I feel like the docs are pretty good at this point. Is there any more to do on this one? |
@breedx-splk We don't have any documentation for instrumentation-specific configurations like |
Just as the label is removed to this issue. still is our only AI waiting for to declare |
Hey @vasireddy99, |
there's a script now to find all configuration options: https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/.github/scripts/find-all-config-properties.sh some of the general ones are documented at https://opentelemetry.io/docs/instrumentation/java/automatic/agent-config/#common-instrumentation-configuration and some of the specific ones are documented in the instrumentation module's README, e.g. https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/graphql-java-12.0/javaagent/README.md |
@trask IIRC there are a number of config items that are "composed" (for lack of a better term right now), where the end configs are based on a prefix or something, right? I didn't bother to look for an example, but I swear I've seen that. Won't this miss those? Not to harsh on the script (it's great!) tho I do think we'd get a lot of mileage out of a more concrete/central config names class or something. |
I think that the instrumentation enabled property ( |
I was unable to find any documentation about supported configuration options.
io.opentelemetry.auto.config.Config
class has quite a lot of options that can be of interest to end-users.The text was updated successfully, but these errors were encountered: