Change in ovirt-engine[ovirt-engine-4.1]: core: fix NumberFormatException when importing a removed VM

[Code Review]

>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: core: fix NumberFormatException when importing a removed VM
......................................................................


core: fix NumberFormatException when importing a removed VM

When importing a VM while another is in the process of being
removed a NumberFormatException will be thrown and the import
will fail.

This is caused by the removed VMs getting a "_remove_me_"
prefix to its GUID when removed. Because of the prefix the
method GetImagesListVDSCommand#executeIrsBrokerCommand fails
when attempting to create an instance of the Guid object
using the prefixed GUID of the removed VM.

Fixed by filtering out the prefixed image GUIDs in
ImagesListReturn constructor.

Change-Id: I564fb9d0c08e6fabbf8ffd05cfdf93d5435a0e0b
Bug-Url: https://bugzilla.redhat.com/1331335
Signed-off-by: Benny Zlotnik <bzlotnik at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/ImagesListReturn.java
A backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/irsbroker/ImagesListReturnTest.java
2 files changed, 51 insertions(+), 4 deletions(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I564fb9d0c08e6fabbf8ffd05cfdf93d5435a0e0b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>