diff options
author | Ben Pfaff <blp@nicira.com> | 2011-09-21 14:56:55 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2011-09-22 10:55:33 -0700 |
commit | 77b8ebcfd4f645f6de6dc438cb1d9f1ee4e88e0b (patch) | |
tree | 54215e01b0acb8f7193cb46030623bfa47f7d6e9 /xenserver | |
parent | e8c02fd7e390d8ee43a6a810d8e16affc36057e0 (diff) | |
download | openvswitch-77b8ebcfd4f645f6de6dc438cb1d9f1ee4e88e0b.tar.gz |
ovs-xapi-sync: Make pychecker-able.
pychecker imports the code that it checks, which means that code at top
level runs, so "ovs-xapi-sync" failed to import unless the user had write
access to /var/log/openvswitch.
Diffstat (limited to 'xenserver')
-rwxr-xr-x | xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync b/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync index 02635568c..57dc2e302 100755 --- a/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync +++ b/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync @@ -38,14 +38,7 @@ import ovs.util import ovs.daemon import ovs.db.idl -s_log = logging.getLogger("ovs-xapi-sync") -l_handler = logging.handlers.RotatingFileHandler( - "/var/log/openvswitch/ovs-xapi-sync.log") -l_formatter = logging.Formatter('%(filename)s: %(levelname)s: %(message)s') -l_handler.setFormatter(l_formatter) -s_log.addHandler(l_handler) -s_log.setLevel(logging.INFO) - +s_log = logging.getLogger("ovs-xapi-sync") vsctl="/usr/bin/ovs-vsctl" session = None force_run = False @@ -263,6 +256,13 @@ def update_tap_from_vif(idl, tap_name, vif_name): def main(argv): global force_run + l_handler = logging.handlers.RotatingFileHandler( + "/var/log/openvswitch/ovs-xapi-sync.log") + l_formatter = logging.Formatter('%(filename)s: %(levelname)s: %(message)s') + l_handler.setFormatter(l_formatter) + s_log.addHandler(l_handler) + s_log.setLevel(logging.INFO) + try: options, args = getopt.gnu_getopt( argv[1:], 'h', ['help'] + ovs.daemon.LONG_OPTIONS) |