-
Notifications
You must be signed in to change notification settings - Fork 7
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 some new metrics for pod ephemeral storage #3
Add some new metrics for pod ephemeral storage #3
Conversation
@ribbybibby Could you review this 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.
Hi @superbrothers. Thanks for the contribution! It looks good.
A few things:
- Can you move the manifests to a separate PR? I'd like to review those separately.
- Could you implement the
inodes
metrics, like we do for the other fsstats?
a37ca88
to
2e24b9e
Compare
I've moved sample manifest files to #6. |
Yes, but I'm going to do it in a different PR. |
How come? Seems like it would fit into this one. |
231fd47
to
cfc51a6
Compare
I misread your comment:bow: I have added the following metrics:
|
cfc51a6
to
7e91d78
Compare
- `kube_summary_pod_ephemeral_storage_available_bytes`: Number of bytes of Ephemeral storage that aren't consumed by the pod - `kube_summary_pod_ephemeral_storage_capacity_bytes`: Number of bytes of Ephemeral storage that can be consumed by the pod - `kube_summary_pod_ephemeral_storage_used_bytes`: Number of bytes of Ephemeral storage that are consumed by the pod - `kube_summary_pod_ephemeral_storage_inodes_free`: Number of available Inodes for pod Ephemeral storage - `kube_summary_pod_ephemeral_storage_inodes`: Number of Inodes for pod Ephemeral storage - `kube_summary_pod_ephemeral_storage_inodes_used`: Number of used Inodes for pod Ephemeral storage
7e91d78
to
c8e6155
Compare
This PR adds some new metrics for pod ephemeral storage.