Maor Lipchuk has uploaded a new change for review.

Change subject: core: Reset OVF_STORE disks list
......................................................................

core: Reset OVF_STORE disks list

several Storage Domains can be attached to the Data Center once it is
being initilaized.
before each Storage Domain we should reset the OVF_STORE disk list.

Change-Id: I32cdc3e3a1dcf84a621ee305e2353d4be6074383
Bug-Url: https://bugzilla.redhat.com/1214408
Signed-off-by: Maor Lipchuk <mlipc...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStoragePoolWithStoragesCommand.java
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java
2 files changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/64/40464/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStoragePoolWithStoragesCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStoragePoolWithStoragesCommand.java
index d0c7bac..193b939 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStoragePoolWithStoragesCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStoragePoolWithStoragesCommand.java
@@ -264,6 +264,7 @@
 
     private void registerOvfStoreDisks() {
         for (final Guid storageDomainId : getParameters().getStorages()) {
+            resetOvfStoreDisks();
             final List<OvfEntityData> unregisteredEntitiesFromOvfDisk =
                     getEntitiesFromStorageOvfDisk(storageDomainId, 
getStoragePool().getId());
             TransactionSupport.executeInNewTransaction(new 
TransactionMethod<Void>() {
diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java
index 2284591..b3ed598 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java
@@ -668,4 +668,8 @@
         }
         return cinderBroker;
     }
+
+    protected void resetOvfStoreDisks() {
+        ovfDisks = null;
+    }
 }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I32cdc3e3a1dcf84a621ee305e2353d4be6074383
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to