Amit Aviram has posted comments on this change. Change subject: webadmin, engine: Gluster domain edit enhancement. ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/37894/1/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java File frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java: Line 1392: Line 1393: if (storageModel.getType().equals(StorageType.NFS)) { Line 1394: updateNFSProperties(storageModel); Line 1395: } Line 1396: else if (storageModel instanceof PosixStorageModel) { > Why do you need this change? because if the storage model is any kind of PosixStorageModel (this includes all the derived classes as well, among them is gluster model) it should enter to updatePosixProperties which sets the vfs type etc... Line 1397: updatePosixProperties(storageModel); Line 1398: } Line 1399: Line 1400: StorageServerConnectionParametersBase parameters = -- To view, visit http://gerrit.ovirt.org/37894 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7320520a15fe804a3a560bf1b8223bfee5b1b46a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <[email protected]> Gerrit-Reviewer: Ala Hino <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Amit Aviram <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
