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

Add IAD to set of regions and that it is the default region. #316

Merged
merged 2 commits into from
Mar 18, 2014

Conversation

nedrocks
Copy link
Contributor

No description provided.

@sivel
Copy link
Contributor

sivel commented Mar 14, 2014

This is not necessarily true. The default region is specified by the API response (resp["access"]["user"]["RAX-AUTH:defaultRegion"])

For me it is DFW.

@nedrocks
Copy link
Contributor Author

That makes much more sense.

I don't know what this should say to be less confusing but I wasted about an hour trying to figure this out earlier assuming that DFW was correct.

@EdLeafe
Copy link
Contributor

EdLeafe commented Mar 17, 2014

There are also HKG and SYD datacenters, and who know what may come in the future. I think it would be best to list a few example DCs, and include a link to http://www.rackspace.com/about/datacenters/ for current complete listing.

@nedrocks
Copy link
Contributor Author

Thanks for the review!

@sivel I've changed the default to explain that it's dependent on account settings.

@EdLeafe I've added a link to datacenters in the documentation.

@EdLeafe
Copy link
Contributor

EdLeafe commented Mar 18, 2014

Hmmm... now that I'm re-reading the whole thing, there are a few other problems. I'll merge this and then fix them myself.

EdLeafe added a commit that referenced this pull request Mar 18, 2014
Update regions information in the docs.
@EdLeafe EdLeafe merged commit b573386 into pycontribs:working Mar 18, 2014
@nedrocks nedrocks deleted the default-region-fix branch March 18, 2014 18:45
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

Successfully merging this pull request may close these issues.

3 participants