Skip to content

docs: disable bundled DB subcharts in external database guides - #3028

Merged
salevine merged 2 commits into
mainfrom
docs/external-db-disable-bundled-subcharts
Aug 12, 2026
Merged

docs: disable bundled DB subcharts in external database guides#3028
salevine merged 2 commits into
mainfrom
docs/external-db-disable-bundled-subcharts

Conversation

@sebastianiv21

@sebastianiv21 sebastianiv21 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

The three external-database guides (MongoDB, PostgreSQL/RDS, Redis) only document the connection URL variables for Kubernetes/Helm installs. With the chart-managed databases left enabled, the chart keeps deploying the embedded database pods, and the Appsmith pod's init containers gate startup on those unused internal databases — the init containers check only the *.enabled flags, not the external URLs.

Changes per guide (Kubernetes snippets):

  • MongoDB: add mongodb.enabled: false and mongodbCommunity.enabled: false (Bitnami subchart and MongoDB Kubernetes Operator resource — whichever the installation uses).
  • PostgreSQL (RDS): add postgresql.enabled: false and APPSMITH_KEYCLOAK_DB_DRIVER: postgresql. Without the explicit driver, charts up to 3.9.0 resolve the driver to h2 and Keycloak silently persists SSO data to a local file inside the pod instead of the external database (chart 3.9.1 infers it via fix(helm): infer Keycloak DB driver from external database URL appsmith#42076, but the explicit value is correct on every version).
  • Redis: add redis.enabled: false.

🤖 Generated with Claude Code

@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
appsmith-docs Ready Ready Preview Aug 12, 2026 10:30pm

Request Review

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@sebastianiv21
sebastianiv21 force-pushed the docs/external-db-disable-bundled-subcharts branch from d4d5d5c to 756fd51 Compare August 12, 2026 22:26
@sebastianiv21
sebastianiv21 marked this pull request as ready for review August 12, 2026 22:28

@salevine salevine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good

@salevine
salevine merged commit f2c0e10 into main Aug 12, 2026
5 checks passed
@sebastianiv21
sebastianiv21 deleted the docs/external-db-disable-bundled-subcharts branch August 12, 2026 23:59
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.

2 participants