Change in ovirt-engine[master]: restapi: trace export template command by job status

mpastern at redhat.com mpastern at redhat.com
Wed Feb 27 12:20:12 UTC 2013


Michael Pasternak has submitted this change and it was merged.

Change subject: restapi: trace export template command by job status
......................................................................


restapi: trace export template command by job status

This patch prevents a situation in which the REST reports that export
template command was finished before the end-action event is added.

This situation occured because the REST part monitored the command
according to the statuses of its VDSM tasks - when they were done,
it reports back to the user that the command has finished and gave
him back the control (the command was invoked synchronously),
sometimes before the engine managed to add the end-action event.

This patch change the way the export template command is monitored by
the REST to be according to the command's job status, that way we ensure
that the end-action event is added before REST is reporting back to the
user that the command is finished.

Change-Id: I4442b337e0242aa756825243dbcf6db1a2b0559f
Bug-Url: https://bugzilla.redhat.com/860194
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendTemplateResource.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Michael Pasternak: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/12432
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4442b337e0242aa756825243dbcf6db1a2b0559f
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>



More information about the Engine-commits mailing list