Change in vdsm[master]: stats: avoid log if cpu stats are known missing

asegurap at redhat.com asegurap at redhat.com
Wed May 21 10:49:59 UTC 2014


Antoni Segura Puimedon has posted comments on this change.

Change subject: stats: avoid log if cpu stats are known missing
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.ovirt.org/#/c/27416/7/vdsm/virt/vm.py
File vdsm/virt/vm.py:

Line 284:                 - self._diff(eInfo, sInfo, 'system_time'),
Line 285:                 sampleInterval)
Line 286: 
Line 287:         except (TypeError, ZeroDivisionError) as e:
Line 288:             self._log.debug("CPU stats not available: %s", e)
What about this patch just be a modification above:
    if sInfo is None:
        self._log.debug('CPU stats not available: %s', e)
Line 289: 
Line 290:     @classmethod
Line 291:     def _getNicStats(cls, name, model, mac,
Line 292:                      start_sample, end_sample, interval):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I62eff2e04a2922fb683d3c1d78605990399c7b76
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek 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: Yes


More information about the vdsm-patches mailing list