Change in ovirt-engine[ovirt-engine-3.6]: restapi: Locate VM data using pool id, not name

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: restapi: Locate VM data using pool id, not name
......................................................................


restapi: Locate VM data using pool id, not name

Currently when a VM pool is updated the VM data is located using the
name of the pool. But the name of the pool may be also updated during
the same operation, which may result in not finding the VM data because
there is no such pool yet. To avoid this potential (and real) conflict
this patch changes the RESTAPI so that it uses the identifier of the
pool instead of the name, as the identifier will never change.

Change-Id: I387d122d1019f25f026806ed0042aa32b5db8698
Bug-Url: https://bugzilla.redhat.com/1256411
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
(cherry picked from commit 955c08a8ef661305da86a658d3597f7e2940b599)
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolResource.java
1 file changed, 3 insertions(+), 4 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I387d122d1019f25f026806ed0042aa32b5db8698
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org