Add known_methods to general section of declarative config
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
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)
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
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 I added the "good first issue" by mistake - this is a bit more complicated than I thought at first
@zeitlinger thanks for letting me know , are there any beginner issues i can work on?
we now have the #opentelemetry-new-contributors slack channel