Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: LOCALFS storage path regex validation. ......................................................................
webadmin: LOCALFS storage path regex validation. LOCALFS storage path is not validated with regex in frontend, being sent to the backend/vdsm as is, even invalid. On the other hand, adding a NFS storage, the path is validated in frontend, avoiding extra processing and quickly/clearly informing user about what's wrong. This patch adds the LOCALFS path validation in frontend, in order to standardize according with the NFS regex path validation. Change-Id: I89a768957974b67754f5a3b78ec8f773e25adc7b Signed-off-by: Amador Pahim <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/LocalStorageModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/validation/LocalfsLinuxMountPointValidation.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Constants.java 3 files changed, 39 insertions(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4857 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89a768957974b67754f5a3b78ec8f773e25adc7b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amador Pahim <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Amador Pahim <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
