Skip to content

Noobaa HA - deploy HA postgres cluster using CNPG operator #1560

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

dannyzaken
Copy link
Member

@dannyzaken dannyzaken commented Mar 19, 2025

Explain the changes

  • DB Deployment Changes:

    • Changed default deployment to use a DB cluster instead of standalone DB
    • Added use-standalone-db flag to the install CLI to deploy a DB sts instead of a cluster.
    • Updated DB image to quay.io/sclorg/postgresql-16-c9s
  • NooBaa CRD Changes:

    • Added NooBaaDBSpec with fields for Postgres major version, instances, resources, and storage
      • NooBaaDBSpec is only used for cnpg cluster reconciliation.
    • Added DBStatus field to reflect the cluster status
  • System Reconciliation Flow:

    • Refactored reconciliation logic to handle DB cluster deployment
    • Added DB cluster creation and updating
    • Added logic to support both standalone DB and clustered DB
  • System Installation Updates:

    • Default installation now deploys a DB cluster.
  • Tests:

    • For test workflows that use minikube - run using --use-standalone-db.
      • This should be changed eventually. Currently, these tests use an old version of minikube\kuberentes that does not allow the installation of cnpg.

Gaps

  • Upgrade and import from an existing DB are not supported yet.
  • Specific tests will be added in future PRs. Current tests cover basic system operations.
  • Update the minikube version used by test workflows to support deploying with CNPG
  • Docs will be added in future PRs

@dannyzaken dannyzaken changed the title Noobaa HA - deploy HA postgres cluster Noobaa HA - deploy HA postgres cluster using CNPG operator Mar 19, 2025
@dannyzaken dannyzaken force-pushed the danny-cnpg branch 12 times, most recently from d6a74ce to 2716abe Compare March 19, 2025 18:18
@dannyzaken dannyzaken force-pushed the danny-cnpg branch 9 times, most recently from 9080b55 to 2c168be Compare March 22, 2025 16:53
Changes:
- **DB Deployment Changes**:
  - Changed default deployment to use a DB cluster instead of standalone DB
  - Added `use-standalone-db` flag to the install CLI to deploy a DB sts instead of a cluster.
  - Updated DB image to `quay.io/sclorg/postgresql-16-c9s`

- **NooBaa CRD Changes**:
  - Added NooBaaDBSpec with fields for Postgres major version, instances, resources, and storage
    - NooBaaDBSpec is only used for cnpg cluster reconciliation.
  - Added DBStatus field to reflect the cluster status

- **System Reconciliation Flow**:
  - Refactored reconciliation logic to handle DB cluster deployment
  - Added DB cluster creation and updating
  - Added logic to support both standalone DB and clustered DB

- **Core System Installation Updates**:
  - **Default installation** now deploys a DB cluster.

- **Gaps**
  - Upgrade and import from an existing DB is not supported yet.

Signed-off-by: Danny Zaken <[email protected]>
@dannyzaken dannyzaken merged commit a03fb37 into noobaa:master Mar 24, 2025
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants