Bug 1459563

Summary: Incorrect storage used in Chargeback reports
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: ReportingAssignee: Šimon Lukašík <slukasik>
Status: CLOSED ERRATA QA Contact: Nandini Chandra <nachandr>
Severity: high Docs Contact:
Priority: high    
Version: 5.7.0CC: cpelland, dajohnso, jhardy, lpichler, nachandr, obarenbo, rspagnol, simaishi, slukasik
Target Milestone: GAKeywords: ZStream
Target Release: 5.7.3   
Hardware: All   
OS: All   
Whiteboard: storage:chargeback
Fixed In Version: 5.7.3.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1455690 Environment:
Last Closed: 2017-06-28 15:04:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On: 1455690    
Bug Blocks:    
Attachments:
Description Flags
screen shot of Chargeback report from 5_7_3_1 none

Comment 2 CFME Bot 2017-06-07 13:11:01 UTC 
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/f4fc22026014b8692513efba4f9892b6d1380505

commit f4fc22026014b8692513efba4f9892b6d1380505
Author:     Gregg Tanzillo <gtanzill>
AuthorDate: Mon Jun 5 23:24:25 2017 -0400
Commit:     Satoe Imaishi <simaishi>
CommitDate: Wed Jun 7 09:03:53 2017 -0400

    Merge pull request #15261 from isimluk/do-not-offer-useless-metrics
    
    Do not offer report columns that are useless
    (cherry picked from commit d07a5ef227d0d8b15ea343a94c914d6396485970)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1459563

 app/models/chargeback_vm.rb | 6 ------
 1 file changed, 6 deletions(-)
Comment 4 Nandini Chandra 2017-06-12 08:54:58 UTC 
Created attachment 1286983 [details]
screen shot of Chargeback report from 5_7_3_1
Comment 5 Šimon Lukašík 2017-06-12 09:17:04 UTC 
> The report has CPU Total Cost, Storage Total Cost.
> Here CPU Total Cost = CPU Used Cost + vCPUs Allocated Cost
>     Storage Total Cost = Storage Used Cost + Storage Allocated Cost

This is on purpose.

Summing the metrics together does not make sense, however summing the costs make sense.

In other words, when you sum allocated and consumed disk, the result is number of terabytes that do not rely to anything in the real world (not useful). However, when you sum cost of allocated disk and cost of consumed disk, you have dollar or Koruna amount that is attached to operating the storage on a given VM (possibly useful).

Thanks!
Comment 7 Nandini Chandra 2017-06-13 13:00:17 UTC 
Verified that the Total column for the following metrics is no longer available while creating a Chargeback report.

 - cpu is useless as it counts usagemhz and vm_numvcpus together
     - cpu_cores is useless as it only reports cpu_cores
     - memory is useless as counts used and available together
     - net_io is useless as it only reports net_ui_rate_avg
     - disk_io is useless as it only reports disk_usage_rate_avg
     - storage is useless as it counts used and allocated together
     - fixed is useless as it counts all the fixed metrics together

Verified in 5.7.3.1
Comment 9 errata-xmlrpc 2017-06-28 15:04:49 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2017:1601