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

Add Lenovo infra provider authentication args for queue task #2549

Merged

Conversation

lucashsilva
Copy link
Contributor

This PR adds Lenovo provider authentication args for the authentication method called on the queue.
This is a follow up to #2243 which changed the authentication method to use the queue.

@lucashsilva lucashsilva changed the title Add Lenovo authentication args for queue task Add Lenovo infra provider authentication args for queue task Oct 30, 2017
@lucashsilva lucashsilva force-pushed the add_lenovo_provider_authentication_args branch from d386949 to ff2a9f8 Compare October 30, 2017 12:49
@lucashsilva
Copy link
Contributor Author

@h-kataria can you please take a look at this?

@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2017

This pull request is not mergeable. Please rebase and repush.

@lucashsilva lucashsilva force-pushed the add_lenovo_provider_authentication_args branch from ff2a9f8 to 9380539 Compare October 30, 2017 15:53
@lucashsilva lucashsilva force-pushed the add_lenovo_provider_authentication_args branch from 9380539 to bd9b7fc Compare October 30, 2017 18:51
@lucashsilva
Copy link
Contributor Author

@miq-bot add_label blocker

@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2017

Checked commit lucashsilva@bd9b7fc with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

@chessbyte
Copy link
Member

Unrelated test failure. Merging.

@chessbyte chessbyte merged commit 0ec9a48 into ManageIQ:master Oct 31, 2017
@chessbyte chessbyte added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 31, 2017
@chessbyte chessbyte removed the blocker label Oct 31, 2017
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.

3 participants