Bug/docker network dns (#368)

* Remove private network from network service containers * Remove private network creation and cleanup * Fix pylint --------- Co-authored-by: Jacob Boddey <[email protected]>
google · Apr 5, 2024 · 61e69f1 · 61e69f1
1 parent 83a678e
commit 61e69f1
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/cmd/prune b/cmd/prune
@@ -36,4 +36,6 @@ fi
 # Remove docker networks
 echo Removing docker networks
 sudo docker network rm endev0 > /dev/null
-sudo docker network rm tr-private-net > /dev/null
+# Private network not used, add cleanup
+# back in if/when implemented
+#sudo docker network rm tr-private-net > /dev/null
diff --git a/framework/python/src/net_orc/network_orchestrator.py b/framework/python/src/net_orc/network_orchestrator.py
@@ -357,7 +357,9 @@ def create_net(self):
     if 'CI' in os.environ:
       self._ci_post_network_create()
 
-    self._create_private_net()
+    # Private network not used, disable until
+    # a use case is determined
+    #self._create_private_net()
 
     # Listener may have already been created. Only create if not
     if self._listener is None:
@@ -479,6 +481,7 @@ def _start_network_service(self, net_module):
     network = 'host' if net_module.net_config.host else PRIVATE_DOCKER_NET
     LOGGER.debug(f"""Network: {network}, image name: {net_module.image_name},
                      container name: {net_module.container_name}""")
+
     try:
       client = docker.from_env()
       net_module.container = client.containers.run(
@@ -487,7 +490,10 @@ def _start_network_service(self, net_module):
         cap_add=['NET_ADMIN'],
         name=net_module.container_name,
         hostname=net_module.container_name,
-        network=PRIVATE_DOCKER_NET,
+        # Undetermined version of docker seems to have broken
+        # DNS configuration (/etc/resolv.conf)  Re-add when/if
+        # this network is utilized and DNS issue is resolved
+        #network=PRIVATE_DOCKER_NET,
         privileged=True,
         detach=True,
         mounts=net_module.mounts,