Change in ovirt-engine[ovirt-engine-3.6]: frontend: Allow ':' in VmInit network name
tnisan at redhat.com
tnisan at redhat.com
Wed Aug 19 11:56:43 UTC 2015
Tal Nisan has submitted this change and it was merged.
Change subject: frontend: Allow ':' in VmInit network name
......................................................................
frontend: Allow ':' in VmInit network name
Created new VmInitNetworkNameValidation that adds support for ':' character in
addition to those supported by AsciiNameValidation.
Also fixed issue regarding highlighting the first run tab on invalid data.
Problem was in usage of the old name for the tab (FIRST_RUN) and incorrectly using
the new one (INITIAL_RUN_TAB).
Change-Id: I8691dec2d9cc5ff645e2b18a73f0f1a8cd118bf3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1248975
Signed-off-by: Martin Betak <mbetak at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/TabName.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInitModel.java
A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/validation/VmInitNetworkNameValidation.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
5 files changed, 22 insertions(+), 9 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, approved
Jenkins CI: Passed CI tests
Martin Betak: Verified
--
To view, visit https://gerrit.ovirt.org/45073
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8691dec2d9cc5ff645e2b18a73f0f1a8cd118bf3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the Engine-commits
mailing list