-
Notifications
You must be signed in to change notification settings - Fork 850
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
Add gRPC authenticator for exporter #6952
base: main
Are you sure you want to change the base?
Add gRPC authenticator for exporter #6952
Conversation
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6952 +/- ##
============================================
- Coverage 90.09% 90.06% -0.03%
- Complexity 6601 6603 +2
============================================
Files 730 730
Lines 19843 19864 +21
Branches 1955 1958 +3
============================================
+ Hits 17877 17890 +13
- Misses 1371 1379 +8
Partials 595 595 ☔ View full report in Codecov by Sentry. |
d4606bf
to
05f6085
Compare
I was trying to figure out why the build-graal check is failing. It's complaining about changing a static variable in io.opentelemetry.exporter.internal.auth.Authenticator. Which is WAI. Is the test flaky? EDIT: fixed by adding the following to
|
05f6085
to
8713eed
Compare
Adding missing gRPC authentication features to the experimental
Authenticator
api. Closes #6953