Skip to content

Updates Chart dependencies and includes missing value for postgresql-ha.pgpool.containerPort#90

Open
mmarquez999 wants to merge 2 commits intothingsboard:masterfrom
mmarquez999:master
Open

Updates Chart dependencies and includes missing value for postgresql-ha.pgpool.containerPort#90
mmarquez999 wants to merge 2 commits intothingsboard:masterfrom
mmarquez999:master

Conversation

@mmarquez999
Copy link
Copy Markdown

The Chart's dependencies were outdated, so the closest available version of each dependency is now set.

In the initialized-job.yaml template, the posgresql-ha.pgpool.containerPort values is used:

initContainers:
- name: check-db-ready
image: postgres:{{ index .Values "postgresql-ha" "postgresqlImage" "tag" }}
command: ['sh', '-c',
'until pg_isready -h {{ include "thingsboard.pgpoolservicename" . }} -p {{ index .Values "postgresql-ha" "pgpool" "containerPort" }};
do echo waiting for database; sleep 2; done;']

However, this value is not defined in the default values.yaml, causing the Kubernetes Job to never be completed. This value is now included in order to fix this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant