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

openstack: load all available authentication plugins #98

Closed

Conversation

alvarolopez
Copy link
Contributor

Instead of only loading the v3password authentication plugin, lets load
all the available keystoneauth plugins so that the user is able to use
their preferred authentication method.

Fixes #86

@egi-bot
Copy link

egi-bot commented Aug 17, 2018

Can one of the admins verify this patch?

1 similar comment
@egi-bot
Copy link

egi-bot commented Aug 17, 2018

Can one of the admins verify this patch?

Copy link
Member

@gwarf gwarf left a comment

Choose a reason for hiding this comment

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

@alvarolopez can you please rebase and fix the problem reported by tox/travis, could you also adjust the line length problems reported by tox and codeclimate?

@gwarf gwarf added this to the 1.0 milestone Aug 30, 2018
Instead of only loading the v3password authentication plugin, lets load
all the available keystoneauth plugins so that the user is able to use
their preferred authentication method.

Fixes EGI-Federation#86
@coveralls
Copy link

Pull Request Test Coverage Report for Build 266

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at ?%

Totals Coverage Status
Change from base Build 257: 0.0%
Covered Lines:
Relevant Lines: 0

💛 - Coveralls

@gwarf
Copy link
Member

gwarf commented Sep 5, 2018

So those merges have been pushed to the GLUE 2.1 PR: #99

@gwarf gwarf closed this Sep 5, 2018
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