[ovs-dev] [PATCH] stp: Log topology change events.

Ethan Jackson ethan at nicira.com
Sat Jan 28 05:46:56 UTC 2012


> I think it would be more better for test-stp.c to just call
> vlog_set_pattern() itself, rather than each use of it including it on
> the command line.  You could also have it disable logging to syslog
> with vlog_set_levels(), since that just makes the unit tests spam
> syslog.  If you make those changes, then test-stp.c wouldn't need the
> more sophisticated command-line parsing that you added; I am
> indifferent whether you keep it or remove it.

I'm not sure I agree.  It seems better to me to implement the full
fledged solution giving the automake tests full control how the
logging should look.  It seems less adhoc and more future proof to me.
 That said, it's testing code and doesn't really matter.  I'll go
ahead and change it.

Ethan



More information about the dev mailing list