Skip to content

docs(dynamic-sampling): Revert to pre-custom-sample-rates state - #18707

Merged
shellmayr merged 3 commits into
masterfrom
constantinius/docs/revert-dynamic-sampling-am3
Jul 22, 2026
Merged

docs(dynamic-sampling): Revert to pre-custom-sample-rates state#18707
shellmayr merged 3 commits into
masterfrom
constantinius/docs/revert-dynamic-sampling-am3

Conversation

@constantinius

@constantinius constantinius commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

DESCRIBE YOUR PR

Reverts the dynamic sampling docs to the state before custom sample rates were added (commit 657da84).

Why: The custom sample rates feature was never made available on AM3 (spans-based billing) plans. Customers on AM3 read the docs, look for the settings screen, can't find it, and contact support. Since this feature is no longer offered to new customers, the docs should not describe it.

Removed:

  • Configuration of Custom Sample Rates (Default/Advanced modes, Project Preview)
  • Sample Rates for Distributed Traces
  • SDK versus Dynamic Sampling comparison
  • All custom sample rates UI screenshots (5 images)

Retained:

  • Overview (rewritten to match the original retention priorities framing)
  • Prerequisites
  • Dynamic Sampling Priorities (automatic, applies to all plans)
  • Deciding on Your SDK Sample Rate (linked from other pages)

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.)
  • Other deadline
  • None: Not urgent, can wait up to 1 week+

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

Closes TET-2499

Remove the custom sample rates documentation that was added in
657da84 (feat(dynamic-sampling): organization settings). This feature
was never made available on AM3 (spans-based billing) plans, causing
confusion and support tickets from customers who read about it but
cannot find the settings screen.

Removed sections:
- Configuration of Custom Sample Rates (Default/Advanced modes)
- Sample Rates for Distributed Traces
- SDK versus Dynamic Sampling comparison
- All custom sample rates UI screenshots

Retained sections:
- Overview (rewritten to match pre-657da84a4 retention priorities framing)
- Prerequisites
- Dynamic Sampling Priorities (automatic, applies to all plans)
- Deciding on Your SDK Sample Rate (linked from other pages)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview Jul 22, 2026 12:18pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Preview Jul 22, 2026 12:18pm

Request Review

@linear-code

linear-code Bot commented Jul 14, 2026

Copy link
Copy Markdown

TET-2499

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

Looks good, thanks for keeping this up-to-date.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Redirect Chain Issues Detected

This PR introduces or contains redirect chains that degrade SEO and add latency.

Content Links Pointing to Redirects (1 found)

These links point to URLs that redirect. Update them to point directly to the final destination:

File Line Current Link Should Be
docs/organization/dynamic-sampling/index.mdx 28 /product/performance/metrics/ /product/dashboards/sentry-dashboards/

Each redirect hop loses ~15% of SEO link equity and adds latency for users.

@shellmayr
shellmayr enabled auto-merge (squash) July 22, 2026 12:12
@shellmayr
shellmayr merged commit f8251c0 into master Jul 22, 2026
23 of 24 checks passed
@shellmayr
shellmayr deleted the constantinius/docs/revert-dynamic-sampling-am3 branch July 22, 2026 12:44
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.

3 participants