Created attachment 1284766 [details] CloudForms notification showing Tower Description of problem: When adding Ansible credentials in CloudForms, a notification appears with the following description: "The operation Ansible Tower Credential creation (name=...) on Tower (manager_id=...) completed successfully." Version-Release number of selected component (if applicable): 5.8.0.17.20170525183055_6317a22 How reproducible: Always Steps to Reproduce: 1. Create new credentials using Ansible Automation Inside in CloudForms 2. Check CloudForms notifications once complete Actual results: The description contains reference to Tower. Expected results: No mention of Tower. Additional info:
Hi Jerome, Where is this requirement from? That no mentioning of Tower? thanks, James
https://github.com/ManageIQ/manageiq/pull/15478
https://github.com/ManageIQ/manageiq-providers-ansible_tower/pull/10
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/f664b19dee18a3812640f5f95dd1252c4c8859b7 commit f664b19dee18a3812640f5f95dd1252c4c8859b7 Author: Greg Blomquist <gblomqui> AuthorDate: Thu Jun 29 12:27:38 2017 -0400 Commit: Greg Blomquist <gblomqui> CommitDate: Thu Jun 29 13:20:30 2017 -0400 Avoid Tower in notifications for embedded ansible ManageIQ should not indicate that anything "Tower" exists when dealing with Embedded Ansible. This should change the logs and notifications to use a more "embedded ansibly" term when talking about embedded ansibly things. Before: "The operation Ansible Tower Credential creation (name=...) on Tower ^^^^^^^^^^^^^ (manager_id=...) completed successfully." After: "The operation Ansible Automation Inside Credential creation (name=...) ^^^^^^^^^^^^^^^^^^^^^^^^^ on EMS(manager_id=1) completed successfully." https://bugzilla.redhat.com/show_bug.cgi?id=1458593 .../embedded_ansible/automation_manager/configuration_script.rb | 2 ++ .../embedded_ansible/automation_manager/configuration_script_source.rb | 2 ++ .../providers/embedded_ansible/automation_manager/credential.rb | 2 ++ 3 files changed, 6 insertions(+)
New commit detected on ManageIQ/manageiq-providers-ansible_tower/master: https://github.com/ManageIQ/manageiq-providers-ansible_tower/commit/9683ab60d1f8f178a032dda7bea68dc04925dba3 commit 9683ab60d1f8f178a032dda7bea68dc04925dba3 Author: Greg Blomquist <gblomqui> AuthorDate: Thu Jun 29 13:18:34 2017 -0400 Commit: Greg Blomquist <gblomqui> CommitDate: Mon Jul 17 16:49:47 2017 -0400 Avoid Tower in notifications for embedded ansible ManageIQ should not indicate that anything "Tower" exists when dealing with Embedded Ansible. This should change the logs and notifications to use a more "embedded ansibly" term when talking about embedded ansibly things. Before: "The operation Ansible Tower Credential creation (name=...) on Tower(manager_id=...) completed successfully." ^^^^^ After: "The operation Ansible Automation Inside Credential creation (name=...) on EMS(manager_id=1) completed successfully." ^^^ https://bugzilla.redhat.com/show_bug.cgi?id=1458593 .../providers/ansible_tower/shared/automation_manager/tower_api.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Verified in 5.9.0.2.20171010190026_0413a06. 'Ansible Tower' is not mentioned in CloudForms notifications.