This is an automated email from the ASF dual-hosted git repository.

bmahler pushed a commit to branch revert-583-add-docker-net-arg
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit f7e74ddae3a1187d75aae150ef980e6169a1e3a8
Author: bmahler <[email protected]>
AuthorDate: Wed May 29 20:01:25 2024 -0400

    Revert "CHANGE: add default network if no net options was set."
    
    This reverts commit 5c3b039db544e937617cd63810185cc95fc2b34b.
---
 src/docker/docker.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/docker/docker.cpp b/src/docker/docker.cpp
index 50239bf32..8ad58339b 100644
--- a/src/docker/docker.cpp
+++ b/src/docker/docker.cpp
@@ -833,12 +833,12 @@ Try<Docker::RunOptions> Docker::RunOptions::create(
   ContainerInfo::DockerInfo::Network network;
   if (dockerInfo.has_network()) {
     network = dockerInfo.network();
-  } else if (!options.network.isSome()) {
-    // If no network and no docker network options was given, then use the OS 
specific default.    
+  } else {
+    // If no network was given, then use the OS specific default.
 #ifdef __WINDOWS__
-      network = ContainerInfo::DockerInfo::BRIDGE;
+    network = ContainerInfo::DockerInfo::BRIDGE;
 #else
-      network = ContainerInfo::DockerInfo::HOST;
+    network = ContainerInfo::DockerInfo::HOST;
 #endif // __WINDOWS__
   }
 

Reply via email to