Change in ovirt-engine[master]: webadmin: support for username+domain on import vm

ahadas at redhat.com ahadas at redhat.com
Thu Jul 9 13:29:28 UTC 2015


Arik Hadas has submitted this change and it was merged.

Change subject: webadmin: support for username+domain on import vm
......................................................................


webadmin: support for username+domain on import vm

When importing VM from vCenter 6.0, one needs to provide the
domain along with the user name (e.g username at domain).

Thus, the validation of ASCII characters in the username is removed
and the '@' in the username is replaced with '%40' so the URI will
be valid.

Change-Id: Icabc34f40195d071c3389cee1bba57073672c93e
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmsModel.java
A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/validation/NameAndOptionalDomainValidation.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
3 files changed, 27 insertions(+), 3 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Martin Betak: Looks good to me, but someone else must approve
  Arik Hadas: Verified; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icabc34f40195d071c3389cee1bba57073672c93e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas 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