[v2] add ability to set controller log level

skupperproject / skupper

Skupper is an implementation of a Virtual Application Network, enabling rich hybrid cloud communication.

http://skupper.io

Apache License 2.0

595 stars 74 forks source link

Closed Karen-Schoener closed 2 days ago

Karen-Schoener commented 3 weeks ago

Controller log level can be configured via a configmap.

Fixes #1698

Karen-Schoener commented 3 weeks ago

With this PR, the user can optionally create configmap skupper-log-config in order to configure the skupper-controller log level.

If the configmap is not present, the v2 skupper-controller defaults the log level to INFO.

If the configmap is deleted, the skupper-controller sets the log level back to INFO.

Karen-Schoener commented 3 weeks ago

The 'SKUPPER_' prefix to the key may be a little redundant given the ConfigMap name is skupper-log-level, but this looks fine to me.

Renamed SKUPPER_CONTROLLER_LOG_LEVEL to CONTROLLER_LOG_LEVEL.