github/deprecated-helm-charts
1
0
Fork 0
mirror of https://github.com/helm/charts.git synced 2026-08-23 22:37:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

[stable/kong] drop cassandra sub-chart (#18567)

Browse Source 
Reasons:
- Cassandra chart has been incubating for a very long time and we have
  stability issues when Cassandra is running on k8s.
- Any serious user who is backing Kong with Cassandra, should be running
  and maintaining cassandra on their own any ways (outside k8s as well).
- Running Kong on k8s with Cassandra buys little. We would like to push
  users to use DB-less mode or use Postgres.
- We still support running Kong with Cassandra on k8s, only the
  sub-chart is being removed, (intentionally) increasing the friction when
  trying to use Kong with Cassandra on k8s.

Signed-off-by: Harry Bagdi <harrybagdi@gmail.com>
This commit is contained in:
Harry
2019-11-05 14:30:31 -08:00
committed by Kubernetes Prow Robot
parent 3350265788
commit ef9e0ebb86
11 changed files with 15 additions and 59 deletions
Download Patch File Download Diff File Expand all files Collapse all files
stable/kong/Chart.yaml
+1 -1
View File
@@ -12,5 +12,5 @@ maintainers:
name: kong
sources:
- https://github.com/Kong/kong
version: 0.25.0
version: 0.26.0
appVersion: 1.3
stable/kong/README.md
+11 -13
View File
@@ -139,17 +139,19 @@ for the service definition to work properly.
Kong has a choice of either Postgres or Cassandra as a backend datatstore.
This chart allows you to choose either of them with the `env.database`
parameter. Postgres is chosen by default.
parameter. Postgres is chosen by default.
Additionally, this chart allows you to use your own database or spin up a new
instance by using the `postgres.enabled` or `cassandra.enabled` parameters.
Enabling both will create both databases in your cluster, but only one
will be used by Kong based on the `env.database` parameter.
Postgres is enabled by default.
This chart allows you to bring your own database that you manage or spin up
separately (recommended) or spin up a new Postgres instance using
the `postgres.enabled` parameter.
Note: Cassandra deployment via a sub-chart was previously supported but
the support has now been dropped due to stability issues.
You can still deploy Cassandra on your own and configure Kong to use
that via the `env.database` parameter.
| Parameter | Description | Default |
| ------------------------------| ------------------------------------------------------------------------| ----------------------|
| cassandra.enabled | Spin up a new cassandra cluster for Kong | `false` |
| postgresql.enabled | Spin up a new postgres instance for Kong | `true` |
| waitImage.repository | Image used to wait for database to become ready | `busybox` |
| waitImage.tag | Tag for image used to wait for database to become ready | `latest` |
@@ -159,10 +161,6 @@ Postgres is enabled by default.
| env.pg_password | Postgres database password (required if you are using your own database)| `kong` |
| env.pg_host | Postgres database host (required if you are using your own database) | `` |
| env.pg_port | Postgres database port | `5432` |
| env.cassandra_contact_points | Cassandra contact points (required if you are using your own database) | `` |
| env.cassandra_port | Cassandra query port | `9042` |
| env.cassandra_keyspace | Cassandra keyspace | `kong` |
| env.cassandra_repl_factor | Replication factor for the Kong keyspace | `2` |
| dblessConfig.configMap | Name of an existing ConfigMap containing the `kong.yml` file. This must have the key `kong.yml`.| `` |
| dblessConfig.config | Yaml configuration file for the dbless (declarative) configuration of Kong | see in `values.yaml` |
@@ -182,13 +180,13 @@ kong:
```
For complete list of Kong configurations please check https://getkong.org/docs/latest/configuration/.
For complete list of Kong configurations please check https://docs.konghq.com/latest/configuration/.
Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`. For example,
```console
$ helm install stable/kong --name my-release \
--set=image.tag=1.3,env.database=cassandra,cassandra.enabled=true
--set=image.tag=1.3,env.database=postgres,postgres.enabled=true
```
Alternatively, a YAML file that specifies the values for the above parameters
stable/kong/ci/cassandra.yaml
-7
View File
@@ -1,7 +0,0 @@
env:
database: cassandra
cassandra:
enabled: true
postgresql:
enabled: false
stable/kong/requirements.lock
+3 -6
View File
@@ -1,9 +1,6 @@
dependencies:
- name: postgresql
repository: https://kubernetes-charts.storage.googleapis.com/
version: 6.3.12
- name: cassandra
repository: https://kubernetes-charts-incubator.storage.googleapis.com/
version: 0.13.3
digest: sha256:6c1d6473c869ee26836bec19f3638ae148d11398a4f16dce25f62beeaf67fb67
generated: "2019-10-10T11:19:21.654989947-07:00"
version: 6.3.15
digest: sha256:1110c0b8497a990af34f196753420ab9b1b00fed77c540bce43542bdedd74b9b
generated: "2019-10-31T09:56:30.102440266-07:00"
stable/kong/requirements.yaml
-4
View File
@@ -3,7 +3,3 @@ dependencies:
version: ~6.3.0
repository: https://kubernetes-charts.storage.googleapis.com/
condition: postgresql.enabled
- name: cassandra
version: ~0.13.0
repository: https://kubernetes-charts-incubator.storage.googleapis.com/
condition: cassandra.enabled
stable/kong/templates/_helpers.tpl
-9
View File
@@ -18,11 +18,6 @@ We truncate at 63 chars because some Kubernetes name fields are limited to this
{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
{{- end -}}
{{- define "kong.cassandra.fullname" -}}
{{- $name := default "cassandra" .Values.cassandra.nameOverride -}}
{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
{{- end -}}
{{- define "kong.dblessConfig.fullname" -}}
{{- $name := default "kong-custom-dbless-config" .Values.dblessConfig.nameOverride -}}
{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
@@ -234,10 +229,6 @@ the extra slash.
name: {{ template "kong.postgresql.fullname" . }}
key: postgresql-password
{{- end }}
{{- if .Values.cassandra.enabled }}
- name: KONG_CASSANDRA_CONTACT_POINTS
value: {{ template "kong.cassandra.fullname" . }}
{{- end }}
{{- include "kong.env" . | nindent 2 }}
command: [ "/bin/sh", "-c", "until kong start; do echo 'waiting for db'; sleep 1; done; kong stop" ]
volumeMounts:
stable/kong/templates/deployment.yaml
-4
View File
@@ -181,10 +181,6 @@ spec:
name: {{ template "kong.postgresql.fullname" . }}
key: postgresql-password
{{- end }}
{{- if .Values.cassandra.enabled }}
- name: KONG_CASSANDRA_CONTACT_POINTS
value: {{ template "kong.cassandra.fullname" . }}
{{- end }}
{{- if (and (not .Values.ingressController.enabled) (eq .Values.env.database "off")) }}
- name: KONG_DECLARATIVE_CONFIG
value: "/kong_dbless/kong.yml"
stable/kong/templates/migrations-post-upgrade.yaml
-4
View File
@@ -71,10 +71,6 @@ spec:
name: {{ template "kong.postgresql.fullname" . }}
key: postgresql-password
{{- end }}
{{- if .Values.cassandra.enabled }}
- name: KONG_CASSANDRA_CONTACT_POINTS
value: {{ template "kong.cassandra.fullname" . }}
{{- end }}
{{- include "kong.env" . | indent 8 }}
command: [ "/bin/sh", "-c", "kong migrations finish" ]
volumeMounts:
stable/kong/templates/migrations-pre-upgrade.yaml
-4
View File
@@ -71,10 +71,6 @@ spec:
name: {{ template "kong.postgresql.fullname" . }}
key: postgresql-password
{{- end }}
{{- if .Values.cassandra.enabled }}
- name: KONG_CASSANDRA_CONTACT_POINTS
value: {{ template "kong.cassandra.fullname" . }}
{{- end }}
{{- include "kong.env" . | indent 8 }}
command: [ "/bin/sh", "-c", "kong migrations up" ]
volumeMounts:
stable/kong/templates/migrations.yaml
-4
View File
@@ -66,10 +66,6 @@ spec:
name: {{ template "kong.postgresql.fullname" . }}
key: postgresql-password
{{- end }}
{{- if .Values.cassandra.enabled }}
- name: KONG_CASSANDRA_CONTACT_POINTS
value: {{ template "kong.cassandra.fullname" . }}
{{- end }}
{{- include "kong.env" . | indent 8 }}
command: [ "/bin/sh", "-c", "kong migrations bootstrap" ]
volumeMounts:
stable/kong/values.yaml
-3
View File
@@ -355,9 +355,6 @@ podSecurityPolicy:
# will be used by Kong based on the `env.database` parameter.
# Postgres is enabled by default.
# Cassandra chart configs
cassandra:
enabled: false
# PostgreSQL chart configs
postgresql: