Change in ovirt-engine[master]: frontend: Allow ':' in VmInit network name

tjelinek at redhat.com tjelinek at redhat.com
Wed Aug 19 10:29:36 UTC 2015


Tomas Jelinek 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/45037
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8691dec2d9cc5ff645e2b18a73f0f1a8cd118bf3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak 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