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@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@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Ori Liel <oliel@redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: automation@ovirt.org