Skip to content

Support for v3#179

Merged
jjasghar merged 2 commits intotest-kitchen:masterfrom
andybrucenet:master
Mar 28, 2018
Merged

Support for v3#179
jjasghar merged 2 commits intotest-kitchen:masterfrom
andybrucenet:master

Conversation

@andybrucenet
Copy link
Copy Markdown

Hi - I modified the plugin to work with identity v3 where the domain_id can be passed in and "tenant" is renamed to "project". Seems to work just fine; I'm provisioning new instances on OpenStack version Pike no problems at all.

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 20, 2017

Coverage Status

Coverage increased (+0.002%) to 99.303% when pulling f5c9ee0 on andybrucenet:master into c1398fb on test-kitchen:master.

2 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.002%) to 99.303% when pulling f5c9ee0 on andybrucenet:master into c1398fb on test-kitchen:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.002%) to 99.303% when pulling f5c9ee0 on andybrucenet:master into c1398fb on test-kitchen:master.

@ryanfaerman
Copy link
Copy Markdown

Is there anything preventing this from being merged? I'm basically dead in the water when it comes to using this plugin without something like what this PR provides.

@jjasghar
Copy link
Copy Markdown
Contributor

I need to build a devstack instance to test this before merging. I will attempt to do this ASAP.

@VagabondAzulien
Copy link
Copy Markdown

Any update?

@jjasghar
Copy link
Copy Markdown
Contributor

I haven’t had cycled to test this. If someone else has and willing to state here that is works, I’ll merge and release.

@dewdo
Copy link
Copy Markdown

dewdo commented Mar 28, 2018

This PR works pretty well with identity v3.

@jjasghar
Copy link
Copy Markdown
Contributor

Awesome! I'll do the release today. thanks for the confirmation @dewdo

@jjasghar jjasghar merged commit a5b88fb into test-kitchen:master Mar 28, 2018
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.

6 participants