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

Fix: make chargeback generation for service to be region aware and do not generate for retired service #19101

Merged

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Aug 5, 2019

Issue: chargeback generation was triggered for all services, regardless service been retired or belongs to different region.

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

@miq-bot add-label bug, core, changelog/yes, hammer/yes, ivanchuk/yes

\cc @gtanzillo

@yrudman yrudman force-pushed the make-chargeback-for-service-region-aware branch from 3071645 to b510356 Compare August 8, 2019 14:08
@yrudman yrudman changed the title Fix: make chargeback generation for service region aware and do not generate for retired service Fix: make chargeback generation for service to be region aware and do not generate for retired service Aug 8, 2019
@miq-bot
Copy link
Member

miq-bot commented Aug 8, 2019

Checked commits yrudman/manageiq@4622cf5~...b510356 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@bdunne
Copy link
Member

bdunne commented Aug 8, 2019

After discussion, this is only accessible from the API and the only consumer (SSUI) knows to go to each region to get the report.

@bdunne bdunne merged commit 3bcda04 into ManageIQ:master Aug 8, 2019
@bdunne bdunne self-assigned this Aug 8, 2019
@bdunne bdunne added this to the Sprint 118 Ending Aug 19, 2019 milestone Aug 8, 2019
simaishi pushed a commit that referenced this pull request Aug 8, 2019
…gion-aware

Fix: make chargeback generation for service to be region aware and do not generate for retired service
(cherry picked from commit 3bcda04)

https://bugzilla.redhat.com/show_bug.cgi?id=1736749
@simaishi
Copy link
Contributor

simaishi commented Aug 8, 2019

Ivanchuk backport details:

$ git log -1
commit 98b25ce54c302f31e3a3c7798eb045f48c1ba31f
Author: Brandon Dunne <bdunne@redhat.com>
Date:   Thu Aug 8 15:35:13 2019 -0400

    Merge pull request #19101 from yrudman/make-chargeback-for-service-region-aware
    
    Fix: make chargeback generation for service to be region aware and do not generate for retired service
    (cherry picked from commit 3bcda0474cb200acb526c8ea3be5974700a021e2)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1736749

@yrudman yrudman deleted the make-chargeback-for-service-region-aware branch August 9, 2019 12:29
simaishi pushed a commit that referenced this pull request Oct 21, 2019
…gion-aware

Fix: make chargeback generation for service to be region aware and do not generate for retired service
(cherry picked from commit 3bcda04)

https://bugzilla.redhat.com/show_bug.cgi?id=1763862
@simaishi
Copy link
Contributor

Hammer backport details:

$ git log -1
commit c8b91c34881e53a735bca4cd112b91793a0208e1
Author: Brandon Dunne <bdunne@redhat.com>
Date:   Thu Aug 8 15:35:13 2019 -0400

    Merge pull request #19101 from yrudman/make-chargeback-for-service-region-aware
    
    Fix: make chargeback generation for service to be region aware and do not generate for retired service
    (cherry picked from commit 3bcda0474cb200acb526c8ea3be5974700a021e2)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1763862

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