Skip to content

Improve chart usability and default handling

Clemens Beckrequested to merge
cb-change-tls-and-logs into main

What does this MR do?

Change default settings related to:

  • Logging: Use json logs and expose log level.
  • TLS: Derive probe scheme, Ingress configuration and certmanager configuration from config.tlsDisable to simplify configure the chart for Ingress-terminated TLS and no TLS at all.
  • UI: Disable the OpenBao UI by default. No user has to interact through the UI, so we can disable it by default.

Author's Checklist

For anything in this list which will not be completed, please provide a reason in the MR discussion.

Required

  • Ensure a release milestone is set.
  • MR title and description are up to date, accurate, and descriptive.
  • MR targeting the appropriate branch.
  • MR has a green pipeline on GitLab.com.
  • When ready for review, MR is labeled workflowready for review per the MR workflow.

Expected

  • Test plan indicating conditions for success has been posted and passes.
  • Documentation is created or updated.
    • Will be done in the GitLab chart MR once this merged
  • Tests are added.

Related issues

Closes #23

Edited by Clemens Beck

Merge request reports