Skip to content

Conversation

@Abhilash-Garapati
Copy link

This PR renames the configuration key:

otel.instrumentation.experimental.span-suppression-strategy

to the new key:

otel.instrumentation.common.experimental.span-suppression-strategy

Changes:

  • Updated all references to new key.
  • Added backward compatibility: old key still works and logs a warning.
  • Build successful locally.

Fixes #15566

@Abhilash-Garapati Abhilash-Garapati requested a review from a team as a code owner December 11, 2025 20:20
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 11, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Member

@trask trask left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It may be worth copying DeprecatedConfigProperties.java into this module (just to stay consistent with this pattern)

@Abhilash-Garapati Abhilash-Garapati force-pushed the my-feature-branch branch 2 times, most recently from 1c4d5da to 876fc97 Compare December 12, 2025 06:17
@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Dec 12, 2025
@Abhilash-Garapati
Copy link
Author

Thanks! I have now copied DeprecatedConfigProperties.java into the instrumentation-api module as suggested, to stay consistent with the project structure. The PR has been updated accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test native This label can be applied to PRs to trigger them to run native tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename otel.instrumentation.experimental.span-suppression-strategy

2 participants