Change in ovirt-engine[master]: webadmin: represent occupied name on import differently

ahadas at redhat.com ahadas at redhat.com
Wed May 6 07:50:54 UTC 2015


Arik Hadas has submitted this change and it was merged.

Change subject: webadmin: represent occupied name on import differently
......................................................................


webadmin: represent occupied name on import differently

I tried to add a generic field for errors on ImportVmData that will also
indicate if the name of the VM we're about to import already exists in
the system. However, it is problematic so it is better to add another
field for it.

Change-Id: I4411e79f7018d5b2c1a8d8ac9d3b0b79de26a4e3
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmData.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmFromExportDomainModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportVmFromExportDomainPopupView.java
3 files changed, 24 insertions(+), 6 deletions(-)

Approvals:
  Jenkins CI: Looks good to me, but someone else must approve
  Arik Hadas: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4411e79f7018d5b2c1a8d8ac9d3b0b79de26a4e3
Gerrit-PatchSet: 5
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