This is an automated email from the ASF dual-hosted git repository.
bmahler pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git
The following commit(s) were added to refs/heads/master by this push:
new e1752d198 Revert "CHANGE: add default network if no net options was
set."
e1752d198 is described below
commit e1752d19805ffd2229c786153a374ed89eb38853
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__
}