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

[juan.hernandez at redhat.com]

Juan Hernandez 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>
---
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
  Omer Frenkel: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I387d122d1019f25f026806ed0042aa32b5db8698
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: automation at ovirt.org