fix(chart): use http scheme for INTERNAL_API_HOST

[justenwalker]

What

The current chart doesn't use a scheme for INTERNAL_API_HOST causing the api-server fail to reach the internal config API.

How

This fix adds an explicit http:// prefix to the value of INTERNAL_API_HOST in the configmap.

Recommended reading order

1. charts/airbyte/templates/env-configmap.yaml

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 🚨

None

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
2.6% Duplication

[justenwalker]

Ok, so this doesn't actually work as-is. The reason is the worker expects this value in the form host:port so when the http:// prefix is added, that client setup breaks:

2023-12-07 20:26:16 ERROR i.m.r.Micronaut(handleStartupException):338 - Error starting Micronaut server: Error instantiating bean of type  [io.airbyte.api.client.AirbyteApiClient]

Message: For input string: "//airbyte-airbyte-server-svc"
Path Taken: new ApplicationInitializer() --> ApplicationInitializer.checkConnectionActivities --> List.checkConnectionActivities([CheckConnectionActivity checkConnectionActivity]) --> new CheckConnectionActivityImpl(WorkerConfigsProvider workerConfigsProvider,ProcessFactory processFactory,SecretsR
io.micronaut.context.exceptions.BeanInstantiationException: Error instantiating bean of type  [io.airbyte.api.client.AirbyteApiClient]

Message: For input string: "//airbyte-airbyte-server-svc"
Path Taken: new ApplicationInitializer() --> ApplicationInitializer.checkConnectionActivities --> List.checkConnectionActivities([CheckConnectionActivity checkConnectionActivity]) --> new CheckConnectionActivityImpl(WorkerConfigsProvider workerConfigsProvider,ProcessFactory processFactory,SecretsR
    at io.micronaut.context.DefaultBeanContext.resolveByBeanFactory(DefaultBeanContext.java:2367) ~[micronaut-inject-3.10.1.jar:3.10.1]
                                                                             

[terencecho]

this issue should be fixed by this commit: b28dc9c