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.

Sign up for GitHub

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

Jump to bottom

[Actionable Observability] [SLO] Remove context from SLI data #143310

Closed
Tracked by #137323
kdelemme opened this issue Oct 13, 2022 · 1 comment · Fixed by #143318
Closed
Tracked by #137323

[Actionable Observability] [SLO] Remove context from SLI data #143310

kdelemme opened this issue Oct 13, 2022 · 1 comment · Fixed by #143318
Assignees
@kdelemme
Labels
release_note:skip Skip the PR/issue when compiling release notes Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.6.0

Comments

@kdelemme
Copy link
Contributor

📝 Summary

Part of #137323

The SLI data aggregated by the transform contains a "context" object containing the filter query used for the aggregation. There is no use case for it at the moment, and this data can be retrieved from the SLO definition itself. Thus storing only the slo id, slo revision, numerator, denominator (and isGoodSlice) is sufficient and will make the index smaller.
@kdelemme kdelemme added the Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" label Oct 13, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

@kdelemme kdelemme added release_note:skip Skip the PR/issue when compiling release notes v8.6.0 labels Oct 13, 2022
This was referenced Oct 13, 2022
Closed
Merged
@kdelemme kdelemme self-assigned this Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kdelemme kdelemme

Labels
release_note:skip Skip the PR/issue when compiling release notes Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.6.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore(slo): Remove context in aggregated data kdelemme/kibana
2 participants
@kdelemme @elasticmachine