Skip to content

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

Open
weizhikuan wants to merge 7 commits intoopen-telemetry:mainfrom
weizhikuan:multi-dest-exporting
Open

docs: add new example for multi-destination exporting to Sphinx Doc#5087
weizhikuan wants to merge 7 commits intoopen-telemetry:mainfrom
weizhikuan:multi-dest-exporting

Conversation

@weizhikuan
Copy link
Copy Markdown

@weizhikuan weizhikuan commented Apr 14, 2026 •
edited
Loading

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
Copy link
Copy Markdown

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

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
Open
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
Copy link
Copy Markdown
Contributor

tammy-baylis-swi commented Apr 14, 2026

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
Copy link
Copy Markdown
Author

weizhikuan commented Apr 15, 2026 •
edited
Loading

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
Copy link
Copy Markdown
Contributor

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

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
Copy link
Copy Markdown
Member

@MikeGoldsmith MikeGoldsmith left a comment

Choose a reason for hiding this comment

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

Thanks @weizhikuan 👍🏻

@weizhikuan
Copy link
Copy Markdown
Author

weizhikuan commented Apr 23, 2026

@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
Copy link
Copy Markdown
Member

MikeGoldsmith commented Apr 24, 2026

@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
Copy link
Copy Markdown
Author

weizhikuan commented Apr 25, 2026

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!

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

Reviewers

@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: Approved PRs

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

OTLP exporter should support multi-destination exporting

3 participants

@weizhikuan @tammy-baylis-swi @MikeGoldsmith