edgelesssys / contrast

Deploy and manage confidential containers on Kubernetes
https://docs.edgeless.systems/contrast
GNU Affero General Public License v3.0
160 stars 6 forks source link

logging: enable subsystem warn level by default #565

Closed davidweisse closed 2 weeks ago

davidweisse commented 2 weeks ago

This enables the all log subsystems at warn level or above. These logs will be get printed regardless of the CONTRAST_LOG_SUBSYSTEMS list. The CONTRAST_LOG_LEVEL variable is also considered, so when the global log level is set to error, only error messages will get printed from the subsystems

burgerdev commented 2 weeks ago

@katexochen I wonder if we should enable all log subsystems at WARN level by default, and then ensure some rigour so that actually relevant messages (and only those) are reported at that level.

katexochen commented 2 weeks ago

@katexochen I wonder if we should enable all log subsystems at WARN level by default, and then ensure some rigour so that actually relevant messages (and only those) are reported at that level.

Fine for me! :)

katexochen commented 2 weeks ago

@davidweisse could you update PR title and description?