Arik Hadas has posted comments on this change. Change subject: core: introduce convert vm command ......................................................................
Patch Set 83: "Arik uses the doPolling anyway.." - well.. in the first patches there was an alternative mechanism, more event-based, that I replaced just for the sake of using the command coordination framework so of course it is a bit awkward. I agree with Michal that in the long-term we should have something that is event-based (and maybe to use polling to get an initial state). Not to add a verb and call it in each cycle. -- To view, visit https://gerrit.ovirt.org/33055 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7bc67ff237d5c01fc5f3c9f21c822573e5db32a3 Gerrit-PatchSet: 83 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Shahar Havivi <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
