-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add healthcheck to report to zui for errors #2143
Conversation
This view is a bit confusing, since it shows |
It is showing OK because a check for Readonly cache hasn't been added yet, it will be added with the other remaining changes. A separate section for errors in ZOS node is more helpful to get more details of what is wrong without cluttering the first section (it already has a lot of information) and to support multiple health checks. But that's my opinion. |
Okay, that makes sense, @AbdelrahmanElawady. In that case it looks good to me. |
Description
Add healthcheck task to execute multiple checks (only cache for now) and report to zui for errors. It also set flags that can be checked by Power daemon to stop uptime reports.
Changes
Related Issues
Checklist