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

feat(dynamic-sampling): organization settings #11791

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

constantinius
Copy link

@constantinius constantinius commented Nov 12, 2024

Add section under organization settings to explain the benefits of dynamic sampling and its relevant settings

Closes https://github.com/getsentry/projects/issues/214

Copy link

vercel bot commented Nov 12, 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 14, 2024 10:55am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Nov 14, 2024 10:55am
develop-docs ⬜️ Ignored (Inspect) Visit Preview Nov 14, 2024 10:55am

Copy link

codecov bot commented Nov 12, 2024

Bundle Report

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

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

Copy link
Contributor

@lizokm lizokm left a comment

Choose a reason for hiding this comment

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

Thanks so much for starting on this! I know it's still in draft. Let me know when you'd like me to take another look. Also, take a look at our current DS docs to see if there's anything we can copy over. (https://docs.sentry.io/product/performance/retention-priorities/)

docs/organization/dynamic-sampling/index.mdx Outdated Show resolved Hide resolved
docs/organization/dynamic-sampling/index.mdx Outdated Show resolved Hide resolved
docs/organization/dynamic-sampling/index.mdx Outdated Show resolved Hide resolved
constantinius and others added 4 commits November 14, 2024 10:22
Co-authored-by: Liza Mock <liza.mock@sentry.io>
Co-authored-by: Liza Mock <liza.mock@sentry.io>
Co-authored-by: Liza Mock <liza.mock@sentry.io>
Copy link
Member

@jan-auer jan-auer left a comment

Choose a reason for hiding this comment

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

Some ideas for information that could be added:

  • In the introduction, explain what "dynamic sampling" truly is. As in, what about it is dynamic and what it tries to achieve.
    • How does DS fit with sampling in the SDKs and when do I use which?
    • What is the default (100% = no sampling)
    • How does DS re-compute rates? Note that here we shouldn't give any strict guarantees (e.g. "every 10 minutes") but mention that there's a short sliding window so that DS can adjust to seasonality like on weekends and over night traffic patterns.
  • AM2 and AM3 have different capabilities. The sample rates are chosen automatically on AM2, while only on AM3 do you have the new settings screen and control over rates. We should make this clear. @lizokm, how do we usually talk about the plan generations?
  • I'd structure this document into multiple larger blocks (not necessarily sections):
    • Organization-wide settings, including the sampling mode, project preview, sample rates
    • Project-wide settings, which for now are just the priorities
    • Connection to other product surface, esp subscription settings and stats
  • We can expand the description of the modes, particularly
    • How does DS work in each of the modes (i.e. what's the same and what's different)
    • How do I think about configuring each of the modes?
    • Especially in manual mode, describe the org-level assist. That's where we could use an arcade.
    • What happens when I change into this mode?
  • On project preview, what do the estimated rates mean, what's the difference between the two time frames, and how do they relate to what DS actually does?
  • Retention priorities are now called "[Dynamic] Sampling Priorities", so let's rename them here, too.

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