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

Fix OpenTelemetry rest client span name #767

Merged

Conversation

michalvavrik
Copy link
Member

Summary

Logic behind Rest Client span names has changed in quarkusio/quarkus#26694, this reflect changes and fixes related test failures in daily CI.

Please select the relevant options.

  • Bug fix (non-breaking change which fixes an issue)
  • Dependency update
  • Refactoring
  • Backport
  • New scenario (non-breaking change which adds functionality)
  • This change requires a documentation update

Checklist:

  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

Logic behind Rest Client span names has changed in quarkusio/quarkus#26694, this reflect changes and fixes related test failures in daily CI.
@michalvavrik michalvavrik requested a review from fedinskiy July 27, 2022 08:45
@michalvavrik michalvavrik self-assigned this Jul 27, 2022
@fedinskiy fedinskiy merged commit 00fc230 into quarkus-qe:main Jul 27, 2022
@michalvavrik michalvavrik deleted the feature/fix-rest-client-span-name branch July 27, 2022 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants