Skip to content

[APM] integration new tail sampling settings#224479

Merged
cauemarcondes merged 1 commit intoelastic:mainfrom
cauemarcondes:apm-ttl-config
Jun 19, 2025
Merged

[APM] integration new tail sampling settings#224479
cauemarcondes merged 1 commit intoelastic:mainfrom
cauemarcondes:apm-ttl-config

Conversation

@cauemarcondes
Copy link
Contributor

@cauemarcondes cauemarcondes commented Jun 18, 2025

closes #219303
closes #221441

New settings:

  • tail_sampling_ttl
  • tail_sampling_discard_on_write_failure
Screenshot 2025-06-18 at 13 06 29

@cauemarcondes cauemarcondes requested a review from a team June 18, 2025 17:08
@botelastic botelastic bot added the Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. label Jun 18, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@isaacaflores2
Copy link

@cauemarcondes Will the discard on write failure toggle be set to false by default? Just double checking since my guess is you set it to true before you took the screenshot

@cauemarcondes
Copy link
Contributor Author

@cauemarcondes Will the discard on write failure toggle be set to false by default? Just double checking since my guess is you set it to true before you took the screenshot

yes, it'll use wherever is defined on the setting definition. Indeed I took a screenshot after enabling it manually.

Copy link
Contributor

@MiriamAparicio MiriamAparicio left a comment

Choose a reason for hiding this comment

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

LGTM

@cauemarcondes cauemarcondes merged commit f857b34 into elastic:main Jun 19, 2025
22 checks passed
@cauemarcondes cauemarcondes deleted the apm-ttl-config branch June 19, 2025 13:52
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/15759545719

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 19, 2025
closes elastic#219303
closes elastic#221441

New settings:
- `tail_sampling_ttl`
- `tail_sampling_discard_on_write_failure`

<img width="728" alt="Screenshot 2025-06-18 at 13 06 29"
src="https://github.com/user-attachments/assets/dec21e4f-a6f8-411e-aadd-c96ab1186a6c"
/>

(cherry picked from commit f857b34)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jun 19, 2025
# Backport

This will backport the following commits from `main` to `8.19`:
- [[APM] integration new tail sampling settings
(#224479)](#224479)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Cauê
Marcondes","email":"55978943+cauemarcondes@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-06-19T13:52:48Z","message":"[APM]
integration new tail sampling settings (#224479)\n\ncloses
https://github.com/elastic/kibana/issues/219303\ncloses
https://github.com/elastic/kibana/issues/221441\n\nNew settings:\n-
`tail_sampling_ttl`\n- `tail_sampling_discard_on_write_failure`\n\n<img
width=\"728\" alt=\"Screenshot 2025-06-18 at 13 06
29\"\nsrc=\"https://github.com/user-attachments/assets/dec21e4f-a6f8-411e-aadd-c96ab1186a6c\"\n/>","sha":"f857b34115cb07c77bfdd8634f2e2cc7ee91d9ba","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","Team:obs-ux-infra_services","backport:version","v9.1.0","v8.19.0"],"title":"[APM]
integration new tail sampling
settings","number":224479,"url":"https://github.com/elastic/kibana/pull/224479","mergeCommit":{"message":"[APM]
integration new tail sampling settings (#224479)\n\ncloses
https://github.com/elastic/kibana/issues/219303\ncloses
https://github.com/elastic/kibana/issues/221441\n\nNew settings:\n-
`tail_sampling_ttl`\n- `tail_sampling_discard_on_write_failure`\n\n<img
width=\"728\" alt=\"Screenshot 2025-06-18 at 13 06
29\"\nsrc=\"https://github.com/user-attachments/assets/dec21e4f-a6f8-411e-aadd-c96ab1186a6c\"\n/>","sha":"f857b34115cb07c77bfdd8634f2e2cc7ee91d9ba"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/224479","number":224479,"mergeCommit":{"message":"[APM]
integration new tail sampling settings (#224479)\n\ncloses
https://github.com/elastic/kibana/issues/219303\ncloses
https://github.com/elastic/kibana/issues/221441\n\nNew settings:\n-
`tail_sampling_ttl`\n- `tail_sampling_discard_on_write_failure`\n\n<img
width=\"728\" alt=\"Screenshot 2025-06-18 at 13 06
29\"\nsrc=\"https://github.com/user-attachments/assets/dec21e4f-a6f8-411e-aadd-c96ab1186a6c\"\n/>","sha":"f857b34115cb07c77bfdd8634f2e2cc7ee91d9ba"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Cauê Marcondes <55978943+cauemarcondes@users.noreply.github.com>
@rubvs
Copy link

rubvs commented Jun 20, 2025

Tested manually:

  • Create a deployment with 9.1.0-SNAPSHOT
  • Spin up EA locally via Docker
  • Send some data with storage limit 1B and discard on write failure disabled
Screenshot 2025-06-20 at 4 29 24 PM - Observe correct output in Discovery Screenshot 2025-06-20 at 3 16 50 PM - Repeat with discard on write failure **enabled** Screenshot 2025-06-20 at 4 26 18 PM - Observe correct result in Discovery Screenshot 2025-06-20 at 4 27 00 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:enhancement Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. v8.19.0 v9.1.0

Projects

None yet

7 participants