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

Bug 1506987: per_volume_gigabytes_used definition is missing #276

Merged
merged 1 commit into from
Apr 19, 2018

Conversation

sseago
Copy link
Contributor

@sseago sseago commented Apr 19, 2018

https://bugzilla.redhat.com/show_bug.cgi?id=1506987

For cloud resource quota reporting, per_volume_gigabytes_used is
missing, causing -1 to be shown instead of a real value.

The per_volume_gigabytes quota parameter is not a true quota limit,
since it's not an aggregate across volumes but a per-volume limit.
However, since OpenStack defines it as a quota param along with the
aggregate values, MIQ treats it in the same way as the other
parameters.

In this case, we're reporting the size of the largest volume, since
the limit is a per-volume limit rather than an aggregate one.

https://bugzilla.redhat.com/show_bug.cgi?id=1506987

For cloud resource quota reporting, per_volume_gigabytes_used is
missing, causing -1 to be shown instead of a real value.

The per_volume_gigabytes quota parameter is not a true quota limit,
since it's not an aggregate across volumes but a per-volume limit.
However, since OpenStack defines it as a quota param along with the
aggregate values, MIQ treats it in the same way as the other
parameters.

In this case, we're reporting the size of the largest volume, since
the limit is a per-volume limit rather than an aggregate one.
@miq-bot
Copy link
Member

miq-bot commented Apr 19, 2018

Checked commit sseago@6601f8b with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

@mansam mansam merged commit 6c0ac99 into ManageIQ:master Apr 19, 2018
@aufi aufi added the bug label Apr 24, 2018
@aufi aufi added this to the Sprint 84 Ending Apr 23, 2018 milestone Apr 24, 2018
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.

4 participants