Helm INTERNAL_API_URL fix

[a-monteiro]

What

It fixes the Airbyte API Server helm chart.

How

We added an extra env variable for the Airbyte internal API URL prefixed with http://.

Recommended reading order

Can this PR be safely reverted / rolled back?

If you know that your PR is backwards-compatible and can be simply reverted or rolled back, check the YES box.

Otherwise if your PR has a breaking change, like a database migration for example, check the NO box.

If unsure, leave it blank.

• YES 💚
• NO ❌

🚨 User Impact 🚨

Are there any breaking changes? What is the end result perceived by the user? If yes, please merge this PR with the 🚨🚨 emoji so changelog authors can further highlight this if needed.

[saisona]

Could that be possible to merge that PR ASAP please !

[Anton-Shutik]

Any updates on this one ?

[Santhin]

For workaround, you need to add to the values

  airbyte-api-server:
    extraEnv:
      - name: INTERNAL_API_HOST
        valueFrom:
          configMapKeyRef:
            name: airbyte-additional-envs
            key: INTERNAL_API_URL

And create cm

apiVersion: v1
data:
  INTERNAL_API_URL: http://airbyte-airbyte-server-svc:8001
kind: ConfigMap
metadata:
  name: airbyte-additional-envs

And restart deployment to inject new env_vars and cm

[stevenmurphy12]

For workaround, you need to add to the values

  airbyte-api-server:
    extraEnv:
      - name: INTERNAL_API_HOST
        valueFrom:
          configMapKeyRef:
            name: airbyte-additional-envs
            key: INTERNAL_API_URL

And create cm

apiVersion: v1
data:
  INTERNAL_API_URL: http://airbyte-airbyte-server-svc:8001
kind: ConfigMap
metadata:
  name: airbyte-additional-envs

And restart deployment to inject new env_vars and cm

This worked for me, thank you.

[ldmoose]

This appears to be fixed as of helm version 0.50.10 with this commit: b28dc9c

[a-monteiro]

This appears to be fixed as of helm version 0.50.10 with this commit: b28dc9c

Indeed! Closing this.