From 766d0baa8f2c2c4754b9e06534d2cb07f7d54d32 Mon Sep 17 00:00:00 2001 From: Matt Kaar Date: Mon, 2 May 2022 09:01:23 -0400 Subject: [PATCH] Fix pgpass.conf ownership Ownership of `/var/lib/pgadmin` in the pgAdmin Docker image changed with [this commit](https://github.com/postgres/pgadmin4/commit/e7dc6df7230c9ccc74bf9853fb614407b861a42a). The appliance deployment of pgAdmin places `pgpass.conf` in that directory with credentials for the database. The ownership workaround is patterned after this fix: https://github.com/rowanruseler/helm-charts/issues/72#issuecomment-1002300143 Also, this commit pins pgAdmin to the latest Helm chart release (`dpage/pgadmin4:6.7`). --- foundry/common/pgadmin4.values.yaml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/foundry/common/pgadmin4.values.yaml b/foundry/common/pgadmin4.values.yaml index a1b5907..228927e 100644 --- a/foundry/common/pgadmin4.values.yaml +++ b/foundry/common/pgadmin4.values.yaml @@ -8,7 +8,7 @@ image: registry: docker.io repository: dpage/pgadmin4 # Overrides the image tag whose default is the chart appVersion. - tag: "" + tag: "6.7" pullPolicy: IfNotPresent ## Deployment annotations @@ -228,11 +228,11 @@ VolumePermissions: ## extraInitContainers: | - name: prepare-pgpass - image: "dpage/pgadmin4:latest" + image: "dpage/pgadmin4:6.7" command: - "sh" - "-c" - - "cp /pgpass.conf /var/lib/pgadmin/pgpass.conf && chown pgadmin:pgadmin /var/lib/pgadmin/pgpass.conf && chmod 600 /var/lib/pgadmin/pgpass.conf" + - "cp /pgpass.conf /var/lib/pgadmin/pgpass.conf && chown pgadmin:root /var/lib/pgadmin/pgpass.conf && chmod 600 /var/lib/pgadmin/pgpass.conf" volumeMounts: - name: pgadmin-data mountPath: /var/lib/pgadmin