Expose Storage Metrics #363

jingxu97 · 2017-07-26T17:19:12Z

Feature Description

One-line feature description (can be used as a release note): Expose storage metrics to end users
Primary contact (assignee): @vkamra @jingxu97
Responsible SIGs: @kubernetes/sig-storage-feature-requests
Design proposal link (community repo): Proposal for Exposing Storage Metrics community#855
Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred: @derekwaynecarr, @vishh, @dashpole
Approver (likely from SIG/area to which feature belongs): @thockin, @vishh, @derekwaynecarr
Feature target (which target equals to which milestone):
- Stable release target (x.y): 1.8/1.9

tamalsaha · 2017-08-02T21:13:33Z

cc: @aerokite

Automatic merge from submit-queue Proposal for adding PVC info to VolumeStats Flushes out details for part 1 of the changes described in [#855](#855) Feature: [#363](kubernetes/enhancements#363)

Automatic merge from submit-queue (batch tested with PRs 51513, 51515, 50570, 51482, 51448) Add PVCRef to VolumeStats **What this PR does / why we need it**: For pod volumes that reference a PVC, add a PVCRef to the corresponding volume stat. This allows metrics to be indexed/queried by PVC name which is more user-friendly than Pod reference **Which issue this PR fixes** : [#363](kubernetes/enhancements#363) **Special notes for your reviewer**: **Release note**: ``` `VolumeStats` reported by the kubelet stats summary API (http://<node>:10255/stats/summary) now include a PVCRef field describing the PVC referenced by the volume (if any). ```

Automatic merge from submit-queue Expose PVC metrics via kubelet prometheus This depends on #51448, opening early though. second commit is mine and mostly a copy/paste job. implements metrics listed in here kubernetes/community#855 following method here kubernetes/community#930 (comment) **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: kubernetes/enhancements#363 **Special notes for your reviewer**: **Release note**: ```release-note PersistentVolumeClaim metrics like "volume_stats_inodes" and "volume_stats_capacity_bytes" are now reported via kubelet prometheus ```

idvoretskyi · 2017-09-12T14:38:21Z

@jingxu97 please, update the features tracking board with the relevant data.

jdumars · 2017-09-15T19:15:24Z

@jingxu97 any update on missing docs? PR is due today.

Automatic merge from submit-queue. 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 e2e test to verify PVC metrics Adds an e2e test for kubernetes/enhancements#363 fixes #52459

Flushes out details for part 1 of the changes described in kubernetes#855 Feature: kubernetes/enhancements#363

andyxning · 2017-11-02T00:41:10Z

@jingxu97 @vishh I am currently work on adding disk io quota to Kubernetes and adding disk io metrics to heapster. What is the currently status about this?

BTW, i will update heapster #1450 ASAP.

fejta-bot · 2018-01-31T01:32:33Z

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

fejta-bot · 2018-03-02T02:18:15Z

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten
/remove-lifecycle stale

fejta-bot · 2018-04-01T03:05:51Z

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

PravinMagdum · 2018-07-28T18:05:34Z

/reopen

k8s-ci-robot · 2018-07-28T18:05:34Z

@PravinMagdum: you can't re-open an issue/PR unless you authored it or you are assigned to it.

In response to this:

/reopen

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.

Proposal: Server-side `kubectl get`

Flushes out details for part 1 of the changes described in kubernetes/community#855 Feature: kubernetes#363

Automatic merge from submit-queue Proposal for adding PVC info to VolumeStats Flushes out details for part 1 of the changes described in [kubernetes#855](kubernetes/community#855) Feature: [kubernetes#363](kubernetes#363)

Flushes out details for part 1 of the changes described in kubernetes/community#855 Feature: kubernetes/enhancements#363