You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Greg Mann (JIRA)" <ji...@apache.org> on 2018/08/03 21:19:00 UTC

[jira] [Commented] (MESOS-9129) Port mapper CNI plugin should use '-n' option with 'iptables --list'

    [ https://issues.apache.org/jira/browse/MESOS-9129?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16568778#comment-16568778 ] 

Greg Mann commented on MESOS-9129:
----------------------------------

{code:java}
commit 6177503f52e765885a78721b086de504346578ca
Author: Greg Mann <gr...@mesosphere.io>
Date: Fri Aug 3 13:42:30 2018 -0700

Avoided hostname lookups in the CNI port mapper.

Using the `-n` option with `iptables --list` prevents potentially
costly reverse hostname lookups. This patch adds this option to the
relevant invocation of `iptables` in the CNI port mapper.

Review: https://reviews.apache.org/r/68167/{code}

> Port mapper CNI plugin should use '-n' option with 'iptables --list'
> --------------------------------------------------------------------
>
>                 Key: MESOS-9129
>                 URL: https://issues.apache.org/jira/browse/MESOS-9129
>             Project: Mesos
>          Issue Type: Improvement
>    Affects Versions: 1.6.1
>            Reporter: Greg Mann
>            Assignee: Greg Mann
>            Priority: Major
>              Labels: mesosphere
>             Fix For: 1.7.0
>
>
> Without the {{-n}} option, [this iptables command|https://github.com/apache/mesos/blob/9457dce1d99b5616d1b5eeb9a344733f6320d7b5/src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp#L313] could result in a large number of reverse hostname lookups, which could take a while.
> We should use the -n option there to avoid this issue.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)