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

OrderAnsiblePlaybook method should collect machine credential. #118

Merged

Conversation

lfu
Copy link
Member

@lfu lfu commented May 22, 2017

OrderAnsiblePlaybook method should collect machine credential
and pass it to ServiceTemplae's provision_request.

https://bugzilla.redhat.com/show_bug.cgi?id=1452557

@miq-bot assign @gmcculloug
@miq-bot add_label bug, fine/yes, services

OrderAnsiblePlaybook method should collect machine credential
and pass it to ServiceTemplae's provision_request.

https://bugzilla.redhat.com/show_bug.cgi?id=1452557
@miq-bot
Copy link
Member

miq-bot commented May 22, 2017

Checked commit lfu@bb7af8b with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

@gmcculloug gmcculloug merged commit eae08b4 into ManageIQ:master May 22, 2017
@gmcculloug gmcculloug added this to the Sprint 61 Ending May 22, 2017 milestone May 22, 2017
simaishi pushed a commit that referenced this pull request Jun 9, 2017
OrderAnsiblePlaybook method should collect machine credential.
(cherry picked from commit eae08b4)

https://bugzilla.redhat.com/show_bug.cgi?id=1460293
@simaishi
Copy link
Contributor

simaishi commented Jun 9, 2017

Fine backport details:

$ git log -1
commit 6c87b04e9633d5a51845a0ebeb52dbc70b276848
Author: Greg McCullough <[email protected]>
Date:   Mon May 22 16:07:13 2017 -0400

    Merge pull request #118 from lfu/credential_in_service_dialog_1452557
    
    OrderAnsiblePlaybook method should collect machine credential.
    (cherry picked from commit eae08b4005f3215439fda376dda7188a4fc3c23d)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1460293

@lfu lfu deleted the credential_in_service_dialog_1452557 branch September 29, 2018 14:33
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