Skip to content

Ability to configure wait for the xtables lock#58

Open
retr0h wants to merge 1 commit into
k3s-io:masterfrom
retr0h:iptables-wait-seconds
Open

Ability to configure wait for the xtables lock#58
retr0h wants to merge 1 commit into
k3s-io:masterfrom
retr0h:iptables-wait-seconds

Conversation

@retr0h
Copy link
Copy Markdown

@retr0h retr0h commented Oct 4, 2023

Any invocation of the iptables command results in an xtables lock, which can lead to the iptables command exiting(4). Racy systems which make use of inspecting iptables can cause an error.

However, maybe this is ultimately not an issue as kubernetes will retry failed pods.

IPTABLES(8):
-w, --wait [seconds]
Wait for the xtables lock. To prevent multiple instances of the
program from running concurrently, an attempt will be made to obtain
an exclusive lock at launch. By default, the program will exit if
the lock cannot be obtained. This option will make the program wait
(indefinitely or for optional seconds) until the exclusive lock can
be obtained.

@retr0h retr0h force-pushed the iptables-wait-seconds branch from a2cf382 to 32dadc6 Compare October 4, 2023 18:57
Any invocation of the iptables command results in an xtables lock, which
can lead to the iptables command exiting(4).  Racy systems which make
use of inspecting iptables can cause an error.

However, maybe this is ultimately not an issue as kubernetes will retry
failed pods.

> IPTABLES(8):
  -w, --wait [seconds]
          Wait for the xtables lock.  To prevent multiple instances of the
          program from running concurrently, an attempt will be made to obtain
          an exclusive lock at launch.  By default, the program will exit if
          the lock cannot be obtained.  This option will make the program wait
          (indefinitely or for optional seconds) until the exclusive lock can
          be obtained.

Signed-off-by: נυαη נυαηѕση <[email protected]>
@retr0h retr0h force-pushed the iptables-wait-seconds branch from 32dadc6 to d246770 Compare October 4, 2023 18:58
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.

1 participant