Bug 1444853 - Standard output is missing in provisioned ansible playbook service
Summary: Standard output is missing in provisioned ansible playbook service
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.8.0
Hardware: All
OS: All
high
high
Target Milestone: GA
: 5.9.0
Assignee: Harpreet Kataria
QA Contact: Dmitry Misharov
URL:
Whiteboard: ansible:service:playbook
Depends On:
Blocks: 1446245
TreeView+ depends on / blocked
 
Reported: 2017-04-24 12:28 UTC by Dmitry Misharov
Modified: 2018-03-06 15:15 UTC (History)
10 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1446245 (view as bug list)
Environment:
Last Closed: 2018-03-06 15:15:36 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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.


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