datacratic / StarCluster

StarCluster is a utility for creating and managing computing clusters hosted on Amazon's Elastic Compute Cloud (EC2).
http://star.mit.edu/cluster
GNU Lesser General Public License v3.0
37 stars 13 forks source link

Allow aws_access_key_id and aws_secret_access_key to be left blank #70

Closed cancan101 closed 7 years ago

cancan101 commented 7 years ago

boto will then employ it's own resolution mechanism allowing the user to specify in config file or env var.

FinchPowers commented 7 years ago

If the keys are not defined in the "boto way", is the error clear?

cancan101 commented 7 years ago

I think it will fall back to boto barfing with something like:

NoAuthHandlerFound: No handler was ready to authenticate. 1 handlers were checked. ['HmacAuthV1Handler'] Check your credentials
FinchPowers commented 7 years ago

Do you think it would be worth it then to handle the error to print a message telling the 2 ways StarCluster can support keys?

cancan101 commented 7 years ago

Seems overkill to me. Also not sure of all the points to catch this error and re-throw.

vasisht commented 7 years ago

@cancan101 Just out of curiosity, what's the use case for leaving the credentials blank?

cancan101 commented 7 years ago

So i don't have to put my creds into config file (and I can source control it)