Tal Nisan has uploaded a new change for review.

Change subject: webadmin: When adding new domain disable V1 format in Posix & 
Gluster
......................................................................

webadmin: When adding new domain disable V1 format in Posix & Gluster

When adding a new domain without a storage pool, there was an option to add
a Posix & Gluster domain in format V1, this makes no sense since Posix and
Gluster both are not supported on V3.0 which is the only version that
supports V1 format for data domains

https://bugzilla.redhat.com/955641
Signed-off-by: Tal Nisan <[email protected]>
Change-Id: I4c53773f9d034bcee71e033df56adafbb516e474
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/30/28130/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java
index 72ed8ce..21e7206 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java
@@ -652,7 +652,9 @@
 
                 getFormat().setIsChangable(true);
 
-                formats.add(StorageFormatType.V1);
+                if (getSelectedItem().getType() != StorageType.POSIXFS && 
getSelectedItem().getType() != StorageType.GLUSTERFS) {
+                    formats.add(StorageFormatType.V1);
+                }
 
                 if ((getSelectedItem().getType() == StorageType.FCP || 
getSelectedItem().getType() == StorageType.ISCSI)
                         && getSelectedItem().getRole() == 
StorageDomainType.Data)


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

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

Reply via email to