Skip to content

Commit

Permalink
Merge pull request #60 from colearendt/make-required-values-mandatory…
Browse files Browse the repository at this point in the history
…-in-postgrest

Make required values mandatory in postgrest
  • Loading branch information
colearendt authored Jul 29, 2023
2 parents bbd89b6 + 7fb27a9 commit 9470843
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 8 deletions.
2 changes: 1 addition & 1 deletion charts/postgrest/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: postgrest
description: A Helm chart for deploying Postgrest to Kubernetes
type: application
version: 0.3.4
version: 0.3.5
appVersion: v11.1.0
icon: https://avatars.githubusercontent.com/u/15115011?s=200&v=4
maintainers:
Expand Down
4 changes: 4 additions & 0 deletions charts/postgrest/NEWS.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
# 0.3.5

- Make values `postgrest.dbUri` and `postgrest.jwtSecret` required

# 0.3.4

- Add configuration for the `pod.adminPort` and `service.adminPort`
Expand Down
10 changes: 5 additions & 5 deletions charts/postgrest/README.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
# postgrest

![Version: 0.3.4](https://img.shields.io/badge/Version-0.3.4-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v11.1.0](https://img.shields.io/badge/AppVersion-v11.1.0-informational?style=flat-square)
![Version: 0.3.5](https://img.shields.io/badge/Version-0.3.5-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v11.1.0](https://img.shields.io/badge/AppVersion-v11.1.0-informational?style=flat-square)

## Installing the Chart

To install the chart with the release name `my-release` at version 0.3.4:
To install the chart with the release name `my-release` at version 0.3.5:

```bash
helm repo add colearendt https://colearendt.github.io/helm
helm install my-release colearendt/postgrest --version=0.3.4
helm install my-release colearendt/postgrest --version=0.3.5
```

#### _A Helm chart for deploying Postgrest to Kubernetes_
Expand Down Expand Up @@ -57,9 +57,9 @@ helm install my-release colearendt/postgrest --version=0.3.4
| postgrest.dbPool | string | `""` | |
| postgrest.dbPoolTimeout | string | `""` | |
| postgrest.dbSchema | string | `""` | |
| postgrest.dbUri | string | `""` | |
| postgrest.dbUri | string | `""` | The dbUri for connecting to the database. Required |
| postgrest.jwtAud | string | `""` | |
| postgrest.jwtSecret | string | `""` | |
| postgrest.jwtSecret | string | `""` | The JWT Secret used for constructing JWTs. Required |
| postgrest.maxRows | string | `""` | |
| postgrest.openApiServerProxyUri | string | `""` | |
| postgrest.preRequest | string | `""` | |
Expand Down
4 changes: 2 additions & 2 deletions charts/postgrest/templates/secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@ metadata:
namespace: {{ $.Release.Namespace }}
stringData:
db-uri.txt: |-
{{- .Values.postgrest.dbUri | nindent 4 }}
{{- required "Missing required value postgrest.dbUri" .Values.postgrest.dbUri | nindent 4 }}
jwt-secret.txt: |-
{{- .Values.postgrest.jwtSecret | nindent 4 }}
{{- required "Missing required value postgrest.jwtSecret" .Values.postgrest.jwtSecret | nindent 4 }}
{{- if .Values.secret }}
{{- toYaml .Values.secret | nindent 2 }}
{{- end }}
2 changes: 2 additions & 0 deletions charts/postgrest/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ command: []
args: []

postgrest:
# -- The dbUri for connecting to the database. Required
dbUri: ""
dbSchema: ""
dbAnonRole: ""
Expand All @@ -52,6 +53,7 @@ postgrest:
serverHost: ""
serverPort: "9000"
openApiServerProxyUri: ""
# -- The JWT Secret used for constructing JWTs. Required
jwtSecret: ""
secretIsBase64: ""
jwtAud: ""
Expand Down

0 comments on commit 9470843

Please sign in to comment.