Skip to content

FIX: Add line requiring tenant_id to docs #520

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pratikmallya
Copy link

I wasn't able to authenticate without specifying an auth_endpoint and tenant_id. This PR aims at providing that information to future users.

@briancurtin
Copy link
Contributor

Can you share some code to show what you were trying to do? Neither of those settings are needed to authenticate with public cloud.

@briancurtin
Copy link
Contributor

Just getting right to the point: does pyrax.set_setting("identity_type", "rackspace") not work for you?

@pratikmallya
Copy link
Author

@briancurtin: i tried again and I don't seem to need the auth_endpoint. But I do need to provide a tenant_id; otherwise the authentication fails

@pratikmallya pratikmallya force-pushed the add/docs-add-auth-endpoint branch from 3780c20 to aad2bbf Compare December 9, 2014 19:52
@pratikmallya pratikmallya changed the title FIX: Add line requiring auth endpoint and tenant_id to docs FIX: Add line requiring tenant_id to docs Dec 9, 2014
@briancurtin
Copy link
Contributor

You'll have to share some code, failure messages, and your environment then, because that setting has never been needed with public cloud.

@EdLeafe
Copy link
Contributor

EdLeafe commented Dec 9, 2014

If you try to log into the Rackspace public cloud as a user that was created for an account, rather than the main account username, you will need to specify tenant_id.

@sivel
Copy link
Contributor

sivel commented Dec 10, 2014

Looks like based on the discussion, that additional information needs to be added to the documentation stating that the tenant_id is required for RBAC users.

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

Successfully merging this pull request may close these issues.

4 participants