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

runner: configure Helm action cfg log levels #625

Merged
merged 2 commits into from
Mar 2, 2023
Merged

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Mar 1, 2023

This reduces the amount of log lines pushed to debug by configuring the kube client and storage loggers to only log to trace.

In addition, the log buffer used in events will now just contain the most relevant information about a failure as reported by the Helm action itself, and not the in-depth information from the underlying client and/or storage.

@hiddeco hiddeco added enhancement New feature or request area/ux In pursuit of a delightful user experience labels Mar 1, 2023
@hiddeco
Copy link
Member Author

hiddeco commented Mar 1, 2023

@stefanprodan if you quickly want to test this, you can use docker.io/hiddeco/helm-controller:log-levels.

This reduces the amount of log lines pushed to `debug` by configuring the kube
client and storage loggers to only log to `trace`.

In addition, the log buffer used in events will now just contain the
most relevant information about a failure as reported by the Helm action
itself, and not the in-depth information from the underlying client
and/or storage.

Signed-off-by: Hidde Beydals <hidde@hhh.computer>
Signed-off-by: Hidde Beydals <hidde@hhh.computer>
@hiddeco hiddeco merged commit 8d7b650 into main Mar 2, 2023
@hiddeco hiddeco deleted the helm-log-levels branch March 2, 2023 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ux In pursuit of a delightful user experience enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants