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.

Sign up for GitHub

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

Jump to bottom

Automate Ansible: host+input params for playbooks, input params for workflows #6530

Merged
merged 1 commit into from
Dec 12, 2019
Merged

Automate Ansible: host+input params for playbooks, input params for workflows #6530

merged 1 commit into from
Dec 12, 2019

Conversation

martinpovolny
Copy link
Member

@martinpovolny martinpovolny commented Dec 11, 2019 •
edited
Loading

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

Adding:

  • Inputs and Hosts for Ansible Job Template
  • Inputs for Ansible Workflow Template

ansible-meth-2019-12-11_15-52

@martinpovolny martinpovolny changed the title [WIP] Automate Ansible: host+input params for playbooks, input params for playbooks [WIP] Automate Ansible: host+input params for playbooks, input params for workflows Dec 11, 2019
@martinpovolny
Automate Ansible: host+inputs for playbooks, inputs for wrkflws.
b170556 
 * add host and inputs for playbooks,
 * add inputs for workflows,
 * fix label margin.
@martinpovolny martinpovolny force-pushed the ansible_automate_methods_more branch from ebb1d24 to b170556 Compare December 11, 2019 14:55
@miq-bot miq-bot added the wip label Dec 11, 2019
@miq-bot
Copy link
Member

miq-bot commented Dec 11, 2019

Checked commit martinpovolny@b170556 with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 1 offense detected

app/views/layouts/angular/_ansible_form_options_angular.html.haml

  • ⚠️ - Line 329 - Line is too long. [201/160]

@martinpovolny martinpovolny changed the title [WIP] Automate Ansible: host+input params for playbooks, input params for workflows Automate Ansible: host+input params for playbooks, input params for workflows Dec 11, 2019
@martinpovolny martinpovolny removed the wip label Dec 11, 2019
@mzazrivec mzazrivec self-assigned this Dec 12, 2019
@mzazrivec mzazrivec added this to the Sprint 127 Ending Jan 6, 2020 milestone Dec 12, 2019
@mzazrivec mzazrivec merged commit 992c497 into ManageIQ:master Dec 12, 2019
@simaishi
Copy link
Contributor

@martinpovolny Can this be ivanchuk/yes?

@simaishi
Copy link
Contributor

@martinpovolny ^ ping

@martinpovolny
Copy link
Member Author

martinpovolny commented Jan 24, 2020 •
edited
Loading

@simaishi : yes, it can, sorry for the delay

simaishi pushed a commit that referenced this pull request Jan 24, 2020
@mzazrivec @simaishi
Merge pull request #6530 from martinpovolny/ansible_automate_methods_…
d443207 
…more

Automate Ansible: host+input params for playbooks, input params for workflows

(cherry picked from commit 992c497)

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

Ivanchuk backport details:

$ git log -1
commit d44320707dfeb57582cbc814d91d7fc68cdd42b2
Author: Milan Zázrivec <[email protected]>
Date:   Thu Dec 12 09:02:04 2019 +0100

    Merge pull request #6530 from martinpovolny/ansible_automate_methods_more

    Automate Ansible: host+input params for playbooks, input params for workflows

    (cherry picked from commit 992c497d55883aeed4b6eb6ff7971a9549356580)

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mzazrivec mzazrivec mzazrivec approved these changes

@h-kataria h-kataria Awaiting requested review from h-kataria

@tinaafitz tinaafitz Awaiting requested review from tinaafitz

Assignees

@mzazrivec mzazrivec

Labels
automation/automate changelog/yes enhancement ivanchuk/backported
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@martinpovolny @miq-bot @simaishi @mzazrivec