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

konnectivity-agent log level #877

Open
radhupr opened this issue Jan 7, 2025 · 5 comments
Open

konnectivity-agent log level #877

radhupr opened this issue Jan 7, 2025 · 5 comments
Labels
enhancement New feature or request

Comments

@radhupr
Copy link

radhupr commented Jan 7, 2025

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

Can we parameterise the verbose level for konnectivity agent logs?
In our case, the amount of logs created and pushed to datadog is too high. We would like to customize the verbose level than completely avoiding the datadog log export.

Describe the solution you would like

Parameter to change the konnectivity agent log level

Describe alternatives you've considered

No response

Additional context

No response

@radhupr radhupr added the enhancement New feature or request label Jan 7, 2025
@makhov
Copy link
Contributor

makhov commented Jan 7, 2025

You can set log levels for each component using --logging k0s argument, eg -l konnectivity-server=0
https://docs.k0sproject.io/stable/cli/k0s_controller/?h=logging#options

@radhupr
Copy link
Author

radhupr commented Jan 7, 2025

@makhov Thanks for the quick response. I added "-l konnectivity-server=0" under controllerPlaneFlags for our standalone deployment.
The change can be seen in the kmc-entrypoint configmap as below:
k0s controller -l konnectivity-server=0 --config=/etc/k0s/k0s.yaml --enable-dynamic-config
But this doesnt have any effect on the logs being generated by the konnectivity-agent daemonset in the cluster. The log level seems to be same (default being 4 I think)

@makhov
Copy link
Contributor

makhov commented Jan 7, 2025

Well, looks like they hardcoded it:

https://github.com/kubernetes-sigs/apiserver-network-proxy/blob/master/cmd/server/main.go#L39

@radhupr
Copy link
Author

radhupr commented Jan 7, 2025

Will it be possible to make the change ?

@makhov
Copy link
Contributor

makhov commented Jan 8, 2025

Well, I submitted the PR to fix it, let's see how fast they will merge it.

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

No branches or pull requests

2 participants