Project

General

Profile

Feature #6651

UI: Review Health Widget to remove unnecessary Warning states

Added by Neil Levine almost 8 years ago. Updated almost 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Neil Levine
Category:
-
Target version:
% Done:

0%

Source:
other
Tags:
Backport:
Reviewed:
Affected Versions:

History

#1 Updated by Dan Mick almost 8 years ago

  • Status changed from 12 to Rejected

We've redesigned to avoid the health widget

#2 Updated by Neil Levine almost 8 years ago

  • Status changed from Rejected to 12

#3 Updated by Neil Levine almost 8 years ago

  • Target version changed from v1.1rc1 to v1.1rc2

#4 Updated by Neil Levine almost 8 years ago

  • Assignee changed from Dan Mick to Neil Levine

#5 Updated by John Spray almost 8 years ago

Aide memoire: the issue here is that if we remove the HEALTH_WARN etc state outright, we would fail to report some classes of error which aren't captured in our other widgets. The interface by which ceph gives us health data is problematic (because it spams us with large number of detail messages in some cases), but it's better to raise an error with no info than to pretend the system is healthy when it's not.

#6 Updated by Ian Colle almost 8 years ago

  • Subject changed from UI: Review Health Widget to remove unnecessary Error states to UI: Review Health Widget to remove unnecessary Warning states

#7 Updated by Ian Colle almost 8 years ago

  • Target version changed from v1.1rc2 to v1.1rc3

#8 Updated by Yan-Fa Li almost 8 years ago

  • translation missing: en.field_story_points set to 2.0

#9 Updated by Yan-Fa Li almost 8 years ago

Stupid question, but what defines unnecessary? Do we have a list of things we should ignore?

#10 Updated by Neil Levine almost 8 years ago

The proposal is to only show 2 states for the Health widget: OK/Green and ERROR/Red i.e. don't have a WARN/Orange State.

If Ceph Health returns OK, the show OK/Green, no context.
If Ceph Health returns ERROR, then show ERROR/Red, with the context line showing the information provided by Ceph.
If Ceph Health returns WARN, then show OK/Green, but use the context line to show the warning information provided by Ceph.

#11 Updated by Yan-Fa Li almost 8 years ago

Branch bs3-exp commit d68498ea67f41d1e4ee18c9c960d613b39bb5178

#12 Updated by Yan-Fa Li almost 8 years ago

  • Status changed from 12 to Resolved

Also available in: Atom PDF