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

feat: Added support to plugin an external sql database #134

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ChaosInTheCRD
Copy link

I have added support to plug in an external database, hopefully in a way that is relatively universal.

My use case was to be able to use Google Cloud SQL, which can be achieved by adding a sidecar container to the API deployment called Cloud SQL Auth Proxy.

To make this generic and easy for people to configure for other use cases, I have:

  • Added a helm value for adding a sidecar container to the API server (this is implemented in the same way as the Prometheus helm chart
  • changed postgresql field to database to make more generic. Technically, dependency track supports other databases (e.g. MySQL)
  • added internal and external fields for users trying to split the database out to an external database (e.g. Google Cloud SQL)

@grugnog
Copy link

grugnog commented Nov 23, 2022

I have pushed a rebase of this to https://github.com/grugnog/evryfs-helm-charts/tree/cloudsql-support

GitHub
evryfs Open Sourced Helm charts. Contribute to grugnog/evryfs-helm-charts development by creating an account on GitHub.

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