Skip to content
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 kubernetes.volume.fs.used.pct field #23564

Merged
merged 2 commits into from
Jan 20, 2021

Conversation

ChrsMark
Copy link
Member

What does this PR do?

This PR adds kubernetes.volume.fs.used.pct field. This field is calculated as kubernetes.volume.fs.used.bytes / kubernetes.volume.fs.capacity.bytes

Why is it important?

Provide used storage insight with a direct metric.

Closes #23016

Signed-off-by: chrismark <chrismarkou92@gmail.com>
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 19, 2021
@ChrsMark ChrsMark marked this pull request as ready for review January 19, 2021 10:52
@elasticmachine
Copy link
Collaborator

elasticmachine commented Jan 19, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #23564 updated

    • Start Time: 2021-01-20T11:11:09.386+0000
  • Duration: 27 min 57 sec

  • Commit: 335162f

Test stats 🧪

Test Results
Failed 0
Passed 1992
Skipped 470
Total 2462

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 1992
Skipped 470
Total 2462

@andresrc andresrc added the Team:Integrations Label for the Integrations team label Jan 19, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jan 19, 2021
@ChrsMark ChrsMark self-assigned this Jan 20, 2021
@ChrsMark ChrsMark added v7.12.0 needs_backport PR is waiting to be backported to other branches. labels Jan 20, 2021
Copy link
Contributor

@sayden sayden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changelog is missing

Signed-off-by: chrismark <chrismarkou92@gmail.com>
Copy link
Contributor

@sayden sayden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! If CI is happy, we are good to merge

@ChrsMark ChrsMark merged commit 29b440f into elastic:master Jan 20, 2021
ChrsMark added a commit to ChrsMark/beats that referenced this pull request Jan 20, 2021
* Add kubernetes.volume.fs.used.pct field

Signed-off-by: chrismark <chrismarkou92@gmail.com>

* Add changelog entry

Signed-off-by: chrismark <chrismarkou92@gmail.com>
(cherry picked from commit 29b440f)
@ChrsMark ChrsMark removed the needs_backport PR is waiting to be backported to other branches. label Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Integrations Label for the Integrations team v7.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add field kubernetes.volume.fs.used.pct for used storage
4 participants