Server Health: Check if logs are resolved
When determining the server health, check if the logs are
resolved. A resolved log should not cause the server health to
display "Warning" or "Critical".
Change-Id: I9bd46150410719441876b03c129cdf2e280150d4
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
diff --git a/app/common/services/dataService.js b/app/common/services/dataService.js
index 7a3c713..7ca13a6 100644
--- a/app/common/services/dataService.js
+++ b/app/common/services/dataService.js
@@ -98,15 +98,15 @@
};
this.updateServerHealth = function(logs) {
- // If any severity high logs are present, set server health to critical
- // Else if any severity medium logs are present set server health to
- // warning
+ // If any unresolved severity high logs are present, set server health
+ // to critical. Else if any unresolved severity medium logs are present
+ // set server health to warning.
this.server_health = Constants.SERVER_HEALTH.good;
for (var log of logs) {
- if (log.priority == 'High') {
+ if (log.priority == 'High' && !log.Resolved) {
this.server_health = Constants.SERVER_HEALTH.critical;
return;
- } else if (log.priority == 'Medium') {
+ } else if (log.priority == 'Medium' && !log.Resolved) {
this.server_health = Constants.SERVER_HEALTH.warning;
}
}