Skip to content

NH-107047 Completely remove legacy SW span exporter#690

Merged
tammy-baylis-swi merged 1 commit intomainfrom
NH-107047-rm-legacy-span-exporter
Sep 8, 2025
Merged

NH-107047 Completely remove legacy SW span exporter#690
tammy-baylis-swi merged 1 commit intomainfrom
NH-107047-rm-legacy-span-exporter

Conversation

@tammy-baylis-swi
Copy link
Copy Markdown
Contributor

We completely remove legacy SW span exporter, which has been no-op with a warning message since APM Python 4.0.0. This PR wipes the exporter class, its entry point (used for configuration), its test code, and its swap helper. If OTEL_TRACES_EXPORTER includes solarwinds_exporter, upstream will raise RuntimeError.

@tammy-baylis-swi tammy-baylis-swi marked this pull request as ready for review September 5, 2025 17:35
@tammy-baylis-swi tammy-baylis-swi requested a review from a team as a code owner September 5, 2025 17:35
Copy link
Copy Markdown
Contributor

@cheempz cheempz left a comment

Choose a reason for hiding this comment

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

LGTM!

@cheempz
Copy link
Copy Markdown
Contributor

cheempz commented Sep 8, 2025

Maybe this can be part of the breaking release along with ready? API change.

@tammy-baylis-swi
Copy link
Copy Markdown
Contributor Author

Maybe this can be part of the breaking release along with ready? API change.

Yes! All in one go ☑️

@tammy-baylis-swi tammy-baylis-swi merged commit f8b87b5 into main Sep 8, 2025
69 checks passed
@tammy-baylis-swi tammy-baylis-swi deleted the NH-107047-rm-legacy-span-exporter branch September 8, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants