vlog.py currently generates the same log messages, starts with the time stamp information, for console, syslog and file. All messages start with current time stamp information.
Syslogd, by default, prepends time stamp with each message already. Thus the time stamp generated by vlog.py is redundant. This patch removes time stamp from the message before vlog.py sends it to syslogd. Signed-off-by: Andy Zhou <az...@nicira.com> --- python/ovs/vlog.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/python/ovs/vlog.py b/python/ovs/vlog.py index f7ace66..24b234b 100644 --- a/python/ovs/vlog.py +++ b/python/ovs/vlog.py @@ -61,8 +61,8 @@ class Vlog: return now = datetime.datetime.utcnow().strftime("%Y-%m-%dT%H:%M:%SZ") - message = ("%s|%s|%s|%s|%s" - % (now, Vlog.__msg_num, self.name, level, message)) + syslog_message = ("%s|%s|%s|%s" + % (Vlog.__msg_num, self.name, level, message)) level = LEVELS.get(level.lower(), logging.DEBUG) Vlog.__msg_num += 1 @@ -70,6 +70,10 @@ class Vlog: for f, f_level in Vlog.__mfl[self.name].iteritems(): f_level = LEVELS.get(f_level, logging.CRITICAL) if level >= f_level: + if f == "syslog": + message = syslog_message; + else: + message = ("%s|%s" % (now, syslog_message)) logging.getLogger(f).log(level, message, **kwargs) def emer(self, message, **kwargs): -- 1.7.9.5 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev