Implement Health Check for Microservices Observability (#2695) #2774
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The commit introduces Health Check pattern, providing a series of health indicators for system performance and stability monitoring, including checks for system CPU load, process CPU load, database health, memory usage, and garbage collection metrics. It also includes asynchronous execution and caching mechanisms for health checks, and retry configurations for resilience. Implements health checking components as per issue #2695.
…olved or not in CI.
ADDED HIGH_PROCESS_CPU_LOAD_MESSAGE_WITHOUT_PARAM ADDED HIGH_SYSTEM_CPU_LOAD_MESSAGE_WITHOUT_PARAM
…f using a generic one." added HealthCheckInterruptedException refactored CustomHealthIndicator
|
Kudos, SonarCloud Quality Gate passed! |
iluwatar
approved these changes
Dec 2, 2023
Owner
iluwatar
left a comment
iluwatar
left a comment
There was a problem hiding this comment.
The code looks great! The only thing worth mentioning is the explanation section of the README.md. We need to make it more verbose and structure according https://github.com/iluwatar/java-design-patterns/wiki/02.-Pattern-template, but let's do that in a follow-up issue
Owner
|
@all-contributors please add @ylcn91 for code |
Contributor
|
I've put up a pull request to add @ylcn91! 🎉 |
Owner
|
Here is the follow-up issue: #2782 |
Contributor
Author
|
Thank you ! |
Contributor
Author
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The commit introduces the Health Check pattern, providing a series of health indicators for system performance and stability monitoring, including checks for system CPU load, process CPU load, database health, memory usage, and garbage collection metrics. It also includes asynchronous execution and caching mechanisms for health checks, and retry configurations for resilience.
Implements health-checking components as per issue #2695.
This pull request introduces the Health Check pattern, an essential feature for the observability of microservices. The implementation allows services within a system to regularly check their health status and report it back for monitoring purposes. The health indicators cover various aspects such as system CPU load, process CPU load, database connectivity, memory usage, and garbage collection status. These indicators are crucial for proactive system maintenance and resilience in a microservice architecture.
Changes included in this pull request:
This implementation addresses the issue reported in #2695 and follows the guidelines provided in the How to Contribute section of the repository's wiki.
Additional notes:
/etcfolder for a visual understanding of the pattern.