Yair Zaslavsky has posted comments on this change.

Change subject: engine : Fail tasks that dont have Vdsm Id
......................................................................


Patch Set 2:

i want all patches of "missing vdsm task id" to move to the beginning of the 
patchset (correct me i'm wrong - we're talking about 3 patches?)
The minus is - your code will change - you will not be able to Rely on coco, 
and when you introduce coco - well, in all places async_tasks has taskId will 
have to replace with vdsmId.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I110c483842027d1596f64dd8c3f635d62924a93f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to