Skip to content

docs: add new example for multi-destination exporting to Sphinx Doc#5087

Merged
xrmx merged 11 commits into
open-telemetry:mainfrom
weizhikuan:multi-dest-exporting
May 15, 2026
Merged

docs: add new example for multi-destination exporting to Sphinx Doc#5087
xrmx merged 11 commits into
open-telemetry:mainfrom
weizhikuan:multi-dest-exporting

Conversation

@weizhikuan

@weizhikuan weizhikuan commented Apr 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Add an example for multi-destination exporting to Sphinx documentation build.

Fixes #1109

Type of change

Please delete options that are not relevant.

  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Did test the new content to the doc with tox -e docs and checked the generated doc page.

For each python example added, ran as it is and getting the expected output for multi-destination exporting output including console, otlp-grpc and otlp-http

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@weizhikuan
weizhikuan requested a review from a team as a code owner April 14, 2026 03:32
@linux-foundation-easycla

linux-foundation-easycla Bot commented Apr 14, 2026 •
edited
Loading

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

@weizhikuan
weizhikuan force-pushed the multi-dest-exporting branch from 8c4dab5 to 7127c68 Compare April 14, 2026 03:32
@weizhikuan weizhikuan mentioned this pull request Apr 14, 2026
Closed
tammy-baylis-swi
tammy-baylis-swi reviewed Apr 14, 2026
View reviewed changes
Comment thread docs/examples/multi-destination-exporting/README.rst
tammy-baylis-swi
tammy-baylis-swi reviewed Apr 14, 2026
View reviewed changes
Comment thread docs/examples/multi-destination-exporting/README.rst Outdated
@tammy-baylis-swi
tammy-baylis-swi requested a review from a team April 14, 2026 22:44
@tammy-baylis-swi

tammy-baylis-swi commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Thanks @weizhikuan ! Please could you also run tox -e precommit to iron out some formatting.

@tammy-baylis-swi tammy-baylis-swi moved this to Reviewed PRs that need fixes in Python PR digest Apr 14, 2026
@weizhikuan

weizhikuan commented Apr 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Thanks @weizhikuan ! Please could you also run tox -e precommit to iron out some formatting.

Yep, did that and fixed the formatting issue.

@weizhikuan
weizhikuan force-pushed the multi-dest-exporting branch from 1387b71 to e2e2912 Compare April 15, 2026 02:46
tammy-baylis-swi
tammy-baylis-swi approved these changes Apr 15, 2026
View reviewed changes

@tammy-baylis-swi tammy-baylis-swi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you! Lgtm

@tammy-baylis-swi tammy-baylis-swi moved this from Reviewed PRs that need fixes to Approved PRs in Python PR digest Apr 15, 2026
@weizhikuan
weizhikuan force-pushed the multi-dest-exporting branch from e2e2912 to d06c744 Compare April 16, 2026 02:21
@MikeGoldsmith MikeGoldsmith added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 17, 2026
MikeGoldsmith
MikeGoldsmith approved these changes Apr 17, 2026
View reviewed changes

@MikeGoldsmith MikeGoldsmith left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @weizhikuan 👍🏻

@weizhikuan

weizhikuan commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

@MikeGoldsmith NP! Also just want to check and I am not quite familiar with the workflow, will this be merged anytime soon? or still needs some improvments?

@MikeGoldsmith

MikeGoldsmith commented Apr 24, 2026

Copy link
Copy Markdown
Member

@MikeGoldsmith NP! Also just want to check and I am not quite familiar with the workflow, will this be merged anytime soon? or still needs some improvments?

Once an approver has given feedback approved, it then goes to the project maintainers to do final review and merge. Hopefully someone can get to it soon. The project has a lot of open PRs right now and we're working on a patch.

@weizhikuan

weizhikuan commented Apr 25, 2026

Copy link
Copy Markdown
Contributor Author

Once an approver has given feedback approved, it then goes to the project maintainers to do final review and merge. Hopefully someone can get to it soon. The project has a lot of open PRs right now and we're working on a patch.

Thanks for the explanation!

@github-actions

github-actions Bot commented May 12, 2026

Copy link
Copy Markdown

This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment.
If you're still working on this, please add a comment or push new commits.

@github-actions github-actions Bot added the Stale label May 12, 2026
xrmx
xrmx approved these changes May 12, 2026
View reviewed changes
Comment thread docs/examples/multi-destination-exporting/multi_destination_logs.py Outdated
Comment thread docs/examples/multi-destination-exporting/multi_destination_metrics.py Outdated
Comment thread docs/examples/multi-destination-exporting/multi_destination_traces.py Outdated
@xrmx
xrmx enabled auto-merge (squash) May 12, 2026 12:57
@github-actions github-actions Bot removed the Stale label May 13, 2026
@xrmx
xrmx added this pull request to the merge queue May 13, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks May 13, 2026
@xrmx xrmx moved this from Approved PRs to Ready for merge in Python PR digest May 15, 2026
@xrmx
xrmx enabled auto-merge (squash) May 15, 2026 09:30
@xrmx
xrmx merged commit 369644c into open-telemetry:main May 15, 2026
480 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for merge to Done in Python PR digest May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xrmx xrmx xrmx approved these changes
@MikeGoldsmith MikeGoldsmith MikeGoldsmith approved these changes
@tammy-baylis-swi tammy-baylis-swi tammy-baylis-swi approved these changes

Assignees

No one assigned

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

Python PR digest
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

OTLP exporter should support multi-destination exporting

4 participants

@weizhikuan @tammy-baylis-swi @MikeGoldsmith @xrmx