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 cinder capacity stats #89

Conversation

newrushbolt
Copy link
Contributor

Add metrics:
openstack_cinder_pool_capacity_free_gb
openstack_cinder_pool_capacity_total_gb

@newrushbolt newrushbolt force-pushed the feature/add-cinder-capaity-stats branch 4 times, most recently from 4317b52 to f894ef1 Compare March 17, 2020 06:38
@newrushbolt newrushbolt changed the title Feature/add cinder capaity stats Add cinder capacity stats Mar 17, 2020
@newrushbolt newrushbolt mentioned this pull request Mar 17, 2020
@newrushbolt newrushbolt force-pushed the feature/add-cinder-capaity-stats branch from a0a511c to 50f48c8 Compare March 17, 2020 14:42
@newrushbolt
Copy link
Contributor Author

Rebased from master due to Update and clean versions commit merged.

@newrushbolt
Copy link
Contributor Author

newrushbolt commented Mar 24, 2020

@niedbalski
Can you have a look at this PR and #91 ?

@niedbalski
Copy link
Member

@newrushbolt Hey Sergei, pretty solid contribution, thanks. Do you mind to add these metrics into the README.md for reference?

@niedbalski niedbalski added this to the 1.0.0 milestone Mar 27, 2020
@niedbalski niedbalski merged commit bd991bc into openstack-exporter:master Mar 27, 2020
niedbalski added a commit that referenced this pull request Mar 27, 2020
Fixes and PR(s) merged.

f1b6596 Fix a bug can cause duplicate metrics and improve the implementation (#93)
a96e106 Add disabled reason label for Nova and Cinder (#91)
bd991bc Add cinder capacity metrics (#89)
f611a68 fix failed tests by adding tenant_id (#94)
c99b822 Added label tenant_id to limits_vcpus_max, limits_vcpus_used, limits_memory_max, limits_memory_used (#92)
767d43f Add log level flag (#90)
9282693 Added check on FIPs associated but not active (#85)
d431ae6 Loadbalancer (#81)
f3c8143 New Neutron metrics + inconsistency checks (#82)
ecad378 Improve testing and add ip_version to availabilities (#77)

Signed-off-by: Jorge Niedbalski <[email protected]>
niedbalski added a commit that referenced this pull request Mar 27, 2020
Fixes and PR(s) merged.

f1b6596 Fix a bug can cause duplicate metrics and improve the implementation (#93)
a96e106 Add disabled reason label for Nova and Cinder (#91)
bd991bc Add cinder capacity metrics (#89)
f611a68 fix failed tests by adding tenant_id (#94)
c99b822 Added label tenant_id to limits_vcpus_max, limits_vcpus_used, limits_memory_max, limits_memory_used (#92)
767d43f Add log level flag (#90)
9282693 Added check on FIPs associated but not active (#85)
d431ae6 Loadbalancer (#81)
f3c8143 New Neutron metrics + inconsistency checks (#82)
ecad378 Improve testing and add ip_version to availabilities (#77)

Signed-off-by: Jorge Niedbalski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants