Add mount and network options to the cleanup of network rules#54477
Add mount and network options to the cleanup of network rules#54477kayoch1n wants to merge 2 commits intokubernetes:mainfrom
Conversation
|
Welcome @kayoch1n! |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
Description
This PR attempts to make minor changes to the command of the network traffic cleanup:
/lib/modules, and--network=hostFirstly, the cleanup runs inside container and invokes the command
kube-proxy --cleanup, which instead invokesmodprobe -va ip_vsand fails due to the fact that/lib/modulesis not mounted.This warning won't cause kubeproxy to fail, but it is somewhat frustrating. Adding read-only mount of
-v /lib/modules:/lib/modules:roeliminate this warning.Secondly, without
--network=host, kube-proxy inside container cannot touch the host's network namespace. Adding this option allows kube-proxy to remove iptable rules.Issue
Closes: #