Bug 1478429 - 'Ansible Tower' should not be mentioned in CloudForms notification when using Ansible Automation Inside
'Ansible Tower' should not be mentioned in CloudForms notification when using...
Status: VERIFIED
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers (Show other bugs)
5.8.0
Unspecified Unspecified
high Severity high
: GA
: 5.8.2
Assigned To: Greg Blomquist
Dmitry Misharov
ansible_embed:notification
: ZStream
Depends On: 1458593
Blocks:
  Show dependency treegraph
 
Reported: 2017-08-04 10:22 EDT by Satoe Imaishi
Modified: 2017-08-28 04:32 EDT (History)
11 users (show)

See Also:
Fixed In Version: 5.8.2.0
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1458593
Environment:
Last Closed:
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Ansible


Attachments (Terms of Use)

  None (edit)
Comment 2 CFME Bot 2017-08-04 10:31:16 EDT
New commit detected on ManageIQ/manageiq/fine:
https://github.com/ManageIQ/manageiq/commit/864f8b325e2321f2f0fd9c29de189d5049c5c3f8

commit 864f8b325e2321f2f0fd9c29de189d5049c5c3f8
Author:     Adam Grare <agrare@redhat.com>
AuthorDate: Thu Jun 29 15:35:30 2017 -0400
Commit:     Satoe Imaishi <simaishi@redhat.com>
CommitDate: Fri Aug 4 10:24:18 2017 -0400

    Merge pull request #15478 from blomquisg/tower_not_tower
    
    Avoid Tower in notifications for embedded ansible
    (cherry picked from commit 2b6dd956c78620a609247ddfe7cfb13ee1108e35)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1478429

 .../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(+)
Comment 3 CFME Bot 2017-08-04 15:31:23 EDT
New commit detected on ManageIQ/manageiq/fine:
https://github.com/ManageIQ/manageiq/commit/519584923ddcf622f5351730641773f0c20ad53b

commit 519584923ddcf622f5351730641773f0c20ad53b
Author:     Adam Grare <agrare@redhat.com>
AuthorDate: Mon Jul 17 18:22:43 2017 -0400
Commit:     Satoe Imaishi <simaishi@redhat.com>
CommitDate: Fri Aug 4 15:25:47 2017 -0400

    Merge pull request #10 from blomquisg/tower_not_tower
    
    Avoid Tower in notifications for embedded ansible
    (cherry picked from commit b9c12e84e9904c64c094067397ab2dad8b281f7d)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1478429

 .../providers/ansible_tower/shared/automation_manager/tower_api.rb  | 2 +-
 .../automation_manager/configuration_script_source.rb               | 6 +++---
 spec/support/ansible_shared/automation_manager/credential.rb        | 6 +++---
 3 files changed, 7 insertions(+), 7 deletions(-)
Comment 4 Dmitry Misharov 2017-08-28 04:32:09 EDT
Fixed and verified in 5.8.2.0.20170824192913_b09a5f8. 'Ansible Tower' is not mentioned in CloudForms notifications.

Note You need to log in before you can comment on or make changes to this bug.