Ravi Nori has posted comments on this change. Change subject: engine, webadmin: Re-work engine ovirt-node host-deploy sequence ......................................................................
Patch Set 2: @Alon After looking at the code, there is an extra level of indirection in the backend so the command can be executed asynchronously wit out having to wait for the result. So I would leave that in place. Renaming InstallVdsCommandBase and UpdateVdsCommandBase to more appropriate names to reflect that they are used by both Vds and oVirt Node might be the right approach. Please let me know what you think. -- To view, visit http://gerrit.ovirt.org/25685 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7a897581e2b75f7cdc57db40418504a79b9d2787 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches