-
Notifications
You must be signed in to change notification settings - Fork 355
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 storage status:degraded #769
Add storage status:degraded #769
Conversation
Codecov Report
@@ Coverage Diff @@
## master #769 +/- ##
==========================================
+ Coverage 70.32% 70.33% +0.01%
==========================================
Files 163 163
Lines 15875 15876 +1
Branches 1967 1967
==========================================
+ Hits 11164 11167 +3
Misses 4034 4034
+ Partials 677 675 -2
|
|
||
ALL = (NORMAL, OFFLINE, ABNORMAL) | ||
ALL = (NORMAL, OFFLINE, ABNORMAL, DEGRADED) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After adding this, where would use it? Would it be in another PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Storage devices, such as netapp and 3par and hnas storage devices, have the status value degraded. This enumeration can be added to more accurately describe the device status.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What this PR does / why we need it:
Add storage status:degraded
Which issue this PR fixes (optional, in fixes #(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note: