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

Refactor used quota method and test. #191

Merged
merged 1 commit into from
Oct 5, 2017

Conversation

billfitzgerald0120
Copy link
Contributor

This PR is based on on the issue below.
ManageIQ/manageiq#12038

@miq-bot add_label refactoring

@billfitzgerald0120
Copy link
Contributor Author

@miq-bot assign @gmcculloug

@billfitzgerald0120
Copy link
Contributor Author

@tinaafitz Please Review

@billfitzgerald0120
Copy link
Contributor Author

@tinaafitz Made changes as requested, Please Review

This PR is based on on the issue below.
ManageIQ/manageiq#12038
@miq-bot
Copy link
Member

miq-bot commented Oct 5, 2017

Checked commit billfitzgerald0120@9cd896a with ruby 2.3.1, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 1 offense detected

spec/content/automate/ManageIQ/System/CommonMethods/QuotaMethods.class/methods/used_spec.rb

@mkanoor mkanoor merged commit 5cc73ae into ManageIQ:master Oct 5, 2017
@mkanoor mkanoor added this to the Sprint 71 Ending Oct 16, 2017 milestone Oct 5, 2017
@billfitzgerald0120
Copy link
Contributor Author

@miq-bot add_label fine/yes

@simaishi
Copy link
Contributor

simaishi commented Nov 20, 2017

@billfitzgerald0120 Is there a BZ for this issue?

Never mind... found it in #196 (comment)
https://bugzilla.redhat.com/show_bug.cgi?id=1489507

simaishi pushed a commit that referenced this pull request Nov 20, 2017
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 137152446e08d45ccaaf326715e95cae169603f8
Author: Madhu Kanoor <[email protected]>
Date:   Thu Oct 5 09:59:18 2017 -0400

    Merge pull request #191 from billfitzgerald0120/used_active_provisions
    
    Refactor used quota method and test.
    (cherry picked from commit 5cc73ae823ff9ec8ec3bf2c4253cb8f416a25fc8)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1489507

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.

5 participants