Change in ovirt-engine[master]: webadmin: error fix in defaultRoute canEdit
Code Review
gerrit at ovirt.org
Thu Aug 3 09:35:20 UTC 2017
>From Tomas Jelinek <tjelinek at redhat.com>:
Tomas Jelinek has submitted this change and it was merged. ( https://gerrit.ovirt.org/80103 )
Change subject: webadmin: error fix in defaultRoute canEdit
......................................................................
webadmin: error fix in defaultRoute canEdit
Before test in canEdit was done only comparing management network.
That blocked mgmt network from losing default route role, but it
also prevented it from acquiring it, which is not ok.
Change-Id: I95ae7fa9131c47f3e4f60786d97369d82205cc13
Bug-Url: https://bugzilla.redhat.com/1443412
Signed-off-by: Martin Mucha <mmucha at redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterManageNetworkPopupView.java
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Alexander Wels: Looks good to me, approved
Martin Mucha: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/80103
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I95ae7fa9131c47f3e4f60786d97369d82205cc13
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list