Change in vdsm[master]: hooks: ovs: don't log every stats change

phoracek at redhat.com phoracek at redhat.com
Mon Feb 1 11:47:58 UTC 2016


Petr Horáček has uploaded a new change for review.

Change subject: hooks: ovs: don't log every stats change
......................................................................

hooks: ovs: don't log every stats change

VDSM log is spammed with 'ovs_after_get_stats' and every hour the VDSM
log is rotating.

This patch is ovirt-3.6-only because of native implementation will
be released soon on master.

Change-Id: I3b3386118d0d43bec2e6eaa1799eb3c93a2e4c96
Bug-Url: https://bugzilla.redhat.com/1234867
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Label: ovirt-3.6-only
---
M vdsm_hooks/ovs/ovs_after_get_stats.py
1 file changed, 0 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/20/52920/1

diff --git a/vdsm_hooks/ovs/ovs_after_get_stats.py b/vdsm_hooks/ovs/ovs_after_get_stats.py
index 9e8e1bf..fc27b5f 100755
--- a/vdsm_hooks/ovs/ovs_after_get_stats.py
+++ b/vdsm_hooks/ovs/ovs_after_get_stats.py
@@ -17,7 +17,6 @@
 #
 # Refer to the README and COPYING files for full details of the license
 #
-from functools import partial
 import traceback
 
 import six
@@ -27,9 +26,6 @@
 import hooking
 
 from ovs_utils import is_ovs_network
-import ovs_utils
-
-log = partial(ovs_utils.log, tag='ovs_after_get_stats: ')
 
 
 def ovs_networks_stats(stats):
@@ -60,7 +56,6 @@
                 ovs_networks_stats[vlan_name] = stats[network]
                 ovs_networks_stats[vlan_name]['name'] = vlan_name
 
-    log('Updating network stats with OVS networks: %s' % ovs_networks_stats)
     return ovs_networks_stats
 
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3b3386118d0d43bec2e6eaa1799eb3c93a2e4c96
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>


More information about the vdsm-patches mailing list