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

Enforce CLI option dependencies in a user-friendly way #5

Closed
nchammas opened this issue Oct 5, 2015 · 0 comments
Closed

Enforce CLI option dependencies in a user-friendly way #5

nchammas opened this issue Oct 5, 2015 · 0 comments
Milestone

Comments

@nchammas
Copy link
Owner

nchammas commented Oct 5, 2015

If you are launching clusters into EC2 (i.e. when --provider is set to ec2), then --ec2-region and --ec2-ami are required.

At the moment we don't enforce this dependency directly in the CLI, partly because Click doesn't directly support option dependencies like this. So if you forget to specify --ec2-ami, it gets caught somewhere in the bowels of Flintrock as opposed to at the front gate, and some ugly error gets barfed out.

This is a user experience problem, and it will get worse when we add support for other providers like GCE since there will be more opportunities for users to forget to specify required options (hah!).

We should fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant