Dan Kenigsberg has posted comments on this change.

Change subject: Remove mac address from host interface statistics
......................................................................


Patch Set 3: Looks good to me, but someone else must approve

(1 inline comment)

....................................................
File vdsm/sampling.py
Line 464
Line 465
Line 466
Line 467
Line 468
Thank you guys for correcting me. Indeed, it has been quite some time since 
Federico moved getAllVmStats to use sampling.AdvancedStatsThread, and not the 
StatsThread.

The StatsThread is used by nothing but HostStatsThread, and mainly serves as a 
distraction.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If5c2b455387f30bfb386662fa040a82b051f153d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Petr Ĺ ebek <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to