hypertrace / javaagent

Hypertrace OpenTelemetry Java agent with payload/body and headers data capture.
Apache License 2.0
28 stars 14 forks source link

Upgrade Hypertrace Java agent to 1.13.1 level of Open Telemetry agent #368

Closed bradAtTraceable closed 1 year ago

bradAtTraceable commented 1 year ago

Description

Please include a summary of the change, motivation and context.

Testing

Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally.

Checklist:

Documentation

Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.

tim-mwangi commented 1 year ago

Hey @bradAtTraceable why didn't we just upgrade to 1.16.0? (or the latest 1.17.0)

Seems as if 1.13.1 is most current version of OTEL java agent:

https://mvnrepository.com/artifact/io.opentelemetry.javaagent/opentelemetry-javaagent-instrumentation-api

tim-mwangi commented 1 year ago

Seems like the kotlin issue hasn't been fixed even in the latest version. So no need for the constraint. You can just add to the snyk ignore file.

bradAtTraceable commented 1 year ago
ryanericson commented 1 year ago

@bradAtTraceable looks like the package name has changed for otel java: https://mvnrepository.com/artifact/io.opentelemetry.instrumentation/opentelemetry-instrumentation-api.