Change in ovirt-engine[master]: restapi: Locate VM data using pool id, not name
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(a)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(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org