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.

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

[Stack Monitoring] Fix shard size alerts #114357

Merged

Conversation

simianhacker
Copy link
Member

Summary

This PR fixes #114356 by removing the source_node data from the fetchIndexShardSize method. The source_node data is not being used in the evaluation code so I think it's safe to remove it.

@simianhacker simianhacker added release_note:fix v8.0.0 Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services Feature:Stack Monitoring v7.16.0 labels Oct 7, 2021
@simianhacker simianhacker requested a review from a team as a code owner October 7, 2021 20:42
@simianhacker simianhacker requested review from a team October 7, 2021 20:42
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@neptunian neptunian self-requested a review October 11, 2021 20:39
@simianhacker simianhacker added the auto-backport Deprecated - use backport:version if exact versions are needed label Oct 13, 2021
@simianhacker simianhacker enabled auto-merge (squash) October 13, 2021 16:02
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@simianhacker simianhacker merged commit 6d24de9 into elastic:master Oct 13, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 13, 2021
* [Stack Monitoring] Fix shard size alerts

* Removing the source filter for source_node.*

* Removing superfluous types

* Removing superfluous nodeId and nodeName from test
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Oct 13, 2021
* [Stack Monitoring] Fix shard size alerts

* Removing the source filter for source_node.*

* Removing superfluous types

* Removing superfluous nodeId and nodeName from test

Co-authored-by: Chris Cowan <chris@chriscowan.us>
artem-shelkovnikov pushed a commit to artem-shelkovnikov/kibana that referenced this pull request Oct 20, 2021
* [Stack Monitoring] Fix shard size alerts

* Removing the source filter for source_node.*

* Removing superfluous types

* Removing superfluous nodeId and nodeName from test
@simianhacker simianhacker deleted the issue-114356-fix-shard-size-alerts branch April 17, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Stack Monitoring release_note:fix Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v7.16.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Stack Monitoring] Shard size alert never fires
4 participants