-
Notifications
You must be signed in to change notification settings - Fork 900
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
For Ansible Tower service include extra_vars needed to connect to our REST API #18005
Labels
Comments
@bzwei @tinaafitz |
Briefly discussed with @lfu and she is going to look into solving this. |
lfu
added a commit
to lfu/manageiq-automation_engine
that referenced
this issue
Oct 4, 2018
This was referenced Oct 4, 2018
Merged
lfu
added a commit
to lfu/manageiq-automation_engine
that referenced
this issue
Oct 4, 2018
lfu
added a commit
to lfu/manageiq
that referenced
this issue
Oct 4, 2018
lfu
added a commit
to lfu/manageiq
that referenced
this issue
Oct 4, 2018
Refactor extra_vars support for ServiceAnsiblePlaybook. ManageIQ#18005
lfu
added a commit
to lfu/manageiq
that referenced
this issue
Oct 4, 2018
lfu
added a commit
to lfu/manageiq
that referenced
this issue
Oct 10, 2018
Refactor extra_vars support for ServiceAnsiblePlaybook. ManageIQ#18005
lfu
added a commit
to lfu/manageiq
that referenced
this issue
Oct 10, 2018
lfu
added a commit
to lfu/manageiq-automation_engine
that referenced
this issue
Oct 10, 2018
lfu
added a commit
to lfu/manageiq
that referenced
this issue
Oct 15, 2018
Refactor extra_vars support for ServiceAnsiblePlaybook. ManageIQ#18005
lfu
added a commit
to lfu/manageiq
that referenced
this issue
Oct 15, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Our Ansible Tower Service doesn't pass in the extra vars necessary to connect to the REST API end point.
manageiq/app/models/service_ansible_tower.rb
Line 12 in 52f4215
These extra vars are under the key manageiq
manageiq
The request_task allows us to access a task based on the RBAC request. A Task cannot be directly accessed it can only be accessed via the request
For Ansible Playbook service we pass in extra vars that allows us to connect to REST API
manageiq/app/models/service_ansible_playbook.rb
Line 64 in 52f4215
Maybe we should have a mixin that allows both the tower and ansible playbook service to share the same code.
When run as an Automate Method we also pass in extra vars
https://github.com/ManageIQ/manageiq-automation_engine/blob/7d4752980d668d626177cabe63206381edd7f4aa/lib/miq_automation_engine/engine/miq_ae_engine/miq_ae_playbook_method.rb#L112
All these 3 pieces of code that interact with either the tower or embedded ansible need to pass in extra vars.
The text was updated successfully, but these errors were encountered: