Try to use Fog credentials if no config file is provided #20

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@dvestal
dvestal commented Mar 10, 2013

It seemed it would be nice to not have to provide, or specify, the config/ec2.yml file since that information is often going to be duplicated in the ~/.fog file anyway. If CapifyEc2 fails to load the yaml config then it will try to use the Fog credentials.

I am open to suggestions for what the default region should be. Since Fog defaults to us-east-1 that's what I used.

The pull request also contains an update to the current version of Fog. I haven't tested it with the older version, but I believe that it should work since I'm only using Fog.credentials and not any of the actual api.

@Rylon
Member
Rylon commented Mar 28, 2013

Thanks for this, I'll look at merging it into the next release, once we're finished testing the rolling deployment feature.

@Rylon
Member
Rylon commented May 16, 2013

Hey there,

Just looking at this now, could I ask what the reason is for increasing the Fog dependency version?

Thanks,
Ryan

@dvestal
dvestal commented May 16, 2013

The dependency version increase is not necessary. I did it locally, but since there is no functional reason I really shouldn't have included it in this pull request. If you'd like I can redact that part of the request.

@Rylon Rylon closed this in f21dde5 May 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment