Skip to content

Commit b6ee8fd

Browse files
committed
fix(peach-sync): added missing env variables and secrets to peach charts
Signed-off-by: qhanson55 <quinn.hanson@gov.bc.ca>
1 parent f8d7571 commit b6ee8fd

4 files changed

Lines changed: 15 additions & 5 deletions

File tree

charts/pcns/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: nr-permitconnect-navigator-service
33
# This is the chart version. This version number should be incremented each time you make changes
44
# to the chart and its templates, including the app version.
55
# Versions are expected to follow Semantic Versioning (https://semver.org/)
6-
version: 0.0.30
6+
version: 0.0.31
77
kubeVersion: ">= 1.13.0"
88
description: PermitConnect Navigator Service
99
# A chart can be either an 'application' or a 'library' chart.

charts/pcns/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# nr-permitconnect-navigator-service
22

3-
![Version: 0.0.30](https://img.shields.io/badge/Version-0.0.30-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.9.0](https://img.shields.io/badge/AppVersion-0.9.0-informational?style=flat-square)
3+
![Version: 0.0.31](https://img.shields.io/badge/Version-0.0.31-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.9.0](https://img.shields.io/badge/AppVersion-0.9.0-informational?style=flat-square)
44

55
PermitConnect Navigator Service
66

@@ -37,7 +37,7 @@ Kubernetes: `>= 1.13.0`
3737
| autoscaling.targetCPUUtilizationPercentage | int | `80` | |
3838
| chesSecretOverride.password | string | `nil` | |
3939
| chesSecretOverride.username | string | `nil` | |
40-
| config.configMap | object | `{"FRONTEND_APIPATH":"api/v1","FRONTEND_CHES_ROADMAP_BCC":null,"FRONTEND_COMS_APIPATH":null,"FRONTEND_COMS_BUCKETID":null,"FRONTEND_GEOCODER_APIPATH":null,"FRONTEND_OIDC_AUTHORITY":null,"FRONTEND_OIDC_CLIENTID":null,"FRONTEND_OPENSTREETMAP_APIPATH":null,"FRONTEND_ORGBOOK_APIPATH":null,"SERVER_APIPATH":"/v1","SERVER_ATS_APIPATH":null,"SERVER_ATS_TOKENURL":null,"SERVER_BODYLIMIT":"30mb","SERVER_CHEFS_APIPATH":null,"SERVER_CHES_APIPATH":null,"SERVER_CHES_SUBMISSION_CC":null,"SERVER_CHES_TOKENURL":null,"SERVER_DB_HOST":null,"SERVER_DB_POOL_MAX":"10","SERVER_DB_POOL_MIN":"2","SERVER_DB_PORT":"5432","SERVER_ENV":null,"SERVER_LOGLEVEL":"http","SERVER_OBJECTSTORAGE_BUCKET":null,"SERVER_OBJECTSTORAGE_ENDPOINT":null,"SERVER_OBJECTSTORAGE_KEY":null,"SERVER_OIDC_AUTHORITY":null,"SERVER_OIDC_PUBLICKEY":null,"SERVER_OPENMAPS_APIPATH":null,"SERVER_PCNS_APPURL":null,"SERVER_PCNS_NAVEMAIL":null,"SERVER_PEACH_APIPATH":null,"SERVER_PORT":"8080","SERVER_SSO_APIPATH":null,"SERVER_SSO_INTEGRATION":null,"SERVER_SSO_TOKENURL":null}` | These values will be wholesale added to the configmap as is; refer to the pcns documentation for what each of these values mean and whether you need them defined. Ensure that all values are represented explicitly as strings, as non-string values will not translate over as expected into container environment variables. For configuration keys named `*_ENABLED`, either leave them commented/undefined, or set them to string value "true". |
40+
| config.configMap | object | `{"FRONTEND_APIPATH":"api/v1","FRONTEND_CHES_ROADMAP_BCC":null,"FRONTEND_COMS_APIPATH":null,"FRONTEND_COMS_BUCKETID":null,"FRONTEND_GEOCODER_APIPATH":null,"FRONTEND_OIDC_AUTHORITY":null,"FRONTEND_OIDC_CLIENTID":null,"FRONTEND_OPENSTREETMAP_APIPATH":null,"FRONTEND_ORGBOOK_APIPATH":null,"SERVER_APIPATH":"/v1","SERVER_ATS_APIPATH":null,"SERVER_ATS_TOKENURL":null,"SERVER_BODYLIMIT":"30mb","SERVER_CHEFS_APIPATH":null,"SERVER_CHES_APIPATH":null,"SERVER_CHES_SUBMISSION_CC":null,"SERVER_CHES_TOKENURL":null,"SERVER_DB_HOST":null,"SERVER_DB_POOL_MAX":"10","SERVER_DB_POOL_MIN":"2","SERVER_DB_PORT":"5432","SERVER_ENV":null,"SERVER_LOGLEVEL":"http","SERVER_OBJECTSTORAGE_BUCKET":null,"SERVER_OBJECTSTORAGE_ENDPOINT":null,"SERVER_OBJECTSTORAGE_KEY":null,"SERVER_OIDC_AUTHORITY":null,"SERVER_OIDC_PUBLICKEY":null,"SERVER_OPENMAPS_APIPATH":null,"SERVER_PCNS_APPURL":null,"SERVER_PCNS_NAVEMAIL":null,"SERVER_PEACH_APIPATH":null,"SERVER_PEACH_TOKENURL":null,"SERVER_PORT":"8080","SERVER_SSO_APIPATH":null,"SERVER_SSO_INTEGRATION":null,"SERVER_SSO_TOKENURL":null}` | These values will be wholesale added to the configmap as is; refer to the pcns documentation for what each of these values mean and whether you need them defined. Ensure that all values are represented explicitly as strings, as non-string values will not translate over as expected into container environment variables. For configuration keys named `*_ENABLED`, either leave them commented/undefined, or set them to string value "true". |
4141
| config.enabled | bool | `false` | Set to true if you want to let Helm manage and overwrite your configmaps. |
4242
| config.releaseScoped | bool | `false` | This should be set to true if and only if you require configmaps and secrets to be release scoped. In the event you want all instances in the same namespace to share a similar configuration, this should be set to false |
4343
| dbSecretOverride.password | string | `nil` | |
@@ -96,4 +96,4 @@ Kubernetes: `>= 1.13.0`
9696
| ssoSecretOverride.username | string | `nil` | |
9797

9898
----------------------------------------------
99-
Autogenerated from chart metadata using [helm-docs v1.13.1](https://github.com/norwoodj/helm-docs/releases/v1.13.1)
99+
Autogenerated from chart metadata using [helm-docs v1.14.2](https://github.com/norwoodj/helm-docs/releases/v1.14.2)

charts/pcns/templates/peachsynccronjob.yaml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ spec:
5252
value: production
5353
- name: SERVER_LOGLEVEL
5454
value: "verbose"
55-
# Note: PEACH currently unauthenticated. If/when PEACH requires auth, add the required secrets here.
5655
- name: SERVER_DB_DATABASE
5756
valueFrom:
5857
secretKeyRef:
@@ -80,6 +79,16 @@ spec:
8079
secretKeyRef:
8180
key: password
8281
name: {{ include "pcns.configname" . }}-ches-service-account
82+
- name: SERVER_PEACH_CLIENTID
83+
valueFrom:
84+
secretKeyRef:
85+
key: username
86+
name: {{ include "pcns.configname" . }}-peach
87+
- name: SERVER_PEACH_CLIENTSECRET
88+
valueFrom:
89+
secretKeyRef:
90+
key: password
91+
name: {{ include "pcns.configname" . }}-peach
8392
envFrom:
8493
- configMapRef:
8594
name: {{ include "pcns.configname" . }}-config

charts/pcns/values.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,7 @@ config:
173173
SERVER_PCNS_NAVEMAIL: ~
174174

175175
SERVER_PEACH_APIPATH: ~
176+
SERVER_PEACH_TOKENURL: ~
176177

177178
SERVER_PORT: "8080"
178179

0 commit comments

Comments
 (0)