Default host binding is 0.0.0.0.

Move the file uploader template.
UDP port allocation (removed the host).
Minor changes with iouyap lookup and remote server project directory.

gns3server/modules/dynamips/__init__.py

@@ -110,10 +110,11 @@ class Dynamips(IModule):
         self._frame_relay_switches = {}
         self._atm_switches = {}
         self._ethernet_hubs = {}
-        self._projects_dir = kwargs["projects_dir"]
+        self._projects_dir = os.path.join(kwargs["projects_dir"], "dynamips")
         self._tempdir = kwargs["temp_dir"]
         self._working_dir = self._projects_dir
         self._dynamips = ""
+        self._default_host = "0.0.0.0"
 
         self._callback = self.add_periodic_callback(self._check_hypervisors, 5000)
         self._callback.start()
@@ -199,16 +200,18 @@ class Dynamips(IModule):
         if not os.access(self._dynamips, os.X_OK):
             raise DynamipsError("Dynamips {} is not executable".format(self._dynamips))
 
-        # check if the working directory exists
         if not os.path.exists(self._working_dir):
-            raise DynamipsError("Working directory {} doesn't exist".format(self._working_dir))
+            try:
+                os.makedirs(self._working_dir)
+            except EnvironmentError as e:
+                raise DynamipsError("Could not create working directory {}".format(e))
 
         # check if the working directory is writable
         if not os.access(self._working_dir, os.W_OK):
             raise DynamipsError("Cannot write to working directory {}".format(self._working_dir))
 
         log.info("starting the hypervisor manager with Dynamips working directory set to '{}'".format(self._working_dir))
-        self._hypervisor_manager = HypervisorManager(self._dynamips, self._working_dir)
+        self._hypervisor_manager = HypervisorManager(self._dynamips, self._working_dir, self._default_host)
 
         for name, value in self._hypervisor_manager_settings.items():
             if hasattr(self._hypervisor_manager, name) and getattr(self._hypervisor_manager, name) != value:
@@ -245,7 +248,8 @@ class Dynamips(IModule):
             else:
                 self._remote_server = True
                 log.info("this server is remote")
-                self._working_dir = self._projects_dir
+                self._working_dir = os.path.join(self._projects_dir, request["project_name"])
+                log.info("this server is remote with working directory path to {}".format(self._working_dir))
 
             self._hypervisor_manager_settings = request
 
@@ -329,8 +333,7 @@ class Dynamips(IModule):
                                                                             node.id,
                                                                             port,
                                                                             host))
-        response = {"lport": port,
-                    "lhost": host}
+        response = {"lport": port}
 
         return response
 
@@ -421,5 +424,8 @@ class Dynamips(IModule):
         :param request: JSON request
         """
 
-        import netifaces
+        try:
+            import netifaces
+        except ImportError:
+            raise DynamipsError("The netifaces module is not installed")
         self.send_response(netifaces.interfaces())

gns3server/modules/dynamips/backends/atmsw.py

@@ -141,7 +141,6 @@ class ATMSW(object):
 
         Response parameters:
         - port_id (port identifier)
-        - lhost (local host address)
         - lport (allocated local port)
 
         :param request: JSON request

gns3server/modules/dynamips/backends/ethhub.py

@@ -140,7 +140,6 @@ class ETHHUB(object):
 
         Response parameters:
         - port_id (port identifier)
-        - lhost (local host address)
         - lport (allocated local port)
 
         :param request: JSON request

gns3server/modules/dynamips/backends/ethsw.py

@@ -159,7 +159,6 @@ class ETHSW(object):
 
         Response parameters:
         - port_id (port identifier)
-        - lhost (local host address)
         - lport (allocated local port)
 
         :param request: JSON request

gns3server/modules/dynamips/backends/frsw.py

@@ -139,7 +139,6 @@ class FRSW(object):
 
         Response parameters:
         - port_id (port identifier)
-        - lhost (local host address)
         - lport (allocated local port)
 
         :param request: JSON request

gns3server/modules/dynamips/backends/vm.py

@@ -537,7 +537,6 @@ class VM(object):
 
         Response parameters:
         - port_id (unique port identifier)
-        - lhost (local host address)
         - lport (allocated local port)
 
         :param request: JSON request

gns3server/modules/dynamips/hypervisor.py

@@ -273,5 +273,5 @@ class Hypervisor(DynamipsHypervisor):
         if self._host != '0.0.0.0':
             command.extend(['-H', self._host + ':' + str(self._port)])
         else:
-            command.extend(['-H', self._port])
+            command.extend(['-H', str(self._port)])
         return command

gns3server/modules/iou/__init__.py

@@ -54,13 +54,18 @@ class IOU(IModule):
         config = Config.instance()
         iou_config = config.get_section_config(name.upper())
         self._iouyap = iou_config.get("iouyap")
-        if not self._iouyap:
-            for path in os.environ["PATH"].split(":"):
-                if "iouyap" in os.listdir(path) and os.access("iouyap", os.X_OK):
-                    self._iouyap = os.path.join(path, "iouyap")
-                    break
-
         if not self._iouyap or not os.path.exists(self._iouyap):
+            iouyap_in_cwd = os.path.join(os.getcwd(), "iouyap")
+            if os.path.exists(iouyap_in_cwd):
+                self._iouyap = iouyap_in_cwd
+            else:
+                # look for iouyap if none is defined or accessible
+                for path in os.environ["PATH"].split(":"):
+                    if "iouyap" in os.listdir(path) and os.access(os.path.join(path, "iouyap"), os.X_OK):
+                        self._iouyap = os.path.join(path, "iouyap")
+                        break
+
+        if not self._iouyap:
             log.warning("iouyap binary couldn't be found!")
         elif not os.access(self._iouyap, os.X_OK):
             log.warning("iouyap is not executable")
@@ -75,8 +80,8 @@ class IOU(IModule):
         self._udp_start_port_range = 30001
         self._udp_end_port_range = 40001
         self._current_udp_port = self._udp_start_port_range
-        self._host = "127.0.0.1"  # FIXME: used by ZeroMQ...
-        self._projects_dir = kwargs["projects_dir"]
+        self._default_host = "0.0.0.0"
+        self._projects_dir = os.path.join(kwargs["projects_dir"], "iou")
         self._tempdir = kwargs["temp_dir"]
         self._working_dir = self._projects_dir
         self._iourc = ""
@@ -146,6 +151,7 @@ class IOU(IModule):
 
         if self._iourc and os.path.exists(self._iourc):
             try:
+                log.info("deleting iourc file {}".format(self._iourc))
                 os.remove(self._iourc)
             except EnvironmentError as e:
                 log.warn("could not delete iourc file {}: {}".format(self._iourc, e))
@@ -162,6 +168,7 @@ class IOU(IModule):
 
         Optional request parameters:
         - working_dir (path to a working directory)
+        - project_name
         - console_start_port_range
         - console_end_port_range
         - udp_start_port_range
@@ -196,8 +203,8 @@ class IOU(IModule):
                 iou_instance.working_dir = self._working_dir
         else:
             self._remote_server = True
-            log.info("this server is remote")
-            self._working_dir = self._projects_dir
+            self._working_dir = os.path.join(self._projects_dir, request["project_name"])
+            log.info("this server is remote with working directory path to {}".format(self._working_dir))
 
         if "console_start_port_range" in request and "console_end_port_range" in request:
             self._console_start_port_range = request["console_start_port_range"]
@@ -233,11 +240,17 @@ class IOU(IModule):
         iou_path = request["path"]
 
         try:
-            iou_instance = IOUDevice(iou_path, self._working_dir, name=name)
+            if not os.path.exists(self._working_dir):
+                try:
+                    os.makedirs(self._working_dir)
+                except EnvironmentError as e:
+                    raise IOUError("Could not create working directory {}".format(e))
+
+            iou_instance = IOUDevice(iou_path, self._working_dir, host=self._default_host, name=name)
             # find a console port
             if self._current_console_port >= self._console_end_port_range:
                 self._current_console_port = self._console_start_port_range
-            iou_instance.console = IOUDevice.find_unused_port(self._current_console_port, self._console_end_port_range, self._host)
+            iou_instance.console = IOUDevice.find_unused_port(self._current_console_port, self._console_end_port_range, self._default_host)
             self._current_console_port += 1
         except IOUError as e:
             self.send_custom_error(str(e))
@@ -441,7 +454,6 @@ class IOU(IModule):
 
         Response parameters:
         - port_id (unique port identifier)
-        - lhost (local host address)
         - lport (allocated local port)
 
         :param request: JSON request
@@ -461,15 +473,14 @@ class IOU(IModule):
             # find a UDP port
             if self._current_udp_port >= self._udp_end_port_range:
                 self._current_udp_port = self._udp_start_port_range
-            port = IOUDevice.find_unused_port(self._current_udp_port, self._udp_end_port_range, host=self._host, socket_type="UDP")
+            port = IOUDevice.find_unused_port(self._current_udp_port, self._udp_end_port_range, host=self._default_host, socket_type="UDP")
             self._current_udp_port += 1
 
             log.info("{} [id={}] has allocated UDP port {} with host {}".format(iou_instance .name,
                                                                                 iou_instance .id,
                                                                                 port,
-                                                                                self._host))
-            response = {"lport": port,
-                        "lhost": self._host}
+                                                                                self._default_host))
+            response = {"lport": port}
 
         except IOUError as e:
             self.send_custom_error(str(e))