Change in vdsm[master]: virt: Add a function to execute a qemu monitor command

fromani at redhat.com fromani at redhat.com
Fri May 23 11:48:39 UTC 2014


Francesco Romani has posted comments on this change.

Change subject: virt: Add a function to execute a qemu monitor command
......................................................................


Patch Set 1:

I share Federico's doubt.

I will not object to add this if it is needed by live merge.

However, even though what Adam is saying is 100% correct, and even though yes, sometimes somehow libvirt gets in the way,
I don't like to circumvent it. As general approach, we should not bypass layers.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3e9e07ba0c236c0938b129ae90af825f18f0e644
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list