Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Distributed Tracing Docs for Uptime #11911

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gaprl
Copy link
Member

@gaprl gaprl commented Nov 22, 2024

Adds a new distributed tracing page for uptime that describes how it works, and how it can be enabled or disabled for errors and spans.

Copy link

vercel bot commented Nov 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 2:26am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Nov 22, 2024 2:26am
develop-docs ⬜️ Ignored (Inspect) Visit Preview Nov 22, 2024 2:26am

Copy link

codecov bot commented Nov 22, 2024

Bundle Report

Changes will increase total bundle size by 444 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-server-cjs 10.28MB 450 bytes (0.0%) ⬆️
sentry-docs-client-array-push 9.29MB 6 bytes (-0.0%) ⬇️


## How Uptime Tracing Works

Sentry Uptime Tracing automatically appends a `sentry-trace` header to every outgoing request made to the configured URL in your Uptime Alert settings. This header propagates trace information.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Sentry Uptime Tracing automatically appends a `sentry-trace` header to every outgoing request made to the configured URL in your Uptime Alert settings. This header propagates trace information.
Sentry Uptime Tracing automatically appends a `sentry-trace` header to every outgoing request made to the configured URL in your Uptime Alert settings. This header propagates a trace identifier.

maybe more obvious?

Comment on lines +30 to +34
<Alert level="warning" title="Important">

Uptime check requests do not override your SDK's error sampling rate. If your SDK’s error sample rate is set below 1, some errors from uptime check requests may not appear in the trace.

</Alert>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Alert level="warning" title="Important">
Uptime check requests do not override your SDK's error sampling rate. If your SDK’s error sample rate is set below 1, some errors from uptime check requests may not appear in the trace.
</Alert>
<Alert level="warning" title="Important">
Uptime check requests do not override your SDK's error sampling rate. If your SDK’s error sample rate is set below 1, some errors from uptime check requests may not appear in the trace.
</Alert>

I think you don't need these spaces anymore

Comment on lines +36 to +58
### Disabling Uptime Tracing of Errors

To disable error tracing for uptime checks, configure a [before send](/platform-redirect/?next=/configuration/filtering/) filter in your SDK to ignore errors from Sentry's `User-Agent`. Here's an example:

```typescript {tabTitle: Node.js Express} {filename: instrument.mjs}
import * as Sentry from "@sentry/node";

Sentry.init({
dsn: "___PUBLIC_DSN___",
// Filtering example for a Node.js Express app
beforeSend(event) {
const userAgent = event.request?.headers?.["user-agent"];

// Ignore events captured in a request from SentryUptimeBot
if (userAgent && userAgent.includes("SentryUptimeBot")) {
return null;
}

// Process other events
return event;
},
});
```
Copy link
Member

Choose a reason for hiding this comment

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

This section feels a little high, since I imagine most people don't want to disable error tracing

Comment on lines +72 to +76
<Alert level="warning" title="Important">

Uptime check requests do not override your SDK's trace sampling rate. If your SDK's trace sample rate is set below 1, some spans from uptime check requests may not appear in the trace.

</Alert>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Alert level="warning" title="Important">
Uptime check requests do not override your SDK's trace sampling rate. If your SDK's trace sample rate is set below 1, some spans from uptime check requests may not appear in the trace.
</Alert>
<Alert level="warning" title="Important">
Uptime check requests do not override your SDK's trace sampling rate. If your SDK's trace sample rate is set below 1, some spans from uptime check requests may not appear in the trace.
</Alert>


## Billing Considerations

Captured errors and spans from uptime check requests are billed as regular events in Sentry.
Copy link
Member

Choose a reason for hiding this comment

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

Is there a page we can link to for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants