I'm not a windows expert, but this also solves a problem I was having on a
linux system chroot where /dev/log is absent.

I pushed this patch to master, thanks,

Daniele

2016-07-06 5:38 GMT-07:00 Paul Boca <pb...@cloudbasesolutions.com>:

> Disable the syslog logger in case on Windows, '/dev/log' doesn't exist.
> Seems like on Python34 a default handler is added to the logger and it
> prints
> even if no handler is set by us.
>
> Signed-off-by: Paul-Daniel Boca <pb...@cloudbasesolutions.com>
> ---
> V3: Initial commit
> V4: No changes
> V5: No changes
> V6: No changes
> ---
>  python/ovs/vlog.py | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/python/ovs/vlog.py b/python/ovs/vlog.py
> index 4996387..49210b8 100644
> --- a/python/ovs/vlog.py
> +++ b/python/ovs/vlog.py
> @@ -301,6 +301,7 @@ class Vlog(object):
>          # If there is no infrastructure to support python syslog, increase
>          # the logging severity level to avoid repeated errors.
>          if not os.path.exists("/dev/log"):
> +            logger.disabled = True
>              logger.setLevel(logging.CRITICAL)
>              return
>
> --
> 2.7.2.windows.1
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to