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

[WIP] provider/openstack - Add support for identity v3 #7041

Conversation

sbourlon
Copy link

@sbourlon sbourlon commented Jun 7, 2016

I just sent a PR to the gophercloud repository in order to improve the identity v3 support: rackspace/gophercloud#593. While waiting for them to merge it, here is the code on the Terraform side. I am actually already using it for my Stackato Cloud Foundry cluster deployment automation tool (https://github.com/hpcloud/stackato-cluster-tool).

This PR adds the following keys:

  • user_domain_name and user_domain_id identifying the domain where the user lives
  • project_domain_name and project_domain_id for project scoping
  • default_domain for shared domain when the domain scope and user scope are the same
  • project_name and project_id which is the new naming for tenant since OpenStack cli 1.7.0

Before merging this PR, we would need to bump the Godeps gophercloud dependency after they merge rackspace/gophercloud#593.

Please, let me know if I am missing something or if this PR needs to be improved.

Add:
- default_domain in case the domain scope and user domain are the same
- user_domain_name and user_domain_id identifying the domain where the user lives
- project_domain_name and project_domain_id for project scoping
- project_name and project_id which is the new naming for tenant since OpenStack cli 1.7.0
@jtopjian
Copy link
Contributor

jtopjian commented Jun 7, 2016

@sbourlon Cool! Thank you for this. I've subscribed to rackspace/gophercloud#593 and will also keep an eye on it.

As an aside, I'm awaiting rackspace/gophercloud#528 which will help with #6458.

@sbourlon
Copy link
Author

sbourlon commented Jun 7, 2016

Thanks @jtopjian! The repo https://github.com/rackspace/gophercloud is moving to a new one https://github.com/gophercloud/gophercloud. I will have to test and move the gophercloud PR to this new repo.

@jtopjian
Copy link
Contributor

jtopjian commented Jun 8, 2016

👍

I followed along the conversation today. The move go gophercloud/gophercloud was news to me, too. The Terraform OpenStack provider in general will have to move over to it... one step at a time, I guess 😄

@sbourlon
Copy link
Author

sbourlon commented Jun 10, 2016

If needed, I uploaded the build of this PR to the latest release of my cluster tool: https://github.com/hpcloud/stackato-cluster-tool/releases/tag/v0.9.0

@dpmex4527
Copy link

Looking forward to seeing this PR go through :)

@sbourlon
Copy link
Author

sbourlon commented Jul 11, 2016

@dpmex4527 Me too! However, we would need to have Terraform using the new gophercloud (rackspace/gophercloud#592), then I will migrate rackspace/gophercloud#593 to it.

@dpmex4527
Copy link

@sbourlon would you happen to have a gist or some instructions on how to build this PR for me to test? Would like to validate as we currently use Openstack V3 at work and would love to help test it out.

@dpmex4527
Copy link

@sbourlon nvm just saw in message thread link to the build with V3 supported :) Thanks!

@dpmex4527
Copy link

@sbourlon looks like the new gophercloud repo is up and receiving PR's now (I see a few PR's and issues out there now). Hoping we can see V3 support in terraform soon :)

Also, someone already submitted a V3 related PR. Similar to yours? gophercloud/gophercloud/pull/13

@jtopjian
Copy link
Contributor

Just a heads up: The Terraform OpenStack provider won't immediately be switching to the new gophercloud/gophercloud library. The new library has a good amount of changes from the original library.

I'm in the process of reviewing the code now, basically going through each acceptance test as a way of identifying breaking changes. I'm pretty sure any public-facing changes will just be a few call rewrites, but will take some time to confirm.

I do intend to switch over as soon as possible, though. :)

@fatmcgav
Copy link
Contributor

@jtopjian What's the latest on this change and the switch to gophercloud/gophercloud?

As I've just hit the same issue I believe where I can't scope Terraform auth to a specific project name :(

@jtopjian
Copy link
Contributor

@fatmcgav Check out #7926 for the status of moving to gophercloud/gophercloud. :)

@fatmcgav
Copy link
Contributor

@jtopjian Cool, cheers... I'll have a look...

@ghost
Copy link

ghost commented Apr 20, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants