Bug 1605912

Summary: Change in chargeback report logging output
Product: Red Hat CloudForms Management Engine Reporter: Tasos Papaioannou <tpapaioa>
Component: ReportingAssignee: Libor Pichler <lpichler>
Status: CLOSED CURRENTRELEASE QA Contact: Tasos Papaioannou <tpapaioa>
Severity: medium Docs Contact:
Priority: high    
Version: 5.9.3CC: dmetzger, jrafanie, obarenbo, simaishi
Target Milestone: GAKeywords: Regression, TestOnly, ZStream
Target Release: 5.10.0   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 5.10.0.12 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1623582 (view as bug list) Environment:
Last Closed: 2019-02-11 14:18:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1623582    

Description Tasos Papaioannou 2018-07-20 17:35:08 UTC 
Description of problem:

After changes introduced in commit a518a17a0865554adfd42b3bf625194c6f15c461, chargeback reports log a different number of records found.

For example, a chargeback report for the past day logs 114 records found:

[----] I, [2018-07-20T10:58:27.130534 #12091:489108]  INFO -- : MIQ(Chargeback::ConsumptionHistory.for_report) Found 114 records for time range [Thu, 19 Jul 2018 00:00:00 UTC +00:00, Thu, 19 Jul 2018 23:59:59 UTC +00:00]

This gives the number of VM's with hourly rollups in that time range. Prior to this commit, the total number of rollups is logged instead:

[----] I, [2018-07-20T13:32:05.828170 #2317:cbd108]  INFO -- : MIQ(Chargeback::ConsumptionHistory.for_report) Found 1620 records for time range [Thu, 19 Jul 2018 00:00:00 UTC +00:00, Thu, 19 Jul 2018 23:59:59 UTC +00:00]


Version-Release number of selected component (if applicable):

5.9.3

How reproducible:

100%

Steps to Reproduce:
1.) Run chargeback for the past day.
2.) View evm.log.

Actual results:

Number of VM's is now logged as number of records found for time range.

Expected results:

Number of rollups is logged as number of records found for time range (or both # of rollups and # of VM's).

Additional info:
Comment 3 Joe Rafaniello 2018-08-07 21:53:52 UTC 
Libor, can you take a look when you get a chance?  Thanks!
Comment 6 Tasos Papaioannou 2018-09-18 14:19:58 UTC 
Verified on 5.10.0.15.