Skip to content

refactor(postgres): support external secrets - #7

Merged
cmdoret merged 9 commits into
mainfrom
feat/lago-bridge
Aug 13, 2026
Merged

refactor(postgres): support external secrets#7
cmdoret merged 9 commits into
mainfrom
feat/lago-bridge

Conversation

@cmdoret

@cmdoret cmdoret commented Aug 12, 2026

Copy link
Copy Markdown
Member

Allow providing postgres passwords as reference to external secrets. This is useful to support postgres operators such as cloudnative pg, which use Secret resources.

@cmdoret cmdoret self-assigned this Aug 12, 2026
@cmdoret
cmdoret requested a balanced review from Copilot August 12, 2026 16:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Moves PostgreSQL credentials to externally managed Kubernetes Secrets for OpenWebUI and Authentik.

Changes:

  • Reads OpenWebUI’s password from a Secret.
  • Documents external-secret configuration.
  • Relocates Authentik host validation.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tools/helm/lint-values.yaml Exercises external-secret configuration.
templates/postgres_secret.yaml Removes the chart-managed database Secret.
templates/openwebui/deployment.yaml Builds the DSN from a secret-backed environment variable.
templates/NOTES.txt Retains Authentik host validation.
docs/postgresql.md Documents external PostgreSQL secrets and password restrictions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread templates/openwebui/deployment.yaml Outdated
Comment thread templates/openwebui/deployment.yaml
Comment thread templates/openwebui/deployment.yaml Outdated
@cmdoret
cmdoret requested a review from Panaetius August 13, 2026 09:25
@cmdoret
cmdoret merged commit 9f884a1 into main Aug 13, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants