Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Add missing warning message in storage domain popup

tnisan at redhat.com tnisan at redhat.com
Tue Nov 10 15:29:17 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: Add missing warning message in storage domain popup
......................................................................


webadmin: Add missing warning message in storage domain popup

Patches c273cd696323bab805bc3396b33b8953aa18f518 and
9dd1711da87bbc7dca9b6e7d6c84e9ac06396af8 introduced changes to the NFS and
Posix storage view but by mistake removed also the error message in those
views causing the storage popup not to show errors in storage domain
creation/edit among those errors the error of a path that already exists in
the system, this patch restores the error label so the error will appear

Change-Id: I74274212099014a5eb5f081629e39a705bb4b6fe
Signed-off-by: Tal Nisan <tnisan at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1279192
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.ui.xml
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/PosixStorageView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/PosixStorageView.ui.xml
4 files changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Tal Nisan: Verified
  Daniel Erez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Amit Aviram: Looks good to me, but someone else must approve
  Freddy Rolland: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/48351
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I74274212099014a5eb5f081629e39a705bb4b6fe
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list