Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Configure local storage does not work

tnisan at redhat.com tnisan at redhat.com
Mon Oct 19 11:56:52 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: Configure local storage does not work
......................................................................


webadmin: Configure local storage does not work

Fixed the cluster and data center validations to validate only the stuff
needed for configure local storage, that way we also avoid the exceptions
that are breaking the dialog functionality

Bug-Url: https://bugzilla.redhat.com/1270826
Change-Id: I424a6f618c3c3968b169fc49e1140cddf3c57bb6
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/ConfigureLocalStorageModel.java
3 files changed, 30 insertions(+), 15 deletions(-)

Approvals:
  Tal Nisan: Verified
  Daniel Erez: Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I424a6f618c3c3968b169fc49e1140cddf3c57bb6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez 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