Skip to content

Add SigNoz to Workers OpenTelemetry destinations#31527

Open
mahakkaur22 wants to merge 3 commits into
cloudflare:productionfrom
mahakkaur22:add-signoz-workers-otel
Open

Add SigNoz to Workers OpenTelemetry destinations#31527
mahakkaur22 wants to merge 3 commits into
cloudflare:productionfrom
mahakkaur22:add-signoz-workers-otel

Conversation

@mahakkaur22

@mahakkaur22 mahakkaur22 commented Jun 17, 2026

Copy link
Copy Markdown

Summary

Adds SigNoz to the Cloudflare Workers OpenTelemetry export docs.

This PR:

  • Adds SigNoz to the Available OpenTelemetry destinations table.
  • Adds a short setup guide for exporting Workers traces and logs to SigNoz.
  • Adds an example SigNoz trace screenshot.

SigNoz docs:
https://signoz.io/docs/integrations/outposts/cloudflare-workers/

Screenshots

Adds a SigNoz trace view screenshot in the new guide.

Documentation checklist

  • Is there a changelog entry? N/A - docs-only provider guide and table addition.
  • The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page - an issue has been opened. N/A - provider docs addition for existing Workers OTel export docs.
  • Files which have changed name or location have been allocated redirects. N/A.

@github-actions github-actions Bot added the product:workers Related to Workers product label Jun 17, 2026
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Review

⏸️ Automatic reviews for this PR are paused.

This PR has already received 2 automatic reviews. To run another review, a codeowner can comment /review or /full-review.

Tip: Keep PRs in draft mode until they are ready for review — the bot skips draft PRs automatically.


⚠️ 3 warnings and 12 suggestions found in commit 018fbaa.

Warnings (3)
File Issue
workers/observability/exporting-opentelemetry-data/signoz.mdx line 49 Remove contractions — Line adds "If you're running" Fix: Change to "If you are running"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 57 Remove contractions — Line adds "Now you'll create" Fix: Change to "Now you will create"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 105 WranglerConfig input must be TOML — WranglerConfig contains a ```json fence instead of TOML Fix: Provide TOML as input; JSON is auto-generated
Suggestions (12)
File Issue
workers/observability/exporting-opentelemetry-data/signoz.mdx line 27 Bullet lists with fewer than three items — Prerequisites list has only two bullet items Fix: Write the two prerequisites as prose instead of a bulleted list
workers/observability/exporting-opentelemetry-data/signoz.mdx line 35 Avoid navigate to — Line adds "Navigate to **Settings** in the sidebar" Fix: Replace "Navigate to" with "go to"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 37 Avoid click — Line adds "Click **New Ingestion Key**" Fix: Replace "Click" with "select"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 39 Avoid e.g. — Line adds "(e.g., cloudflare-workers)" Fix: Replace "e.g." with "for example" or restructure
workers/observability/exporting-opentelemetry-data/signoz.mdx line 42 Avoid click — Line adds "Click **Create new Ingestion key**" Fix: Replace "Click" with "select"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 47 Use sentence case for headings — Heading "For Self-Hosted SigNoz" uses title case for "Self-Hosted" Fix: Change to "For self-hosted SigNoz"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 53 Avoid see [link] — Line adds "See [Cloud vs Self-Hosted comparison]..." Fix: Replace with "refer to [Cloud vs Self-Hosted comparison]"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 66 Avoid see [link] — Line adds "See [SigNoz ingestion overview]..." Fix: Replace with "refer to [SigNoz ingestion overview]"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 70 Avoid navigate to — Line adds "Navigate to your Cloudflare account's..." Fix: Replace "Navigate to" with "go to"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 71 Avoid click — Line adds "Click **Add destination**" Fix: Replace "Click" with "select"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 87 Avoid click — Line adds "Click **Add destination** again" Fix: Replace "Click" with "select"
workers/observability/exporting-opentelemetry-data/signoz.mdx line 97 Avoid click — Line adds "Click **Save**" Fix: Replace "Click" with "select"
Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.

@mahakkaur22 mahakkaur22 requested a review from a team as a code owner June 17, 2026 20:21
@mahakkaur22

Copy link
Copy Markdown
Author

Hi @mikenomitch , could you take a look at this Workers OTel docs update when you have a chance?

This adds SigNoz to the Available OpenTelemetry destinations table and includes a short setup guide for exporting Workers traces and logs to SigNoz. I also fixed the docs bot warning by switching the SigNoz link text to monospace.

The earlier larger SigNoz PR (#26565) was closed by the stale bot, not rejected, so I opened this PR with the current Workers OTel docs structure in mind. Thanks!

@mahakkaur22

Copy link
Copy Markdown
Author

Gentle follow-up for the assigned reviewers: @nevikashah @irvinebroque @vy-ton, would one of you be able to take a look when you have a chance?

This is a small docs-only PR adding SigNoz as an OpenTelemetry destination for Cloudflare Workers, plus a short setup guide for traces/logs.

Happy to make any changes needed. Thanks!

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

Labels

product:workers Related to Workers product size/xs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants