Description of problem: When configuring a Migration Plan with pre or post-migration playbook, the credential set on the Ansible Playbook service template is overriden by an empty string. This leads to not passing the credential to Ansible Runner and to failing the playbook execution. How reproducible: 100% Steps to Reproduce: 1. Create an Ansible Playbook service template with valid credential 2. Create a migration plan using the service template as pre-migration playbook 3. Start the migration plan Actual results: The playbook fails with host unreachable Expected results: The playbook succeeds Additional info: When watching the Ansible Runner execution, we can see that the file /tmp/ansible-runnner*/env/passwords is not created.
https://github.com/ManageIQ/manageiq-content/pull/559
https://github.com/ManageIQ/manageiq-automation_engine/pull/348
New commit detected on ManageIQ/manageiq-automation_engine/ivanchuk: https://github.com/ManageIQ/manageiq-automation_engine/commit/c85a21aeae52c1c96bfffc0e103d8958e253ad57 commit c85a21aeae52c1c96bfffc0e103d8958e253ad57 Author: Madhu Kanoor <mkanoor> AuthorDate: Tue Aug 13 09:20:24 2019 -0400 Commit: Madhu Kanoor <mkanoor> CommitDate: Tue Aug 13 09:20:24 2019 -0400 Merge pull request #348 from fdupont-redhat/override_equality_comparison_method Override equality comparison method for MiqAeServiceModelBase (cherry picked from commit 0c498633308578ba84bfb52328234db1ece7c659) https://bugzilla.redhat.com/show_bug.cgi?id=1739247 lib/miq_automation_engine/engine/miq_ae_method_service/miq_ae_service_model_base.rb | 4 + 1 file changed, 4 insertions(+)
New commit detected on ManageIQ/manageiq-content/ivanchuk: https://github.com/ManageIQ/manageiq-content/commit/70c5f459f954f96277920fd073ea43901b303f80 commit 70c5f459f954f96277920fd073ea43901b303f80 Author: tina <tfitzger> AuthorDate: Tue Aug 13 13:18:34 2019 -0400 Commit: tina <tfitzger> CommitDate: Tue Aug 13 13:18:34 2019 -0400 Merge pull request #559 from fdupont-redhat/v2v_fix_launch_ansible_playbook_credentials [Transformation][AnsiblePlaybook] Send service_template credential id as part of service dialog (cherry picked from commit a2fb8c8192327d7a60697fc829b28777f9aa7ca8) https://bugzilla.redhat.com/show_bug.cgi?id=1739247 content/automate/ManageIQ/Transformation/Ansible.class/__methods__/launchplaybookasaservice.rb | 5 +- spec/content/automate/ManageIQ/Transformation/Ansible.class/__methods__/launchplaybookasaservice_spec.rb | 131 + 2 files changed, 135 insertions(+), 1 deletion(-)
Playbook worked so fixed in 5.11.0.21.
It would be great to backport it to CFME 5.10
ack for back port into 5.10.z.