-
Couldn't load subscription status.
- Fork 168
declarative config: support Inferred spans #2030
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
base: main
Are you sure you want to change the base?
Conversation
b119720 to
8d60820
Compare
|
@JonasKunz please have a look |
|
@jaydeluca please have a look |
...rc/test/java/io/opentelemetry/contrib/inferredspans/InferredSpansCustomizerProviderTest.java
Outdated
Show resolved
Hide resolved
...rc/test/java/io/opentelemetry/contrib/inferredspans/InferredSpansCustomizerProviderTest.java
Outdated
Show resolved
Hide resolved
|
@jaydeluca please check again |
This reverts commit 26ce159.
0a1790c to
ef66353
Compare
12b5e4a to
4c26526
Compare
|
🔧 The result from spotlessApply was committed to the PR branch. |
|
@jackshirazi @JonasKunz please have a look |
|
@jackshirazi @JonasKunz bump |
| + " backup:\n" | ||
| + " diagnostic:\n" | ||
| + " files: true\n"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jackshirazi I think we should keep backup_diagnostric_files as a single attribute, WDYT ?
The declarative config bridge has been copied from the agent - eventually, we can use only one, but that's a later step.
Fixes #2029