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

Cloudwatch: Add MeteredIOBytes metric for EFS #64793

Conversation

xlagg5
Copy link
Contributor

@xlagg5 xlagg5 commented Mar 15, 2023

What is this feature?

Adds missing metric from EFS Cloudwatch which is MeteredIOBytes.

Why do we need this feature?

We need this from improved CloudWatch plugin.

Who is this feature for?

For Cloudwatch EFS users.

Which issue(s) does this PR fix?:

Fixes #

Special notes for your reviewer:

@xlagg5 xlagg5 requested a review from a team as a code owner March 15, 2023 08:16
@xlagg5 xlagg5 requested review from sarahzinger and idastambuk and removed request for a team March 15, 2023 08:16
@CLAassistant
Copy link

CLAassistant commented Mar 15, 2023

CLA assistant check
All committers have signed the CLA.

@grafanabot grafanabot added area/backend datasource/CloudWatch pr/external This PR is from external contributor labels Mar 15, 2023
@sarahzinger sarahzinger added this to the 9.5.0 milestone Mar 20, 2023
@sarahzinger
Copy link
Member

@xlagg5 thank you for your PR! Would you be able to sign our CLA so we might be able to merge this change?

@xlagg5 xlagg5 force-pushed the feature/add-MeteredIOBytes-metric-to-cloudwatch branch from 82cebee to 9b323b1 Compare March 21, 2023 09:57
@xlagg5
Copy link
Contributor Author

xlagg5 commented Mar 21, 2023

@xlagg5 thank you for your PR! Would you be able to sign our CLA so we might be able to merge this change?

Done!

Copy link
Member

@sarahzinger sarahzinger left a comment

Choose a reason for hiding this comment

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

@xlagg5 thanks for signing the CLA! Taking a closer look and I'm wondering, does this work for you? It looks like it's under EC2Spot and not under EFS. Mind taking another look?

@xlagg5 xlagg5 force-pushed the feature/add-MeteredIOBytes-metric-to-cloudwatch branch from 9b323b1 to e4d92d3 Compare March 21, 2023 13:50
@xlagg5
Copy link
Contributor Author

xlagg5 commented Mar 21, 2023

@xlagg5 thanks for signing the CLA! Taking a closer look and I'm wondering, does this work for you? It looks like it's under EC2Spot and not under EFS. Mind taking another look?

Oh no! It was a mistake. Now it is fine.

@xlagg5 xlagg5 requested review from sarahzinger and removed request for sarahzinger March 23, 2023 09:28
@sarahzinger sarahzinger enabled auto-merge (squash) March 23, 2023 16:03
@sarahzinger sarahzinger merged commit d5dacf5 into grafana:main Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants