-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Detailed storage metrics of internal state #496
Comments
@gnufied can you fill the "feature target" section please? |
Do we want to add a metric for the number of PVs which are attached to unresponsive hosts? This situation indicates bad cluster health which currently requires manual intervention to resolve. |
@aglitke that sounds like a good idea, we will add it to the google doc. |
@idvoretskyi since these are basically metrics they go to GA. |
@gnufied based on the conversation above, marking as "stable". Can you confirm? |
@idvoretskyi yes that is correct. |
@gnufied 👋 Please indicate in the 1.9 feature tracking board |
@gnufied Bump for docs ☝️ /cc @idvoretskyi |
We didn't do this for 1.9. Lets bump the milestone for next release. |
Automatic merge from submit-queue (batch tested with PRs 58317, 58687, 57872, 59063, 59328). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. add number measurement for bound/unbound pv/pvc **What this PR does / why we need it**: Implement number measurement for bound/unbound pv/pvc defined in the [Metrics Spec](https://docs.google.com/document/d/1Fh0T60T_y888LsRwC51CQHO75b2IZ3A34ZQS71s_F0g/edit#heading=h.bwzmc2tktae) ref feature: [ kubernetes/enhancements#496](kubernetes/enhancements#496) **Release note**: ```release-note Intended for post-1.9 ```
@gnufied Can you please update docs for 1.10? Also 1.10 feature tracking spreadsheet |
This does not need to be tracked for 1.10 milestone |
@saad-ali can you provide the estimated roadmap for the feature? |
This feature was specifically mentioned in 1.10 release notes, but was it actually done or just planned to be done in 1.10 at some point? |
@redbaron This feature in general tracks, general improvement to metrics we emit from storage components. In 1.10 we added additional metrics (namely bound/unbound volume count). In 1.11 the plan is to introduce new metrics that expose problematic state for ease of debugging. |
@redbaron the release notes fix was pushed, thanks for noticing - kubernetes/kubernetes#61840 @gnufied per @saad-ali the progress that was made on this feature hasn't been major enough to be tracked under the features list of 1.10 release. |
@saad-ali @k8s-mirror-storage-feature-requests any plans for 1.11? |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
/remove-lifecycle stale |
This feature current has no milestone, so we'd like to check in and see if there are any plans for this in Kubernetes 1.12. If so, please ensure that this issue is up-to-date with ALL of the following information:
Set the following:
Once this feature is appropriately updated, please explicitly ping @justaugustus, @kacole2, @robertsandoval, @rajendar38 to note that it is ready to be included in the Features Tracking Spreadsheet for Kubernetes 1.12. Please note that Features Freeze is tomorrow, July 31st, after which any incomplete Feature issues will require an Exception request to be accepted into the milestone.In addition, please be aware of the following relevant deadlines:
Please make sure all PRs for features have relevant release notes included as well. Happy shipping! P.S. This was sent via automation |
This issue hasn't had any attention for 1.11 or 1.12. I'm going to close it since the k/commnunity proposal has been closed as well. /close |
@kacole2: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Feature Description
The text was updated successfully, but these errors were encountered: