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

Upgrade keystone API v2 to v3 #168

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

davidbullado
Copy link

What I did

Be able to connect to keystone API v3.

How I did it

Added the missing parameter "os_user_domain_name".
Modified auth_v3.go to select the parametrized region.

How to verify it

Add this parameter to your configuration file:
OS_USER_DOMAIN_NAME="default"
And change this parameter to v3:
OS_AUTH_URL=https://auth.cloud.ovh.net/v3

Description for changelog

Support keystone API v3

@rainfay
Copy link

rainfay commented Feb 16, 2020

It works.

@axi
Copy link

axi commented Feb 17, 2020

Anyone able to merge this at ovh ?

Copy link

@rainfay rainfay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ran tests, on my system, V3 api works

Copy link

@IvanVCH IvanVCH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for me it works, please merge and make the final 0.9.2 deb package

@IvanVCH
Copy link

IvanVCH commented Feb 25, 2020

please upload a deb package amd64 into http://uploadfiles.io/

Copy link

@IvanVCH IvanVCH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good version

@emaktech
Copy link

I have tried contacting someone at OVH support to see if they can escalate to somebody who can manage this project and accept the PR. Hopefully someone from OVH is able to accept this before the March deadline!

@axi
Copy link

axi commented Feb 26, 2020

I keep receiving some mail warnings from Ovh saying that keystone v3 will be deactivated on 24 march 2020... I opened a support ticket

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.

7 participants