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

Reply via email to