-
Notifications
You must be signed in to change notification settings - Fork 129
support agent region #713
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
support agent region #713
Conversation
|
A select would be nice if the property is not provided! With so much remote work I don't think it's safe to assume closest region to host machine. Can regions be listed by the client or does that API need adding? |
i think we can use the 'settings' api offered by cloud agents to return the currently enabled regions |
| } else { | ||
| // we shouldn't ever get here, but if we do, just default to us-east | ||
| logger.Debugw("no available regions found, defaulting to us-east. please contact LiveKit support if this is unexpected.") | ||
| region = "us-east" |
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.
Shouldn't default region be empty?
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.
hmm i wasn't sure here. if it's empty then whichever region picks it up based on dns of the api call would deploy it. not sure if that is better or to use us-east
rektdeckard
left a comment
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.
Nice! Can you add the flag to initAgent too? Just the flag definition needs to be there, we forward everything to createAgent.
added this to initAgent flags, hopefully this is what you need? 47fba4d |
|
@real-danm yep, thank you! |
added a select themed with the standard form to select from available regions.