Vered Volansky has uploaded a new change for review.

Change subject: core: initializing domain sizes with null
......................................................................

core: initializing domain sizes with null

Up till now, available size and used size properties in
StorageDomainDynamic were set to 0 instead of null on StorageDomain creation.
Total size, based on these values, is sometimes set to 0 when either one is
null. This is also not the desired behaviour. Both this issues are
strongly coupled and dealt with in this patch.

Change-Id: I182a4c25ed0bf896adaadb86e8b36cfaf17d93e6
Bug-Url: https://bugzilla.redhat.com/1202363
Signed-off-by: Vered Volansky <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainDynamic.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/28/39028/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainDynamic.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainDynamic.java
index 2fc71d8..c03fa44 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainDynamic.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainDynamic.java
@@ -7,7 +7,7 @@
     private static final long serialVersionUID = -5305319985243261293L;
 
     public StorageDomainDynamic() {
-        this (0, Guid.Empty, 0);
+        this (null, Guid.Empty, null);
     }
 
     public StorageDomainDynamic(Integer availableDiskSize, Guid id, Integer 
usedDiskSize) {


-- 
To view, visit https://gerrit.ovirt.org/39028
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I182a4c25ed0bf896adaadb86e8b36cfaf17d93e6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to