Skip to content

Add health indicator impact to HealthPeriodicLogger#122390

Merged
samxbr merged 7 commits intoelastic:mainfrom
samxbr:feature/slm-health-logger
Feb 21, 2025
Merged

Add health indicator impact to HealthPeriodicLogger#122390
samxbr merged 7 commits intoelastic:mainfrom
samxbr:feature/slm-health-logger

Conversation

@samxbr
Copy link
Contributor

@samxbr samxbr commented Feb 12, 2025

This change adds the indicator impact to the output of HealthPeriodicLogger when indicator is not GREEN. This provides more context to the outputted logs, and specific alerts can be created based on the impact.

@samxbr samxbr requested review from dakrone and mattc58 February 13, 2025 08:27
@samxbr samxbr force-pushed the feature/slm-health-logger branch from a808021 to 8e7ba91 Compare February 20, 2025 17:13
@samxbr samxbr changed the title Add logs for SLM health indicator missing snapshot Feb 20, 2025
@samxbr samxbr marked this pull request as ready for review February 20, 2025 19:00
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Feb 20, 2025
@samxbr samxbr added the :Distributed/Health Issues for the health report API label Feb 20, 2025
@elasticsearchmachine elasticsearchmachine added Team:Data Management (obsolete) DO NOT USE. This team no longer exists. and removed needs:triage Requires assignment of a team area label labels Feb 20, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@samxbr samxbr added >enhancement needs:triage Requires assignment of a team area label and removed Team:Data Management (obsolete) DO NOT USE. This team no longer exists. labels Feb 20, 2025
@elasticsearchmachine elasticsearchmachine added Team:Data Management (obsolete) DO NOT USE. This team no longer exists. and removed needs:triage Requires assignment of a team area label labels Feb 20, 2025
@elasticsearchmachine
Copy link
Collaborator

Hi @samxbr, I've created a changelog YAML for you.

Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Sam!

@samxbr samxbr merged commit 4233310 into elastic:main Feb 21, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Health Issues for the health report API >enhancement Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v9.1.0

3 participants