Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add xt_recent kernel module so that Kubernetes Services with Session Affinity work in WSL #11159

Open
douglascamata opened this issue Feb 15, 2024 · 3 comments
Labels

Comments

@douglascamata
Copy link

Is your feature request related to a problem? Please describe.

The lack of this kernel module's presence makes Kubernetes Service with Session Affinity unusable and breaks a lot of things inside k8s clusters that could be running either inside the WSL VM directly or through containers inside the VM.

Describe the solution you'd like

I would like the xt_recent to be present by default so that I don't need to recompile the kernel myself to get Kubernetes Services to work with their basic feature set.

Describe alternatives you've considered

I tried recompiling the kernel myself, obviously. It's time consuming and annoying that I have to figure out what's broken, what I need to do, then finally do it. The investigation is not easy at all.

I can't patch literally all the services out there in k8s manifests, Helm charts, operators and so on to avoid using the client session affinity option.

Additional context

This is a revival of #7124, but as a feature request instead of a bug.

Would also love to see CONFIG_NETFILTER_XT_TARGET_CT=y and CONFIG_NETFILTER_XT_TARGET_TPROXY=y present so that things like Cilium could work inside WSL.

We already had open PRs to fix this issue, but they were not accepted without a good and clear explanation on why, as I noted on #7124 (comment).

Copy link

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Closed similar issues:

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

@soredake
Copy link

@kelsey-steele Hi! I've seen that you've fixed #8302, can you look at this issue too, please?

@mkarrmann
Copy link

+1 that this is rather important. It's strange that such an important use case which can be addressed with a simple use case has gone unaddressed for so long. People have been manually compiling their kernels just to make this minor config change for years now. Is there any reason the team is uninterested in addressing this @craigloewen-msft ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants