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

Cashe cloud volumes in ChargebackVm #17585

Merged

Conversation

lpichler
Copy link
Contributor

@lpichler lpichler commented Jun 14, 2018

Invoke cloud volume query only when report is run.
Previously query was invoked each time when when method ChargebackVm.attribute_names was called.

Based on @NickLaMuro's code and findings. Thanks!

Stats are as part of bigger issue. (Will be added lately)

@miq-bot assign @gtanzillo

Links

https://bugzilla.redhat.com/show_bug.cgi?id=1566452
one of 5 PRs for a fix.

@lpichler lpichler closed this Jun 14, 2018
@lpichler lpichler force-pushed the cash_cloud_volumes_in_chargeback branch from 1a0f74b to 887cc81 Compare June 14, 2018 16:25
@lpichler lpichler reopened this Jun 14, 2018
@lpichler lpichler force-pushed the cash_cloud_volumes_in_chargeback branch 2 times, most recently from 744c8c1 to 355cece Compare June 14, 2018 17:11
@lpichler
Copy link
Contributor Author

@miq-bot add_label blocker, chargeback, perfomance

@miq-bot
Copy link
Member

miq-bot commented Jun 14, 2018

@lpichler Cannot apply the following label because they are not recognized: perfomance

@lpichler lpichler force-pushed the cash_cloud_volumes_in_chargeback branch from 108ae04 to ce3bb5d Compare June 14, 2018 19:02
@miq-bot
Copy link
Member

miq-bot commented Jun 14, 2018

Checked commit lpichler@ce3bb5d with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 0 offenses detected
Everything looks fine. 🍪

@gtanzillo gtanzillo added this to the Sprint 88 Ending Jun 18, 2018 milestone Jun 14, 2018
@gtanzillo gtanzillo merged commit 4e53336 into ManageIQ:master Jun 14, 2018
@NickLaMuro
Copy link
Member

🤑

("Cash" v.s. "Cache" in PR title)

😉

@lpichler lpichler changed the title Cash cloud volumes in ChargebackVm Cashe cloud volumes in ChargebackVm Jun 15, 2018
@lpichler lpichler deleted the cash_cloud_volumes_in_chargeback branch June 15, 2018 15:03
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.

4 participants