search

coiled / feedback

A place to provide Coiled feedback
14 stars 3 forks source link

Validate AWS region on cluster creation #121

Closed gjoseph92 closed 3 years ago

gjoseph92 commented 3 years ago

By not reading the docs carefully enough, I missed the fact that us-west-2 is not yet a supported region. However, ValueError: Unable to create cluster: No VPC subnets available was not a very clear error to help figure that out. This slack thread answered the question, but since the list of supported regions is pretty well-known, it would be nice to give users a more specific error message, including the list of supported regions they could use instead.

FabioRosado commented 3 years ago

Hello @gjoseph92, thank you for your feedback!

Alex, from our engineering team is currently working on this issue. Soon, we will be able to throw a better error since we will run some validation on the region that users specify and return a more informative error message.

I hope this hasn't caused any major issues for you when using Coiled. Please let me know if you need help with anything or if you have any feedback that could help us improve the user experience 😄

jrbourbeau commented 3 years ago

Thanks for testing things out @gjoseph92! Good to see you here : )

dantheman39 commented 3 years ago

In fact, @gjoseph92 we just deployed this, so now you should get an error saying Unable to create cluster: Unsupported region. We currently support these AWS regions: ['us-east-1', 'us-east-2', 'us-west-1', 'eu-central-1', 'eu-west-2']

Thanks for the feedback!

gjoseph92 commented 3 years ago

Yup, got it! Unsupported region. We currently support these AWS regions: ['us-east-1', 'us-east-2', 'us-west-1', 'eu-central-1', 'eu-west-2']

That's great, thanks for the quick turnaround.

mrocklin commented 3 years ago

I'm curious, what stops us from supporting all regions? I would expect us to be able to create these resources on the fly.

On Mon, Mar 1, 2021 at 5:24 PM Gabe Joseph notifications@github.com wrote:

Yup, got it! Unsupported region. We currently support these AWS regions: ['us-east-1', 'us-east-2', 'us-west-1', 'eu-central-1', 'eu-west-2']

That's great, thanks for the quick turnaround.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/coiled/feedback/issues/121#issuecomment-788389239, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACKZTHXZSL57WPW5QTTH73TBQO3XANCNFSM4YNMRPIQ .