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.

Sign up for GitHub

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

Jump to bottom

chore(helm): add configuration to override model-backend host #462

Merged
merged 1 commit into from
Mar 31, 2024
Merged

chore(helm): add configuration to override model-backend host #462

merged 1 commit into from
Mar 31, 2024

Conversation

donch1989
Copy link
Member

Because

  • In the multi-region architecture, the Instill Model connector needs to connect to the model-backend through an external network instead of an internal network, we need to have a configuration for this.

This commit:

  • Adds configuration to override the model-backend host.

@linear Linear
Copy link

linear bot commented Mar 30, 2024

INS-4020 fix Instill Model connector can't work in multi-region

@donch1989 donch1989 merged commit b42e72f into main Mar 31, 2024
42 of 44 checks passed
@donch1989 donch1989 deleted the huitang/ins-4020 branch March 31, 2024 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jvallesm jvallesm Awaiting requested review from jvallesm jvallesm is a code owner

@heiruwu heiruwu Awaiting requested review from heiruwu heiruwu is a code owner

@Sarthak-instill Sarthak-instill Awaiting requested review from Sarthak-instill Sarthak-instill is a code owner

Assignees
No one assigned
Labels
core
Projects
No open projects
🔮 Instill Core
Status: 👋 Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@donch1989 @droplet-bot