-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Tweak .env in kube/overlays to connect to external db #4403
Conversation
kube/overlays/dev/.env
Outdated
DATA_DOCKER_MOUNT=airbyte_data | ||
DB_DOCKER_MOUNT=airbyte_db | ||
|
||
# Internal Temporal.io deployment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Internal Temporal.io deployment | |
# Temporal.io worker configuration |
kube/overlays/stable/.env
Outdated
DATA_DOCKER_MOUNT=airbyte_data | ||
DB_DOCKER_MOUNT=airbyte_db | ||
|
||
# Internal Temporal.io deployment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Internal Temporal.io deployment | |
# Temporal.io worker configuration |
kube/resources/scheduler.yaml
Outdated
@@ -51,7 +61,7 @@ spec: | |||
- name: WAIT_BEFORE_HOSTS | |||
value: "5" | |||
- name: WAIT_HOSTS | |||
value: airbyte-db-svc:5432 | |||
value: $(DATABASE_HOST):$(DATABASE_PORT) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this works since it never is resolved to the correct value?
→ kubectl describe pod airbyte-scheduler-7764b66c8c-5n6bm | grep WAIT_HOSTS\:
WAIT_HOSTS: $(DATABASE_HOST):$(DATABASE_PORT)
Eventually we should remove wait entirely since we're waiting within code with finer control. For now we can just hard code the host and port again...
a1fe858
to
9f85001
Compare
What
Mirror changes from #4211 to kube
.env
tooHow
Describe the solution