Skip to content

Remove zipkin example#8501

Merged
MrAlias merged 4 commits into
open-telemetry:mainfrom
pradhyum6144:remove-zipkin-example
Feb 4, 2026
Merged

Remove zipkin example#8501
MrAlias merged 4 commits into
open-telemetry:mainfrom
pradhyum6144:remove-zipkin-example

Conversation

@pradhyum6144
Copy link
Copy Markdown
Contributor

Removes the deprecated zipkin example following the deprecation of the Zipkin exporter in v1.40.0.

Fixes #8484

Changes

  • Removed examples/zipkin/ directory and all its files (Dockerfile, README.md, docker-compose.yml, go.mod, go.sum, main.go)
  • Updated versions.yaml to remove zipkin example reference
  • Updated .gitignore to remove zipkin binary reference

Verification

  • Build verification: Passed
  • No remaining references to zipkin example found in codebase
  • Total changes: 8 files, 204 deletions

Signed-off-by: pradhyum6144 <pradhyum314@gmail.com>
@pradhyum6144 pradhyum6144 requested a review from a team as a code owner February 3, 2026 17:13
@pradhyum6144
Copy link
Copy Markdown
Contributor Author

@MrAlias can you please review this. Thank you.

Copy link
Copy Markdown
Contributor

@MrAlias MrAlias left a comment

Choose a reason for hiding this comment

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

Overall, this looks good. Thanks for the contribution.

Please add a changelog entry for the removal of this module.

Signed-off-by: pradhyum6144 <pradhyum314@gmail.com>
@pradhyum6144 pradhyum6144 requested a review from MrAlias February 3, 2026 18:11
@pradhyum6144
Copy link
Copy Markdown
Contributor Author

Overall, this looks good. Thanks for the contribution.

Please add a changelog entry for the removal of this module.

okay sir i have added the changelog entry. now this pr is ready to review/merge.
Thank you

Comment thread CHANGELOG.md Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.2%. Comparing base (82cfc18) to head (9e67535).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #8501   +/-   ##
=====================================
  Coverage   82.2%   82.2%           
=====================================
  Files        179     179           
  Lines      13692   13692           
=====================================
  Hits       11257   11257           
  Misses      2033    2033           
  Partials     402     402           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
@MrAlias MrAlias merged commit a72f3fc into open-telemetry:main Feb 4, 2026
29 checks passed
@MrAlias MrAlias added this to the v1.41.0 milestone Feb 26, 2026
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.

Remove the zipkin example

4 participants