Amit Aviram has posted comments on this change. Change subject: webadmin: Refactoring GlusterModel to inherit from PosixModel. ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/37893/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 535 Line 536 Line 537 Line 538 Line 539 Notice that using preparePosixStorageForEdit here with a GlusterStorageModel actually solves the bug produced by this line. This line lets the user always edit gluster's path which is currently blocked at backend, and never should be allowed when the domain is up. -- To view, visit http://gerrit.ovirt.org/37893 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7a355956f0f30a79f13d0bd72ec7a4d65cc29d1a 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: [email protected] Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
