Bug 1444853

Summary: Standard output is missing in provisioned ansible playbook service
Product: Red Hat CloudForms Management Engine Reporter: Dmitry Misharov <dmisharo>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Dmitry Misharov <dmisharo>
Severity: high Docs Contact:
Priority: high    
Version: 5.8.0CC: bilwei, cpelland, dclarizi, dmisharo, hkataria, jhardy, mpovolny, obarenbo, pakotvan, sshveta
Target Milestone: GAKeywords: TestOnly
Target Release: 5.9.0   
Hardware: All   
OS: All   
Whiteboard: ansible:service:playbook
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1446245 (view as bug list) Environment:
Last Closed: 2018-03-06 15:15:36 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1446245    

Description Dmitry Misharov 2017-04-24 12:28:19 UTC
Description of problem:
There is no standard output box for provisioned service with a certain playbook.

Version-Release number of selected component (if applicable):
5.8.0.11-beta2.20170418170914_a53d2a7 

How reproducible:
Always

Steps to Reproduce:
1. Enable Embedded Ansible Role.
2. Add Ansible repository with this playbook https://github.com/patchkez/ansible_playbooks/blob/master/dump_all_variables.yml
3. Create a Catalog Item with that playbook.
4. Provision a Service from the Catalog Item
5. Navigate to Service/My services. Open provision tab for just provisioned service.

Actual results:
There is no standard output.

Expected results:
Standard output of the provision process should be presented.

Additional info:
/var/lib/awx/job_status directory contains ansible job status which should be shown in the UI, but it's not.

Comment 3 Dave Johnson 2017-04-24 13:49:28 UTC
Shveta, are you seeing the same thing here?  There was some notion that this is or was working, just want to get a second pair of eyes to confirm.  Thx

Comment 7 Shveta 2017-04-25 01:57:03 UTC
When I used https://github.com/patchkez/ansible_playbooks/blob/master/copy_file_example.yml the std output was printed .

So the issue is not std output . The playbooks differ in someway .

Comment 9 Bill Wei 2017-04-25 13:30:28 UTC
Harpreet, back to you.

Comment 11 CFME Bot 2017-04-27 09:23:11 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/9e0cffad1f5a4215e2cb971a064baf188e8dcd8d

commit 9e0cffad1f5a4215e2cb971a064baf188e8dcd8d
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Apr 26 13:01:47 2017 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed Apr 26 13:01:47 2017 -0400

    Removed decodeURI call.
    
    decodeURI was causing an issue rendering html on screen when '%' character is present in html output. Removed decodeURI call as html is getting rendered correctly.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1444853

 app/assets/javascripts/controllers/sanitize_and_render_controller.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 15 CFME Bot 2017-05-17 20:53:41 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/c2766036f3f8ebd6ae227fbe9e9fb564ebf9532d

commit c2766036f3f8ebd6ae227fbe9e9fb564ebf9532d
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed May 17 09:17:52 2017 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed May 17 16:13:48 2017 -0400

    Need to gsub occurrences of {{ }} in the output
    
    Moved gsub logic into a helper method. Having ", ', {{, }} characters in the output was causing an issue while rendering html using ngSanitize directive, needed to escape these characters to get the output to be displayed correctly on screen. Have to gsub " on the first line and then gsub for rest of the characters in a one liner works fine.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1451352
    https://bugzilla.redhat.com/show_bug.cgi?id=1444853

 app/controllers/service_controller.rb       | 13 +++++++++++++
 app/views/service/_svcs_show.html.haml      |  6 ++----
 spec/controllers/service_controller_spec.rb |  7 +++++++
 3 files changed, 22 insertions(+), 4 deletions(-)

Comment 17 Shveta 2017-10-31 18:46:51 UTC
Verification blocked by https://bugzilla.redhat.com/show_bug.cgi?id=1506796

Comment 18 Dmitry Misharov 2017-11-03 15:13:19 UTC
In 5.9.0.5.20171102023815_209f732  standard output is still missing.

Comment 20 Dmitry Misharov 2017-11-20 14:49:07 UTC
Verified in 5.9.0.9.20171115202245_7429f75. Standard output is shown correctly.