opentelemetry-java icon indicating copy to clipboard operation
opentelemetry-java copied to clipboard

Add known_methods to general section of declarative config

Open zeitlinger opened this issue 5 months ago • 6 comments

known_methods should be added to https://github.com/open-telemetry/opentelemetry-configuration/blob/main/examples/kitchen-sink.yaml#L872 as outlined in https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/14144/files#diff-542da2fe8649f0d9a75e16fdae4a7eccd8517664cc1ad671f5e713539442c56aR27

zeitlinger avatar Jul 07 '25 10:07 zeitlinger

Uncomment https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/56e6929bad7478faf48282f7c4ef7affa011a996/javaagent-extension-api/src/main/java/io/opentelemetry/javaagent/extension/DeclarativeConfigPropertiesBridge.java#L72 when done (and also the corresponding line it the test)

zeitlinger avatar Jul 07 '25 10:07 zeitlinger

We also need PRs in

  • https://opentelemetry.io/docs/specs/semconv/http/http-metrics/ (OTEL_INSTRUMENTATION_HTTP_KNOWN_METHODS)
  • https://github.com/open-telemetry/opentelemetry-configuration/blob/main/examples/kitchen-sink.yaml#L872

zeitlinger avatar Jul 07 '25 15:07 zeitlinger

Hi @zeitlinger , can i work on this in your guidance if that's ok for you, so that i have chance to explore opentelemetry as this is my first contribution?

MaheshS08 avatar Jul 13 '25 06:07 MaheshS08

@MaheshS08 I added the "good first issue" by mistake - this is a bit more complicated than I thought at first

zeitlinger avatar Jul 14 '25 10:07 zeitlinger

@zeitlinger thanks for letting me know , are there any beginner issues i can work on?

MaheshS08 avatar Jul 14 '25 10:07 MaheshS08

we now have the #opentelemetry-new-contributors slack channel

zeitlinger avatar Oct 09 '25 13:10 zeitlinger