apache/cloudstack
1
0
Fork 0
mirror of https://github.com/apache/cloudstack.git synced 2025-10-26 08:42:29 +01:00
Code Issues Packages Projects Releases Wiki Activity

CLOUDSTACK-10060: ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot) (#2257)

Browse Source 
Bug Description:
In the listUsage API, the Virtual Size of Snapshot is always displayed as '0'.

Root Cause:
In case of snapshots, the usage is accounted for depending on the value of the global parameter, 'usage.snapshot.virtualsize.select'. If set to 'true', the usage calculation is done based on the Virtual Size, and if set to false, it is done based on the Physical size. In the Usage API, this value, (i.e. virtual or physical) is displayed as 'Size' field correctly, but the field 'VirtualSize' is always displayed as 0. This is misleading.

Expected Output:
Since this is a Usage API, ideally only that size should be displayed which is used for billing, depending on the value of 'usage.snapshot.virtualsize.select'.
There is another API - the ListSnapshot API that displays both, the physical as well as Virtual size and can be used to know both the sizes.

Fix Implemented:
Skipped showing the 'VirtualSize' Field for type=snapshot
This commit is contained in:
PranaliM 2017-10-12 18:17:18 +05:30 committed by Rohit Yadav
parent c3ed1b38e5
commit 189b0e4487
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
usage/src/com/cloud/usage/parser/StorageUsageParser.java
View File

@ -181,8 +181,11 @@ public class StorageUsageParser {
usageDesc += "Snapshot ";
break;
}
// Create the usage record
usageDesc += "Id:" + storageId + " Size:" + size + "VirtualSize:" + virtualSize;
//Create the usage record
usageDesc += "Id:" + storageId + " Size:" + size;
if (type != StorageTypes.SNAPSHOT) {
usageDesc += " VirtualSize:" + virtualSize;
}
//ToDo: get zone id
UsageVO usageRecord =