Skip to content

mbp-1144: Split qtodo-db to a dedicated namespace - #169

Merged
sabre1041 merged 1 commit into
validatedpatterns:mainfrom
mlorenzofr:mbp-1144
Aug 4, 2026
Merged

mbp-1144: Split qtodo-db to a dedicated namespace#169
sabre1041 merged 1 commit into
validatedpatterns:mainfrom
mlorenzofr:mbp-1144

Conversation

@mlorenzofr

Copy link
Copy Markdown
Collaborator

Summary

Extracts the PostgreSQL database from the qtodo namespace into a new, dedicated
qtodo-db namespace as a separate Helm chart and Argo CD application.

Changes

  • New chart: charts/qtodo-db — self-contained Helm chart for the PostgreSQL
    StatefulSet, ExternalSecret, NetworkPolicies, and ServiceAccount
  • values-hub.yaml: registers qtodo-db as a new namespace and ArgoCD
    application (sync-wave 37, ahead of qtodo)
  • charts/qtodo: removes embedded DB resources, updates NetworkPolicy to
    reference the qtodo-db namespace, and updates the app deployment to rely on the
    SPIFFE-based Vault path for DB credentials
  • ACS policies: adds namespace label qtodo-db to relevant policy scope
    annotations
  • Docs: updates docs/multi-tier.md and docs/SYNC-WAVE-INVENTORY.md to
    reflect the new two-namespace layout

@mlorenzofr
mlorenzofr requested review from p-rog and sabre1041 August 3, 2026 14:33

@sabre1041 sabre1041 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks really good. Functionality works as expected. Only 1 small question

Comment thread charts/qtodo/templates/app-deployment.yaml Outdated
@mlorenzofr
mlorenzofr force-pushed the mbp-1144 branch 2 times, most recently from 03d5e76 to 3576062 Compare August 4, 2026 10:17
@mlorenzofr
mlorenzofr requested a review from sabre1041 August 4, 2026 10:21
Comment thread charts/qtodo-db/Chart.yaml Outdated
Comment thread docs/multi-tier.md Outdated

@p-rog p-rog left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Manuel Lorenzo <mlorenzofr@redhat.com>

@sabre1041 sabre1041 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@sabre1041
sabre1041 merged commit 6f3ce20 into validatedpatterns:main Aug 4, 2026
3 checks passed
@mlorenzofr
mlorenzofr deleted the mbp-1144 branch August 4, 2026 13:06
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