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.

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

Consistent Span name and http.route.name #24017

Merged
merged 1 commit into from
Mar 1, 2022

Conversation

radcortez
Copy link
Member

No description provided.

@gsmet
Copy link
Member

gsmet commented Feb 28, 2022

@radcortez it looks like a bugfix?

@radcortez
Copy link
Member Author

@radcortez it looks like a bugfix?

Sort of. Historically, our initial implementation omitted the leading slash from the Span name, which I kept until the OTel API fixed an issue in how to set the http.route which is also used to update the Span name now.

For reference: open-telemetry/opentelemetry-java-instrumentation#4716

@radcortez radcortez requested a review from ebullient March 1, 2022 00:01
@ebullient ebullient merged commit 86a9462 into quarkusio:main Mar 1, 2022
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Mar 1, 2022
rsvoboda added a commit to rsvoboda/beefy-scenarios that referenced this pull request Mar 1, 2022
Sor of breaking change introduced in quarkusio/quarkus#24017
@rsvoboda
Copy link
Member

rsvoboda commented Mar 1, 2022

Ecosystem CI failed on https://github.com/quarkus-qe/beefy-scenarios repo because of this.

I see this change as breaking change as people can be relying on the behavior when Quarkus omitted the leading slash from the Span name.

@radcortez please add entry into https://github.com/quarkusio/quarkus/wiki/Migration-Guide-2.8

@radcortez
Copy link
Member Author

Sure. Thanks!

michalvavrik added a commit to michalvavrik/quarkus-test-suite that referenced this pull request Jun 11, 2022
Expected operation name is prefixed with forward slash is now as span name creation strategy has [recently changed](quarkusio/quarkus#24017)
michalvavrik added a commit to michalvavrik/quarkus-test-suite that referenced this pull request Jun 13, 2022
Expected operation name is prefixed with forward slash is now as span name creation strategy has [recently changed](quarkusio/quarkus#24017)
michalvavrik added a commit to michalvavrik/quarkus-test-suite that referenced this pull request Jun 13, 2022
Expected operation name is prefixed with forward slash is now as span name creation strategy has [recently changed](quarkusio/quarkus#24017)
michalvavrik added a commit to michalvavrik/quarkus-test-suite that referenced this pull request Jun 13, 2022
Expected operation name is prefixed with forward slash is now as span name creation strategy has [recently changed](quarkusio/quarkus#24017)
michalvavrik added a commit to michalvavrik/quarkus-test-suite that referenced this pull request Jun 13, 2022
Expected operation name is prefixed with forward slash is now as span name creation strategy has [recently changed](quarkusio/quarkus#24017)
pjgg pushed a commit to pjgg/quarkus-test-suite that referenced this pull request Jun 14, 2022
Expected operation name is prefixed with forward slash is now as span name creation strategy has [recently changed](quarkusio/quarkus#24017)

(cherry picked from commit 4564b78)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants