Skip to content

chore(ci): update slos#17205

Open
faydef wants to merge 3 commits intomainfrom
fayssal/update-slos
Open

chore(ci): update slos#17205
faydef wants to merge 3 commits intomainfrom
fayssal/update-slos

Conversation

@faydef
Copy link
Copy Markdown

@faydef faydef commented Mar 31, 2026

Description

This PR updates SLOs that were identified as optimizable during OE review in Febuary, 2026. PR sets SLOs at ca. 10% higher than SLIs collected from actual run, to make sure gates are not flaky, but still able to catch regression if it happens.

These SLOs were set too loosely and thus would not catch potential regressions.
Example
image

Testing

Risks

Additional Notes

@faydef faydef requested review from a team as code owners March 31, 2026 07:05
@faydef faydef changed the title update slos chore(ci):update slos Mar 31, 2026
@faydef faydef changed the title chore(ci):update slos chore(ci): update slos Mar 31, 2026
# - WARNING: 90% CI boundary crosses warning_threshold (constant)
#
# DO NOT EDIT MANUALLY - Regenerate using:
# benchmark_analyzer generate slos --help
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.

we might need more details here or in some docs/ somewhere, how do we install/use this tool?

Copy link
Copy Markdown
Author

@faydef faydef Mar 31, 2026

Choose a reason for hiding this comment

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

Sure thing!
Here is a link to the docs I will add it to the slo file itself!

@datadog-prod-us1-3

This comment has been minimized.

@cit-pr-commenter-54b7da
Copy link
Copy Markdown

Codeowners resolved as

.gitlab/benchmarks/bp-runner.macrobenchmarks.fail-on-breach.yml         @DataDog/python-guild @DataDog/apm-core-python

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.

2 participants