Skip to content

Add SigNoz to Workers OpenTelemetry destinations - #31527

Merged
irvinebroque merged 4 commits into
cloudflare:productionfrom
mahakkaur22:add-signoz-workers-otel
Sep 5, 2026
Merged

Add SigNoz to Workers OpenTelemetry destinations#31527
irvinebroque merged 4 commits into
cloudflare:productionfrom
mahakkaur22:add-signoz-workers-otel

Conversation

@mahakkaur22

@mahakkaur22 mahakkaur22 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds SigNoz to the available OpenTelemetry destinations table for Workers, including its traces and logs endpoints and a link to the official SigNoz setup documentation.

Documentation checklist

  • The change adheres to the documentation style guide.

@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

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

@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Hey there, we've marked this pull request as stale because there's no recent activity on it. This label helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant).

@mahakkaur22

Copy link
Copy Markdown
Contributor Author

Hi @nevikashah , following up on this docs-only PR adding SigNoz to the Workers OTel destinations table plus a setup page. It looks like this landed with you after last week's reviewer cleanup. The latest push addresses the earlier docs-bot feedback. Since automatic reviews are paused, could you run /review or take a quick look when you get a chance? Happy to make any changes.

@github-actions github-actions Bot removed the stale label Jul 8, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hey there, we've marked this pull request as stale because there's no recent activity on it. This label helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant).

@github-actions github-actions Bot added the stale label Jul 22, 2026
@mahakkaur22

Copy link
Copy Markdown
Contributor Author

Hi @nevikashah, following up again on this docs-only PR (adds SigNoz to the Workers OTel destinations table plus a short setup guide). It's been a few weeks since the stale bot re-flagged it. Since automatic reviews are paused, a quick /review or manual look would help move it forward. Happy to make any changes needed. Thanks!

@github-actions github-actions Bot removed the stale label Aug 25, 2026
@irvinebroque
irvinebroque force-pushed the add-signoz-workers-otel branch from 104aad0 to 4c4ddd1 Compare September 4, 2026 19:42
@irvinebroque

Copy link
Copy Markdown
Contributor

/bonk keep a line in src/content/docs/workers/observability/exporting-opentelemetry-data/index.mdx for this but remove the other page and screenshot

@ask-bonk

ask-bonk Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

github run

@ask-bonk

ask-bonk Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

@irvinebroque Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

@irvinebroque
irvinebroque enabled auto-merge (squash) September 5, 2026 16:18
@irvinebroque
irvinebroque merged commit 593c443 into cloudflare:production Sep 5, 2026
10 checks passed
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