Vered Volansky has posted comments on this change. Change subject: webadmin: Display correct error when adding new storage ......................................................................
Patch Set 2: Looks good to me, but someone else must approve (1 inline comment) .................................................... Commit Message Line 8: Line 9: When adding a new Storage Domain (of NFS/Local type) the error displayed Line 10: when the connection was failed was always "Error creating a storage domain" Line 11: instead of the actual error from VDSM (no permission error etc.), the connect Line 12: command was changed to propegated the VDSM error to the client there it will /s/propegate/propagate /s/there/, where Line 13: be tranlsated and be displayed as the translated user friendly value Line 14: Line 15: Change-Id: Ib8f721d3f10596253ca194a36139ba9ebcdff246 Line 16: Bug-Url: https://bugzilla.redhat.com/843440 -- To view, visit http://gerrit.ovirt.org/13681 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib8f721d3f10596253ca194a36139ba9ebcdff246 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
