Skip to content

feat(pubsub): add opentelemetry tracing support#10709

Merged
hongalex merged 6 commits into
googleapis:mainfrom
hongalex:pubsub-otel-trace-support
Aug 19, 2024
Merged

feat(pubsub): add opentelemetry tracing support#10709
hongalex merged 6 commits into
googleapis:mainfrom
hongalex:pubsub-otel-trace-support

Conversation

@hongalex

Copy link
Copy Markdown
Member

This PR merges changes two other reviewed PRS: in #8317 and #9594, which merged changes into a feature branch pubsub-otel-trace.

Specifically this includes

  • Tracing support for publishing and subscribing
  • context propagation between publisher and subscriber via Pub/Sub message attributes
  • Linking for batch RPC spans (publish, ack, nack/modack)
  • New setting EnableOpenTelemetryTracing to pass into NewClientWithConfig to enable tracing

NOTE: OpenTelemetry tracing standards are in active development, and thus attributes, links, and span names are EXPERIMENTAL and subject to change or removal without notice.

@hongalex hongalex requested a review from shollyman as a code owner August 19, 2024 21:46
@hongalex hongalex requested review from a team August 19, 2024 21:46
@conventional-commit-lint-gcf

conventional-commit-lint-gcf Bot commented Aug 19, 2024

Copy link
Copy Markdown

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@product-auto-label product-auto-label Bot added the api: pubsub Issues related to the Pub/Sub API. label Aug 19, 2024

@shollyman shollyman 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.

Only scrutinized this superficially, given the nature of the change.

@hongalex hongalex merged commit bbd3366 into googleapis:main Aug 19, 2024
@hongalex hongalex deleted the pubsub-otel-trace-support branch December 10, 2025 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsub Issues related to the Pub/Sub API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants