airbyte icon indicating copy to clipboard operation
airbyte copied to clipboard

[helm] `DATABASE_URL` ignored by Temporal pods

Open monik-a opened this issue 2 years ago • 9 comments

What method are you using to run Airbyte?

Kubernetes

Platform Version or Helm Chart Version

0.50.34

What step the error happened?

Other

Revelant information

We have the externalDatabase.jdbcUrl parameter used to enable SSL when Airbyte connects to its database. (cf. https://github.com/airbytehq/airbyte/issues/17995)

The main database is therefore covered. This however doesn't work for temporal components and their dbs (temporal and temporal-visibility). I tried to inject the DATABASE_URL into the temporal container and it is present there as an env variable, but it is very much ignored when connection happens. There's also no variables I found in the doc that I could pass connection params to and have them respected. I'm logging it as an issue, but if there's another way to enforce it, please let me know 🙏

Cf. this comment too

Relevant log output

No response

monik-a avatar Nov 30 '23 13:11 monik-a

any update to related this issue? because i have also the similar issue while passing the jdbcUrl on helm charts it seems failing:

2024-02-06T06:54:26.452Z [31mERROR[0m Unable to create SQL database. {"error": "unable to connect to DB, tried default DB names: postgres,defaultdb, errors: [pq: connection is insecure (try using `sslmode=require`) pq: connection is insecure (try using `sslmode=require`)]

mrfzy00 avatar Feb 06 '24 07:02 mrfzy00

@monik-a @mrfzy00 Hello guys. Have you ever tried below?

temporal:
  extraEnv:
    - name: SQL_TLS
      value: "true"
    - name: SQL_TLS_DISABLE_HOST_VERIFICATION
      value: "true"
    - name: SQL_TLS_ENABLED
      value: "true"
    - name: SQL_HOST_VERIFICATION
      value: "false"

I had the same temporal SSL encryption problem, which is resolved by that configuration. (Refer: https://github.com/airbytehq/airbyte/issues/11157#issuecomment-1927515718)

minyukim avatar Mar 24 '24 02:03 minyukim

@minyukim thanks, I already resolved that with enabling SSL TLS via the db instances and it worked. cheers!

mrfzy00 avatar Mar 24 '24 03:03 mrfzy00

At Airbyte, we seek to be clear about the project priorities and roadmap. This issue has not had any activity for 180 days, suggesting that it's not as critical as others. It's possible it has already been fixed. It is being marked as stale and will be closed in 20 days if there is no activity. To keep it open, please comment to let us know why it is important to you and if it is still reproducible on recent versions of Airbyte.

octavia-squidington-iii avatar Nov 27 '24 09:11 octavia-squidington-iii

Please keep it open

kev-datams avatar Nov 27 '24 20:11 kev-datams

At Airbyte, we seek to be clear about the project priorities and roadmap. This issue has not had any activity for 180 days, suggesting that it's not as critical as others. It's possible it has already been fixed. It is being marked as stale and will be closed in 20 days if there is no activity. To keep it open, please comment to let us know why it is important to you and if it is still reproducible on recent versions of Airbyte.

octavia-squidington-iii avatar May 28 '25 09:05 octavia-squidington-iii

Please keep it open

kev-datams avatar Jun 04 '25 18:06 kev-datams

At Airbyte, we seek to be clear about the project priorities and roadmap. This issue has not had any activity for 180 days, suggesting that it's not as critical as others. It's possible it has already been fixed. It is being marked as stale and will be closed in 20 days if there is no activity. To keep it open, please comment to let us know why it is important to you and if it is still reproducible on recent versions of Airbyte.

octavia-bot[bot] avatar Dec 02 '25 09:12 octavia-bot[bot]

Please keep it open

kev-datams avatar Dec 02 '25 17:12 kev-datams