You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by gr...@apache.org on 2018/08/03 23:07:03 UTC

[mesos] branch 1.6.x updated (2bc0086 -> 6c362e9)

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

grag pushed a change to branch 1.6.x
in repository https://gitbox.apache.org/repos/asf/mesos.git.


    from 2bc0086  Bumped the Mesos version to 1.6.2.
     new 2119fd1  Added missing iptables argument in CNI port mapper.
     new 6c362e9  Added MESOS-9125 to the 1.6.2 CHANGELOG.

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CHANGELOG                                                               | 1 +
 .../mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp     | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)


[mesos] 01/02: Added missing iptables argument in CNI port mapper.

Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

grag pushed a commit to branch 1.6.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 2119fd15eda6d35bc2c50410771072156bf92b76
Author: Greg Mann <gr...@mesosphere.io>
AuthorDate: Fri Aug 3 13:41:33 2018 -0700

    Added missing iptables argument in CNI port mapper.
    
    Previously, the CNI port mapper was using the `-w` option in all
    invocations of `iptables` except one. This option ensures that if
    an application is currently holding the xtables lock, then `iptables`
    will wait for the lock to be released and then continue execution.
    Without setting the `-w` option, `iptables` will fail immediately if
    the lock is currently held.
    
    This patch adds the missing `-w` option to the port mapper code.
    
    Review: https://reviews.apache.org/r/68152/
---
 .../mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp b/src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp
index c40b57f..b6a619f 100644
--- a/src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp
+++ b/src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp
@@ -342,7 +342,7 @@ Try<Nothing> PortMapper::addPortMapping(
 
       # Within the `chain` go ahead and install the DNAT rule, if it
       # does not exist.
-      (iptables -w -t nat -C %s || iptables -t nat -A %s))~",
+      (iptables -w -t nat -C %s || iptables -w -t nat -A %s))~",
       chain,
       chain,
       chain,


[mesos] 02/02: Added MESOS-9125 to the 1.6.2 CHANGELOG.

Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

grag pushed a commit to branch 1.6.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 6c362e990ce663a484b6800cb969553c5dba4779
Author: Greg Mann <gr...@gmail.com>
AuthorDate: Fri Aug 3 15:51:55 2018 -0700

    Added MESOS-9125 to the 1.6.2 CHANGELOG.
---
 CHANGELOG | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG b/CHANGELOG
index 1e90cd9..d158a99 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -4,6 +4,7 @@ Release Notes - Mesos - Version 1.6.2 (WIP)
 
 ** Bug
   * [MESOS-8418] - mesos-agent high cpu usage because of numerous /proc/mounts reads.
+  * [MESOS-9125] - Port mapper CNI plugin might fail with "Resource temporarily unavailable"
 
 
 Release Notes - Mesos - Version 1.6.1