Skip to content

wait for iptables lock in support script - #2000

Merged
mavenugo merged 1 commit into
moby:masterfrom
trapier:support_script_wait_for_iptables_lock
Oct 24, 2017
Merged

mavenugo merged 1 commit into
moby:masterfrom
trapier:support_script_wait_for_iptables_lock

Conversation

@trapier

@trapier trapier commented Oct 24, 2017

Copy link
Copy Markdown

Addresses failure to collect iptables information if lock is held during
data capture. Follows the reccomendation of iptables stderr in this
scenario:

Another app is currently holding the xtables lock. Perhaps you want to
use the -w option?

Signed-off-by: Trapier Marshall trapier.marshall@docker.com

Addresses failure to collect iptables information if lock is held during
data capture. Follows the reccomendation of iptables stderr in this
scenario:

```
Another app is currently holding the xtables lock. Perhaps you want to
use the -w option?
```

Signed-off-by: Trapier Marshall <trapier.marshall@docker.com>
@codecov-io

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@0caae82). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2000   +/-   ##
=========================================
  Coverage          ?   38.01%           
=========================================
  Files             ?      137           
  Lines             ?    27362           
  Branches          ?        0           
=========================================
  Hits              ?    10402           
  Misses            ?    15688           
  Partials          ?     1272

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0caae82...2491a95. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants