I tested this on the 5.10.0.20 build from sprout and recreated the issue. I am using the default credentials. Billy
Tower 3.3 was added in the 5.10.18 build. Builds 5.10.18 and 5.10.19 had issues where embedded Ansible wouldn't start.
Works in 5.10.0.17
James, I just built a sprout appliance (10.8.198.202) with the latest 5.10.0.21 and still having the credential problem. You can test the credentials there. Feel free to change anything you like. Billy
https://github.com/ManageIQ/manageiq-providers-ansible_tower/pull/134
Along with the PR from James we need to switch the callers to use the new native_ref property instead of manager_ref. Moving the BZ to Lucy to complete this part of the task.
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/e25b6b165860750d6b0ce6b5edf6f882de70b609 commit e25b6b165860750d6b0ce6b5edf6f882de70b609 Author: James Wong <jwong> AuthorDate: Wed Oct 31 15:30:00 2018 -0400 Commit: James Wong <jwong> CommitDate: Wed Oct 31 15:30:00 2018 -0400 Tower 3.3 fix https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/authentication.rb | 5 + 1 file changed, 5 insertions(+)
New commit detected on ManageIQ/manageiq/hammer: https://github.com/ManageIQ/manageiq/commit/e565704b885cb59be1f4dbfbdce94c451b567961 commit e565704b885cb59be1f4dbfbdce94c451b567961 Author: Adam Grare <agrare> AuthorDate: Thu Nov 1 15:54:53 2018 -0400 Commit: Adam Grare <agrare> CommitDate: Thu Nov 1 15:54:53 2018 -0400 Merge pull request #18154 from jameswnl/tower33 Credential.manager_ref needs to be an integer for Tower 3.3 (cherry picked from commit 7d7b570d3d54846c2f923758ca0107c09ac41daf) https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/authentication.rb | 5 + 1 file changed, 5 insertions(+)
https://github.com/ManageIQ/manageiq/pull/18155
New commits detected on ManageIQ/manageiq-providers-ansible_tower/hammer: https://github.com/ManageIQ/manageiq-providers-ansible_tower/commit/a45325609bd2294ae3376d89f98154d757cd23e3 commit a45325609bd2294ae3376d89f98154d757cd23e3 Author: Adam Grare <agrare> AuthorDate: Thu Nov 1 15:55:08 2018 -0400 Commit: Adam Grare <agrare> CommitDate: Thu Nov 1 15:55:08 2018 -0400 Merge pull request #134 from jameswnl/tower33 Tower 3.3 fix: Credential.manager_ref need to be an integer for Tower 3.3 (cherry picked from commit 73ba300544826bf50c8ca635c17a4b625349aeef) https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/manageiq/providers/ansible_tower/shared/automation_manager/credential.rb | 6 +- spec/support/ansible_shared/automation_manager/credential.rb | 14 + 2 files changed, 19 insertions(+), 1 deletion(-) https://github.com/ManageIQ/manageiq-providers-ansible_tower/commit/5789c034d667ad32bed92d1c26541fb0410fc200 commit 5789c034d667ad32bed92d1c26541fb0410fc200 Author: Greg McCullough <gmccullo> AuthorDate: Thu Nov 1 17:15:03 2018 -0400 Commit: Greg McCullough <gmccullo> CommitDate: Thu Nov 1 17:15:03 2018 -0400 Merge pull request #138 from lfu/ansible_cred_ref_1640533 Credential.manager_ref need to be an integer for Tower 3.3 (cherry picked from commit 9e3e1ad903124851722654334055de6f24e1d9c7) https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/manageiq/providers/ansible_tower/shared/automation_manager/configuration_script_source.rb | 2 +- spec/support/ansible_shared/automation_manager/configuration_script_source.rb | 4 +- 2 files changed, 3 insertions(+), 3 deletions(-)
New commits detected on ManageIQ/manageiq-providers-ansible_tower/master: https://github.com/ManageIQ/manageiq-providers-ansible_tower/commit/920cd2eda847f5fdcacb98746dc78485e249bfe4 commit 920cd2eda847f5fdcacb98746dc78485e249bfe4 Author: James Wong <jwong> AuthorDate: Fri Oct 26 18:48:45 2018 -0400 Commit: James Wong <jwong> CommitDate: Fri Oct 26 18:48:45 2018 -0400 Tower 3.3 fix https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/manageiq/providers/ansible_tower/shared/automation_manager/credential.rb | 6 +- spec/support/ansible_shared/automation_manager/credential.rb | 14 + 2 files changed, 19 insertions(+), 1 deletion(-) https://github.com/ManageIQ/manageiq-providers-ansible_tower/commit/c0b0344f10c2e0063e52b2bd037ee008e243fe54 commit c0b0344f10c2e0063e52b2bd037ee008e243fe54 Author: Lucy Fu <lufu> AuthorDate: Thu Nov 1 15:29:50 2018 -0400 Commit: Lucy Fu <lufu> CommitDate: Thu Nov 1 15:29:50 2018 -0400 Credential.manager_ref need to be an integer for Tower 3.3 https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/manageiq/providers/ansible_tower/shared/automation_manager/configuration_script_source.rb | 2 +- spec/support/ansible_shared/automation_manager/configuration_script_source.rb | 4 +- 2 files changed, 3 insertions(+), 3 deletions(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/026f4f12ab0dc21b190ff0a5b5f8331f4bfc18ea commit 026f4f12ab0dc21b190ff0a5b5f8331f4bfc18ea Author: Lucy Fu <lufu> AuthorDate: Thu Nov 1 15:49:10 2018 -0400 Commit: Lucy Fu <lufu> CommitDate: Thu Nov 1 15:49:10 2018 -0400 Credential.manager_ref need to be an integer for Tower 3.3 https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/manageiq/providers/embedded_ansible/automation_manager/playbook.rb | 2 +- app/models/service_ansible_playbook.rb | 2 +- app/models/service_template_ansible_playbook.rb | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) https://github.com/ManageIQ/manageiq/commit/bf62a8c55930333bdcbe03240a56dbbabe539133 commit bf62a8c55930333bdcbe03240a56dbbabe539133 Author: Lucy Fu <lufu> AuthorDate: Thu Nov 1 15:51:09 2018 -0400 Commit: Lucy Fu <lufu> CommitDate: Thu Nov 1 15:51:09 2018 -0400 Changes of test cases for the integer Credential.manager_ref in Tower 3.3 https://bugzilla.redhat.com/show_bug.cgi?id=1640533 spec/factories/authentication.rb | 26 +- spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_spec.rb | 8 +- spec/models/service_ansible_playbook_spec.rb | 12 +- spec/models/service_template_ansible_playbook_spec.rb | 12 +- 4 files changed, 33 insertions(+), 25 deletions(-)
New commit detected on ManageIQ/manageiq/hammer: https://github.com/ManageIQ/manageiq/commit/2031d474ae4465f2fc9cafbba13e8614e0750cc5 commit 2031d474ae4465f2fc9cafbba13e8614e0750cc5 Author: Greg McCullough <gmccullo> AuthorDate: Mon Nov 5 11:00:25 2018 -0500 Commit: Greg McCullough <gmccullo> CommitDate: Mon Nov 5 11:00:25 2018 -0500 Merge pull request #18155 from lfu/ansible_cred_ref_1640533 Credential.manager_ref need to be an integer for Tower 3.3 (cherry picked from commit 9ac7cfad0cccb0109fdb53f4425e34f4ab7a023b) https://bugzilla.redhat.com/show_bug.cgi?id=1640533 app/models/manageiq/providers/embedded_ansible/automation_manager/playbook.rb | 2 +- app/models/service_ansible_playbook.rb | 2 +- app/models/service_template_ansible_playbook.rb | 2 +- spec/factories/authentication.rb | 26 +- spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_spec.rb | 8 +- spec/models/service_ansible_playbook_spec.rb | 12 +- spec/models/service_template_ansible_playbook_spec.rb | 12 +- 7 files changed, 36 insertions(+), 28 deletions(-)
Successfully ordered Ansible Playbook service with any error. Verified Version: 5.10.0.23.20181106165157_92dd189