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.

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

Provision instance in specific AWS zone #459

Closed
martbert opened this issue Mar 23, 2021 · 3 comments · Fixed by #831
Closed

Provision instance in specific AWS zone #459

martbert opened this issue Mar 23, 2021 · 3 comments · Fixed by #831
Labels
cml-runner Subcommand duplicate Déjà lu

Comments

@martbert
Copy link

Hi,

CML looks very promising for allowing easy MLOps. I've been playing around with it and stumbled upon a small issue. Say I want to run a job on a p3.2xlarge (AWS) instance on AWS in the us-east-1 region. Turns out those instances are not available in all zones. Hence when the runner tries to provision the said type of instance in us-east-1c, it fails. Would it be possible to consider the support of zones?

Cheers!

@DavidGOrtega
Copy link
Contributor

👋 @martbert
Yes we are actually contemplating it and actually there is another ticket regarding this.
Will be resolved soon. I will ping you.

@DavidGOrtega DavidGOrtega added cml-runner Subcommand duplicate Déjà lu labels Apr 13, 2021
@0x2b3bfa0
Copy link
Member

0x2b3bfa0 commented Dec 1, 2021

@DavidGOrtega, can you please link to the issue this is a duplicate of?

What if we allow specifying both regions and availability zones using the region attribute in the provider? For example, cml runner --cloud-region=us-east-1c would always the us-east-1c availability zone.

When a native region string ends with a lowercase letter, we can extract both values and use them accordingly. Adding a availability-zone filter condition here should do the trick. The same goes for the task resource.

@dacbd
Copy link
Contributor

dacbd commented Dec 1, 2021

I believe that #814 and #795 are related since the fix would involve the same chunk of code, I intended to get a PR for in around the weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cml-runner Subcommand duplicate Déjà lu
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants