Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Upgrade to OTEL 1.7.X #355

Merged
merged 34 commits into from
Jan 23, 2022
Merged

Upgrade to OTEL 1.7.X #355

merged 34 commits into from
Jan 23, 2022

Conversation

ryandens
Copy link
Member

@ryandens ryandens commented Jan 19, 2022 •
edited
Loading

Description

  • Large refactor to replace usages of ContextStore and InstrumentationContext with the new VirtualField API per this otel PR
  • Small upgrades to netty instrumentation modules based on their transition to the Instrumenter API
  • Small changes to servlet 3 async instrumentation to handle class transformation issues
  • Integration of the InstrumentationModuleMuzzle based on this refactor in otel
  • addition of the muzzle project as a dependency

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:

  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules

Documentation

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

@ryandens
⬆️ upgrade otel SDK
3b8acc3
@ryandens
🚧
e133d1f
@ryandens
🐛 remove all classifier from javaagent artifact
b9344fd
@ryandens
🔥 delete tooling from autoinstrumentation dependency
2614874
@ryandens
🐛 add compile dependencies on relocated otel servlet objects
9cee670
@ryandens
♻️ refactor java-streams to handle InstrumentationContext and Context…
a0891a2 
…Store renaming to VirtualField
@ryandens
♻️ refactor apache-http to handle InstrumentationContext and ContextS…
55b3e5c 
…tore renaming to VirtualField
@ryandens
♻️ rename otel Cache.newBuilder() to builder()
0742cf2
@ryandens
♻️ refactor jaxrs-client to handle InstrumentationContext and Context…
5deff5e 
…Store renaming to VirtualField
@ryandens
🎨 format java-streams
99c6487
@ryandens
🎨 format apache-http
dcf5c94
@ryandens
🎨 format jaxrs-client"
a0b2bab
@ryandens
♻️ refactor servlet-3.0 to handle InstrumentationContext and ContextS…
7b18ca3 
…tore renaming to VirtualField
@ryandens
🐛 add otel to instrumentation classpath'
da8374a
@ryandens
🐛 update netty-4.0 module based on otel refactor to the instrumenter API
35c433d
@ryandens
🐛 update netty-4.1 module based on otel refactor to the instrumenter API
c37122b
@ryandens
♻️ refactor servlet-rw to handle InstrumentationContext and ContextSt…
f80ba13 
…ore renaming to VirtualField
@ryandens
♻️ refactor undertow-1.4 to handle InstrumentationContext and Context…
a0796f3 
…Store renaming to VirtualField
@ryandens
♻️ refactor undertow-servlet to handle InstrumentationContext and Con…
c80ff6f 
…textStore renaming to VirtualField
@ryandens
♻️ refactor vertx to handle InstrumentationContext and ContextStore r…
4b8998c 
…enaming to VirtualField
@ryandens
♻️ account for InstrumentationModule muzzle refactor in java-streams …
5345da2 
…module
@ryandens
♻️ account for InstrumentationModule muzzle refactor in servlet-3.0
e315930
@ryandens
♻️ account for InstrumentationModule muzzle refactor in servlet-rw
80e1fa1
@ryandens
🐛 upgrade gradle plugins, move tooling to convention plugin for ease …
c36e270 
…of configuration
@ryandens
♻️ rename MuzzleCodeGenerationPlugin
eb69ad2
@ryandens
🐛 add constraint for java9 tooling API
e0288d8
@ryandens
🐛 fix expected metric name for span exporter
a844957
@ryandens
🐛 fix class definition error as a result of using Span.fromContext in…
fa18440 
…line
@ryandens
✅ update assertions based on the removal of http.url for http.scheme,…
49e16d3 
… http.host, and http.target
@ryandens ryandens changed the title 🚧 Upgrade otel 1 7 Upgrade to OTEL 1.7.X Jan 20, 2022
@ryandens ryandens marked this pull request as ready for review January 20, 2022 19:12
@ryandens ryandens merged commit 9d80781 into main Jan 23, 2022
@ryandens ryandens deleted the upgrade-otel-1-7 branch January 23, 2022 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davexroth davexroth davexroth approved these changes

@shashank11p shashank11p shashank11p approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ryandens @davexroth @shashank11p