-
Notifications
You must be signed in to change notification settings - Fork 90
Load ipvs modules before enabling conntrack #521
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can move both the changes in a separate commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Thank you @mneverov for the PR, |
/ok-to-test |
/lgtm |
@aroradaman @mcluseau the reason ipvs backend works in e2e tests is that ipvsadm is installed and initialized, which creates the |
@mneverov can you create a new PR removing the step you mentioned to test that out 🙂 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aroradaman, mneverov The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@aroradaman ptal #522 |
82bbfbb
into
kubernetes-retired:master
What kind of PR is this?
Bugfix
Why this PR is needed / What this PR do?
This PR loads ipvs modules.
Which issue(s) this PR fixes?
When run locally with
ipvs
as a backend,kpng
fail to start with error:The error occur because ipvs modules are not loaded, so there is no
/proc/sys/net/ipv4/vs
directory and the file conntrack cannot be created.Fixes #
Additional information about this PR
Node is running
Ubuntu 22.04.2 LTS
: