Nir Soffer has posted comments on this change.
Change subject: debuging: Use __repr__ instead of __str__
......................................................................
Patch Set 1: Verified+1
Verfied using setup with host and one vm.
Most changes are related to the executor, here is some output using manhole:
>>> pprint.pprint(periodic._operations)
[<Operation action=<VmDispatcher operation=<class
'virt.periodic.UpdateVolumes'> at 0x2d40310> at 0x2d40450>,
<Operation action=<VmDispatcher operation=<class
'virt.periodic.NumaInfoMonitor'> at 0x2d40510> at 0x2d402d0>,
<Operation action=<VmDispatcher operation=<class
'virt.periodic.BlockjobMonitor'> at 0x2d40550> at 0x2d40590>,
<Operation action=<virt.sampling.VMBulkSampler object at 0x2d405d0> at
0x2d40750>,
<Operation action=<VmDispatcher operation=<class
'virt.periodic.DriveWatermarkMonitor'> at 0x2d40790> at 0x2d407d0>]
>>> pprint.pprint(periodic._executor._workers)
set([<Worker name=periodic/0 task=None at 0x2d38750>,
<Worker name=periodic/1 task=None at 0x2d38950>,
<Worker name=periodic/2 task=None at 0x2d38cd0>,
<Worker name=periodic/3 task=None at 0x2d38f90>])
--
To view, visit https://gerrit.ovirt.org/48865
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2fd724e32dfde1ca484343730f2e55c37a59fdf3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
