Skip to content

[8.17] TBS: Add sampling.tail.discard_on_write_failure config (backport #15159)#15172

Merged
mergify[bot] merged 1 commit into8.17from
mergify/bp/8.17/pr-15159
Jan 7, 2025
Merged

[8.17] TBS: Add sampling.tail.discard_on_write_failure config (backport #15159)#15172
mergify[bot] merged 1 commit into8.17from
mergify/bp/8.17/pr-15159

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jan 7, 2025

Motivation/summary

Add config sampling.tail.discard_on_write_failure (default=false) for users to opt in to data loss when TBS storage limit is reached, as an escape hatch when increased ES indexing load is unacceptable.

Checklist

- [ ] Update CHANGELOG.asciidoc backported
- [ ] Documentation has been updated intentionally undocumented

For functional changes, consider:

  • Is it observable through the addition of either logging or metrics?
  • Is its use being published in telemetry to enable product improvement?
  • Have system tests been added to avoid regression?

How to test these changes

Enable TBS, trigger storage limit reached, check if behavior aligns with config = {true, false}

Related issues

Fixes #11127


This is an automatic backport of pull request #15159 done by [Mergify](https://mergify.com).

Add config sampling.tail.discard_on_write_failure (default=false) for users to opt in to data loss when TBS storage limit is reached, as an escape hatch when increased ES indexing load is unacceptable.

(cherry picked from commit 9b40574)
@mergify mergify bot requested a review from a team as a code owner January 7, 2025 22:22
@mergify mergify bot added the backport label Jan 7, 2025
@carsonip
Copy link
Member

carsonip commented Jan 7, 2025

@Mergifyio requeue

@mergify
Copy link
Contributor Author

mergify bot commented Jan 7, 2025

requeue

❌ This pull request head commit has not been previously disembarked from queue.

@mergify mergify bot merged commit 4177fba into 8.17 Jan 7, 2025
@mergify mergify bot deleted the mergify/bp/8.17/pr-15159 branch January 7, 2025 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant