ovs-vsctl: /var/run/ovs-vswitchd.*.ctl is a Unix domain socket
authorIan Campbell <Ian.Campbell@citrix.com>
Tue, 15 Sep 2009 16:24:46 +0000 (09:24 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 15 Sep 2009 16:27:08 +0000 (09:27 -0700)
Currently ov-vsctl tries to treat /var/run/ovs-vswitchd.*.ctl as a
file/pipe when it is actually a Unix domain socket:

        # ovs-vsctl add-br TEST
        Traceback (most recent call last):
          File "/usr/bin/ovs-vsctl", line 498, in ?
            main()
          File "/usr/bin/ovs-vsctl", line 493, in main
            function(*args)
          File "/usr/bin/ovs-vsctl", line 345, in cmd_add_br
            cfg_save(cfg, VSWITCHD_CONF)
          File "/usr/bin/ovs-vsctl", line 142, in cfg_save
            cfg_reload()
          File "/usr/bin/ovs-vsctl", line 126, in cfg_reload
            f = open(target, "r+")
        IOError: [Errno 6] No such device or address: ' '
        # ls -l /var/run/ovs-vswitchd.4173.ctl
        srw------- 1 root root 0 Sep 14 12:25 /var/run/ovs-vswitchd.4173.ctl

From strace:
        open("/var/run/ovs-vswitchd.4173.ctl", O_RDWR|O_LARGEFILE) = -1 ENXIO (No such device or address)

utilities/ovs-vsctl.in

index 438b978b89e9a6195be16145782ba84afa6e2f44..675f9dd4110a22624352c1ec71c21919073aac17 100755 (executable)
@@ -19,6 +19,7 @@ import fnmatch
 import getopt
 import os
 import re
+import socket
 import stat
 import sys
 
@@ -125,7 +126,9 @@ def cfg_reload():
         s = os.stat(target)
     if not stat.S_ISSOCK(s.st_mode):
         raise Error("%s is not a Unix domain socket, cannot reload" % target)
-    f = open(target, "r+")
+    skt = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)
+    skt.connect(target)
+    f = os.fdopen(skt.fileno(), "r+")
     f.write("vswitchd/reload\n")
     f.flush()
     f.readline()