Skip to content
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

[bitnami/seaweedfs] Add support for PostgreSQL as alternative db #29400

Merged
merged 10 commits into from
Sep 16, 2024

Conversation

juan131
Copy link
Contributor

@juan131 juan131 commented Sep 13, 2024

Description of the change

This PR adds support for PostgreSQL as alternative db both as a chart dependency or using an external database. The latter use case was already supported but required extra manual steps that can be now automated using a init-db K8s job.

Benefits

Users can choose between MariaDB or PostgreSQL as the database for Filer metadata.

Possible drawbacks

None

Applicable issues

Additional information

Installing SeaweedFS using PostgreSQL as a dependency can be achieved installing the chart with the values below:

filer:
  enabled: true
mariadb:
  enabled: false
postgresql:
  enabled: true

In order to use an existing PostgreSQL server and automatically add required db schemas during initialization:

filer:
  enabled: true
mariadb:
  enabled: false
postgresql:
  enabled: false
externalDatabase:
  enabled: true
  store: postgresql
  host: myexternalhost
  port: 5432
  user: myuser
  database: mydatabase
  password: mypassword
  initDatabaseJob:
    enabled: true

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

Signed-off-by: juan131 <jariza@vmware.com>
@bitnami-bot bitnami-bot added the verify Execute verification workflow for these changes label Sep 13, 2024
@juan131 juan131 marked this pull request as ready for review September 13, 2024 14:54
Signed-off-by: juan131 <jariza@vmware.com>
@juan131
Copy link
Contributor Author

juan131 commented Sep 13, 2024

Please DON'T MERGE until we have finished with internal pipeline required changes

bitnami-bot and others added 6 commits September 13, 2024 16:11
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: juan131 <jariza@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: juan131 <jariza@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: juan131 <jariza@vmware.com>
Signed-off-by: juan131 <jariza@vmware.com>
Copy link
Member

@migruiz4 migruiz4 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@juan131 juan131 merged commit c43d3ba into bitnami:main Sep 16, 2024
11 checks passed
@juan131 juan131 deleted the feat/seaweedfs-postgresql branch September 16, 2024 07:51
bartramakers pushed a commit to bartramakers/bitnami-charts that referenced this pull request Sep 18, 2024
mike12806 pushed a commit to mike12806/charts that referenced this pull request Oct 30, 2024
…nami#29400)

Signed-off-by: Michael Faherty <80599490+mike12806@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitnami seaweedfs solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bitnami/seaweedfs] seaweedfs externalDatabase "postgresql" connection error.
4 participants