You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2014/12/12 18:38:44 UTC

mesos git commit: Fixed the issue with echo command in port mapping isolator.

Repository: mesos
Updated Branches:
  refs/heads/master e2239a02c -> ce2aedf4a


Fixed the issue with echo command in port mapping isolator.

sh: echo: I/O error occurs, It is not possible to set the port range.
Also, echo in /bin/sh performs output a (horizontal) tab character
when encountered the escape sequence of \t.

Review: https://reviews.apache.org/r/28904


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/ce2aedf4
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/ce2aedf4
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/ce2aedf4

Branch: refs/heads/master
Commit: ce2aedf4ae6eeb9600840099d36b30e14cea38b4
Parents: e2239a0
Author: Kiyonari Harigae <la...@cloudysunny14.org>
Authored: Fri Dec 12 09:28:55 2014 -0800
Committer: Jie Yu <yu...@gmail.com>
Committed: Fri Dec 12 09:28:55 2014 -0800

----------------------------------------------------------------------
 src/slave/containerizer/isolators/network/port_mapping.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/ce2aedf4/src/slave/containerizer/isolators/network/port_mapping.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/isolators/network/port_mapping.cpp b/src/slave/containerizer/isolators/network/port_mapping.cpp
index c6fff21..2d81336 100644
--- a/src/slave/containerizer/isolators/network/port_mapping.cpp
+++ b/src/slave/containerizer/isolators/network/port_mapping.cpp
@@ -2646,7 +2646,7 @@ string PortMappingIsolatorProcess::scripts(Info* info)
          << net::IP(hostDefaultGateway.address()) << "\n";
 
   // Restrict the ephemeral ports that can be used by the container.
-  script << "echo -e " << info->ephemeralPorts.lower() << "\t"
+  script << "echo " << info->ephemeralPorts.lower() << " "
          << (info->ephemeralPorts.upper() - 1)
          << " > /proc/sys/net/ipv4/ip_local_port_range\n";