Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Helm Chart: fix connect err when setting off pg bitnami chart #15143

Conversation

iam-robi
Copy link

Current status of chart would only set POSTGRES_SEEDS to the default bitnami pg deployed.

However for production usage it was not possible to set an external host database with temporal . Now chart takes the database host set in global, if none is set it defaults to bitnami pg host deployed with the chart.

Current status of chart would only set POSTGRES_SEEDS to the default bitnami pg deployed by default. However for prod it was not possible to use an external host database. Now chart takes the database host set in global, if none is set it defaults to default bitnami pg host deployed with the chart.
@github-actions github-actions bot added area/platform issues related to the platform kubernetes labels Jul 29, 2022
@marcosmarxm marcosmarxm changed the title 🐛 fix connect err when setting off pg bitnami chart 🐛 Helm Chart: fix connect err when setting off pg bitnami chart Aug 1, 2022
@marcosmarxm
Copy link
Member

thanks for the contribution @iam-robi I'll request the team to take a review this week.

@xpuska513
Copy link
Contributor

Hmm, it should be read from the configmap itself...

I'll include this change in my next PR today

@marcosmarxm
Copy link
Member

Hmm, it should be read from the configmap itself...

I'll include this change in my next PR today

It's possible to review this contribution and merge it @xpuska513

@marcosmarxm marcosmarxm requested a review from xpuska513 August 8, 2022 17:05
@madihakh
Copy link

madihakh commented Aug 9, 2022

Hi, I am also experiencing this issue of the wrong temporal Postgres hostname. I am expecting to set the externalDatabase.host value but it is still taking picking releaseName-airbyte default value
do you know when this issue can be fixed or this PR will merge?
thanks

@xpuska513
Copy link
Contributor

Changes included in #15510

@xpuska513 xpuska513 closed this Aug 10, 2022
@xpuska513
Copy link
Contributor

It's possible to review this contribution and merge it @xpuska513

Sorry for long reply, was on sick leave for half of this week, changes included in #15510

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/platform issues related to the platform community kubernetes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants