Skip to content

Remove SQLite and deploy Postgres by default#1503

Merged
EItanya merged 4 commits intomainfrom
iplay88keys/drop-sqlite
Mar 16, 2026
Merged

Remove SQLite and deploy Postgres by default#1503
EItanya merged 4 commits intomainfrom
iplay88keys/drop-sqlite

Conversation

@iplay88keys
Copy link
Contributor

@iplay88keys iplay88keys commented Mar 13, 2026

Summary

Removes SQLite as a supported database backend and makes PostgreSQL the only supported option. Bundles a PostgreSQL instance (with pgvector) in the Helm chart so that helm install works out of the box with no external prerequisites.

Closes: #1502

Changes

Helm

  • Bundle PostgreSQL in helm/kagent/templates/postgresql.yaml — deployed when database.postgres.url and database.postgres.urlFile are both empty (the default)
  • Set database.postgres.url to skip the bundled instance and use an external PostgreSQL
  • Remove database.type, database.sqlite, SQLite emptyDir volume, and XDG_CACHE_HOME from the controller deployment

CI

  • Remove strategy.matrix: database: [sqlite, postgres] from test-e2e — single database, no matrix needed
  • Remove separate Postgres service container — postgres is deployed inside the Kind cluster by make helm-install

Helm values

database:
  postgres:
    # Leave empty to deploy the bundled postgres. Set to use external PostgreSQL.
    url: ""
    urlFile: ""
    vectorEnabled: true
    bundled:
      image: pgvector/pgvector:pg18-trixie
      storage: 500Mi
      database: postgres
      user: postgres
      password: kagent

Test plan

  • Go unit tests pass (go test -race -skip 'TestE2E.*' ./...)
  • Helm unit tests pass (helm unittest helm/kagent)
  • TestE2EMemoryWithAgent passes — confirms pgvector and memory table work end-to-end
  • Full E2E suite passes (make helm-install push-test-agent push-test-skill followed by e2e tests via go test -v github.com/kagent-dev/kagent/go/core/test/e2e -failfast -shuffle=on)

Signed-off-by: Jeremy Alvis <jeremy.alvis@solo.io>
Copilot AI review requested due to automatic review settings March 13, 2026 17:57
@iplay88keys iplay88keys requested a review from peterj as a code owner March 13, 2026 17:57
@github-actions github-actions bot added the enhancement-proposal Indicates that this PR is for an enhancement proposal label Mar 13, 2026
Signed-off-by: Jeremy Alvis <jeremy.alvis@solo.io>
Signed-off-by: Jeremy Alvis <jeremy.alvis@solo.io>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@dimetron
Copy link
Contributor

Please consider support https://github.com/cloudnative-pg/cloudnative-pg
To have compatibility from the start and allow HA in production env.
@iplay88keys @EItanya

Signed-off-by: Jeremy Alvis <jeremy.alvis@solo.io>
@EItanya EItanya merged commit 52327c3 into main Mar 16, 2026
22 checks passed
@EItanya EItanya deleted the iplay88keys/drop-sqlite branch March 16, 2026 15:57
@iplay88keys
Copy link
Contributor Author

Follow-up to this in: #1527

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement-proposal Indicates that this PR is for an enhancement proposal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Drop SQLite — PostgreSQL-only database backend

4 participants