Add healthzBindAddress parameter to kubeProxy conf#7384
Add healthzBindAddress parameter to kubeProxy conf#7384dodgex wants to merge 1 commit intok0sproject:mainfrom
Conversation
twz123
left a comment
There was a problem hiding this comment.
Thanks! Can you add the new field to docs/configuration.md, too, in the spec.network.kubeProxy section? (Maybe also add the missing metricsBindAddress?)
|
Given that kube-proxy has health endpoints, I wonder if it would make sense to add readiness and liveness probes to the DaemonSet? 🤔 Are other distros doing this? |
188f8d6 to
c2baff3
Compare
|
This pull request has merge conflicts that need to be resolved. |
c2baff3 to
9f830e1
Compare
|
I should have addressed all your comments. As the build does still not work locally, I unfortunately can't verify the results by myself. But I hope CI is now happy. :) |
|
You need to sign-off your commit using
|
Prior to this the value was hardcoded to an empty string. As kubeproxy ignores the related commandline argument when a config file is present, this option was not configurable otherwise. Fixes: k0sproject#7363 Signed-off-by: dodgex <git@kujanssen.de>
9f830e1 to
10bf7dc
Compare
Should be fixed |
Weird! Can you maybe upload the full terminal output? Which OS are you using? |
|
|
As I realized that there are some steps cached, i ran Here is the full log, in case you might see something suspicious. |
Just saw that question as part of your message. I am on windows 11 with Docker Desktop 4.66.1; running the command in WSL2 Ubuntu 24.04 |
Somehow anticipated Windows. Probably git's autocrlf settings mess up the patch file. Can you check if it has CRLF endings and convert it to LF only? If that's the case, then we could set the git attributes for the patch file accordingly. |
|
Do you have any possibility to test this change? Since we don't have any integration test coverage for this, I'd like to have at least a confirmation that this has been tested manually 🙃 |
Strange. While I used Goland from windows to access the files in my WSL, the actual clone was on linux/wsl. I just recloned the repo and tested on wsl only. but even then it was CRLF. after converting the riscv patch file with dos2unix, the
I can give it a try, now that it seems, that I can build the project. Now I only need to know what (and how) I need on my k0s test vm from my local build (can re-build there if it helps). |
|
The I checked my Maybe it would make sense, to consider adding a |
I built k0s on my Test VM and replaced the k0s binary in |
Description
Prior to this the value was hardcoded to an empty string.
As kubeproxy ignores the related commandline argument when a config file is present, this option was not configurable otherwise.
Fixes: #7363
The changes are made to mirror changes of #2378.
Type of change
How Has This Been Tested?
make lintfailed, unable to findgolang.shto check whats wrongunfortunately the same for
make build && git diff --exit-codeand the other commands mentioned in https://docs.k0sproject.io/stable/contributors/testing/Checklist