Maor Lipchuk has uploaded a new change for review.

Change subject: webadmin: Check attached DC only for data Storage.
......................................................................

webadmin: Check attached DC only for data Storage.

Filter only data storage domains when executing the validation for
attached Data Center when importing a file Storage Domain to prevent
checking it for ISO and Export Storage Domains.

Change-Id: I8fc9f48a3849f70306cc2748d1a716924a0a4f76
Bug-Url: https://bugzilla.redhat.com/1186371
Signed-off-by: Maor Lipchuk <mlipc...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/22/37322/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
index cf2de88..5ccd39d 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
@@ -2089,7 +2089,11 @@
                     boolean success = returnVal != null && 
returnVal.getSucceeded();
                     if (success) {
                         storageListModel.fileConnection.setid((String) 
returnVal.getActionReturnValue());
-                        checkFileDomainAttachedToDc("OnImportFile", 
storageListModel.fileConnection); //$NON-NLS-1$
+                        if (storageModel.getRole() == StorageDomainType.Data) {
+                            checkFileDomainAttachedToDc("OnImportFile", 
storageListModel.fileConnection); //$NON-NLS-1$
+                        } else {
+                            getExistingStorageDomainList();
+                        }
                     }
                     else
                     {


-- 
To view, visit http://gerrit.ovirt.org/37322
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8fc9f48a3849f70306cc2748d1a716924a0a4f76
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
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