Skip to content
This repository has been archived by the owner on Sep 26, 2021. It is now read-only.

OpenStack: update gophercloud package and add new auth options #4755

Closed
wants to merge 4 commits into from

Conversation

kayrus
Copy link

@kayrus kayrus commented Sep 24, 2019

Description

This PR adds new auth options into the openstack client and removes deprecated keystone V2 support.

Additionally it adds an ability to remove the VM, which was not created successfully. If it is not removed, new VMs cannot be created, because the resources quota can be reached.

Related issue(s)

Resolves #3774 and rancher/rancher#15796

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "gophercloud" git@github.com:kayrus/machine.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354383416
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Signed-off-by: kayrus <kay.diam@gmail.com>
Signed-off-by: kayrus <kay.diam@gmail.com>
Signed-off-by: kayrus <kay.diam@gmail.com>
Signed-off-by: kayrus <kay.diam@gmail.com>
@DJAyth
Copy link

DJAyth commented Oct 2, 2019

Could really use this as well, since it's causing me major pains. Can someone look at it?

@kayrus
Copy link
Author

kayrus commented Nov 27, 2019

Closing in favor of rancher#54
/cc DJAyth

@kayrus kayrus closed this Nov 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

possible bug in docker-machine openstack: calls v2 of API for tenants when given v3 auth?
3 participants