Skip to content

Commit

Permalink
feat(prerequisites): add postgresql to prerequistes Helm chart as an …
Browse files Browse the repository at this point in the history
…optional dependency (#121)
  • Loading branch information
Masterchen09 authored Sep 10, 2022
1 parent ccc6828 commit 83e3f3a
Show file tree
Hide file tree
Showing 4 changed files with 26 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/datahub/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ description: A Helm chart for LinkedIn DataHub
type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
version: 0.2.94
version: 0.2.95
# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application.
appVersion: 0.8.44
Expand Down
15 changes: 14 additions & 1 deletion charts/datahub/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,7 @@ datahubUpgrade:
batchDelayMs: 100
noCodeDataMigration:
sqlDbType: "MYSQL"
# sqlDbType: "POSTGRES"
podSecurityContext: {}
# fsGroup: 1000
securityContext: {}
Expand Down Expand Up @@ -154,6 +155,17 @@ global:
secretRef: mysql-secrets
secretKey: mysql-root-password

## Use below for usage of PostgreSQL instead of MySQL
# host: "prerequisites-postgresql:5432"
# hostForpostgresqlClient: "prerequisites-postgresql"
# port: "5432"
# url: "jdbc:postgresql://prerequisites-postgresql:5432/datahub"
# driver: "org.postgresql.Driver"
# username: "postgres"
# password:
# secretRef: postgresql-secrets
# secretKey: postgres-password

datahub:
gms:
port: "8080"
Expand Down Expand Up @@ -194,10 +206,11 @@ global:
# hostnames:
# - "broker"
# - "mysql"
# - "postgresql"
# - "elasticsearch"
# - "neo4j"

## Add below to enable SSL for kafka
## Add below to enable SSL for kafka
# credentialsAndCertsSecrets:
# name: datahub-certs
# path: /mnt/datahub/certs
Expand Down
6 changes: 5 additions & 1 deletion charts/prerequisites/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ description: A Helm chart for packages that Datahub depends on
type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
version: 0.0.9
version: 0.0.10
dependencies:
- name: elasticsearch
version: 7.16.2
Expand All @@ -24,6 +24,10 @@ dependencies:
version: 9.1.2
repository: https://charts.bitnami.com/bitnami
condition: mysql.enabled
- name: postgresql
version: 11.2.2
repository: https://charts.bitnami.com/bitnami
condition: postgresql.enabled
# This chart deploys an enterprise version of kafka that requires commercial license
# Note, Schema registry and kafka rest proxy do not require the commercial license
- name: cp-helm-charts
Expand Down
6 changes: 6 additions & 0 deletions charts/prerequisites/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,12 @@ mysql:
# For better security, add mysql-secrets k8s secret with mysql-root-password, mysql-replication-password and mysql-password
existingSecret: mysql-secrets

postgresql:
enabled: false
auth:
# For better security, add postgresql-secrets k8s secret with postgres-password, replication-password and password
existingSecret: postgresql-secrets

cp-helm-charts:
# Schema registry is under the community license
cp-schema-registry:
Expand Down

0 comments on commit 83e3f3a

Please sign in to comment.