Change in vdsm[master]: vm: debug log for getAllVmStats() and getVMList()

michal.skrivanek at redhat.com michal.skrivanek at redhat.com
Mon Mar 9 10:28:59 UTC 2015


Michal Skrivanek has posted comments on this change.

Change subject: vm: debug log for getAllVmStats() and getVMList()
......................................................................


Patch Set 3:

so don't we want to remove the whole block? Looing at the remaining verbs they are even less frequent than getVMList...so I'm all for dropping all exceptions

-- 
To view, visit https://gerrit.ovirt.org/36732
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I23d9361568f51be9b70dab22d3a388f692011402
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.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