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

Additional changes for lima #1555

Closed
wants to merge 3 commits into from
Closed

Additional changes for lima #1555

wants to merge 3 commits into from

Conversation

nirs
Copy link
Member

@nirs nirs commented Sep 12, 2024

This change include 3 changes from k8s-lima repo that seems unneeded in my tests. Posting them for testing on other systems.

@nirs nirs requested a review from rakeshgm September 12, 2024 19:06
@nirs nirs force-pushed the lima-flannel branch 3 times, most recently from 4dd76ad to ce3b880 Compare September 13, 2024 21:43
@nirs nirs changed the title Modify flannel to use lima0 interface Additional changes for lima Sep 13, 2024
@nirs nirs force-pushed the lima-flannel branch 2 times, most recently from 8260598 to affecd7 Compare September 18, 2024 19:01
Change taken from k8s-lima repo[1]. In my tests it does not make any
difference, posting for testing on other system.

[1] https://github.com/raghavendra-talur/k8s-lima

Thanks: Raghavendra Talur <[email protected]>
Signed-off-by: Nir Soffer <[email protected]>
Enable the option for iptables[1] and nftables[2]. I'm not sure which
one we use but it does not harm to change the one we don't use.

Change taken from k8s-lima repo[3]. In my tests it does not make any
difference, posting for testing on other system.

[1] https://kubernetes.io/docs/reference/config-api/kube-proxy-config.v1alpha1/#kubeproxy-config-k8s-io-v1alpha1-KubeProxyIPTablesConfiguration
[2] https://kubernetes.io/docs/reference/config-api/kube-proxy-config.v1alpha1/#kubeproxy-config-k8s-io-v1alpha1-KubeProxyNFTablesConfiguration
[3] https://github.com/raghavendra-talur/k8s-lima

Thanks: Raghavendra Talur <[email protected]>
Signed-off-by: Nir Soffer <[email protected]>
Change taken from k8s-lima repo[1]. In my tests it does not make any
difference, posting for testing on other system.

[1] https://github.com/raghavendra-talur/k8s-lima

Thanks: Raghavendra Talur <[email protected]>
Signed-off-by: Nir Soffer <[email protected]>
@nirs
Copy link
Member Author

nirs commented Oct 8, 2024

Seems that we don't need this.

@nirs nirs closed this Oct 8, 2024
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