Skip to content

[APM] Clarify TBS storage limit indexing destination and TTL expiry behavior#5616

Merged
alexandra5000 merged 3 commits into
elastic:mainfrom
alexandra5000:docs/tail-based-sampling-ttl-storage
Mar 26, 2026
Merged

[APM] Clarify TBS storage limit indexing destination and TTL expiry behavior#5616
alexandra5000 merged 3 commits into
elastic:mainfrom
alexandra5000:docs/tail-based-sampling-ttl-storage

Conversation

@alexandra5000

Copy link
Copy Markdown
Contributor

Summary

Addresses two customer questions about tail-based sampling (TBS) behavior:

  • Storage limit: Clarifies that when the storage limit is reached and discard_on_write_failure is false (default), events bypass TBS and are indexed immediately to the traces-apm-* data streams.
  • TTL: Clarifies that trace events whose TTL expires before a sampling decision is made are silently dropped and never indexed.

Sourced from dev clarification in elastic/sdh-apm#1928.

Closes #631

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No
  1. If you answered "Yes" to the previous question, please specify the tool(s) and model(s) used (e.g., Google Gemini, OpenAI ChatGPT-4, etc.).

Tool(s) and model(s) used: Claude Sonnet 4.6, running inside Claude Code to double-check files to be updated in the repo + write PR description

@alexandra5000
alexandra5000 requested a review from a team as a code owner March 25, 2026 11:56
@alexandra5000
alexandra5000 requested a review from marclop March 25, 2026 11:58
@alexandra5000

Copy link
Copy Markdown
Contributor Author

Hey @marclop! Tagging you for review, since you were responding in the SDH issue. It's a minor change to the docs, would be great if you could take a look and double-check it.

@marclop marclop left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you!

@vishaangelova vishaangelova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@alexandra5000
alexandra5000 enabled auto-merge (squash) March 26, 2026 13:51
@alexandra5000
alexandra5000 merged commit 68b82ec into elastic:main Mar 26, 2026
6 checks passed
@alexandra5000
alexandra5000 deleted the docs/tail-based-sampling-ttl-storage branch March 26, 2026 13:52
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