Bug 1439273

Summary: Ansible output does not contain line breaks and is hard to read
Product: Red Hat CloudForms Management Engine Reporter: Roland Wolters <rwolters>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Pavol Kotvan <pakotvan>
Severity: high Docs Contact:
Priority: high    
Version: 5.8.0CC: cpelland, gmccullo, hkataria, jhardy, mkanoor, mpovolny, obarenbo, simaishi, tfitzger
Target Milestone: GAKeywords: TestOnly
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ansible
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1441295 (view as bug list) Environment:
Last Closed: 2018-03-06 14:41:40 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Ansible Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1441295    
Attachments:
Description Flags
Hard-to-read-Ansible output none

Description Roland Wolters 2017-04-05 14:50:02 UTC
Created attachment 1268994 [details]
Hard-to-read-Ansible output

Description of problem:
The Ansible standard output shown in the self service portal for executed services does not contain line brakes and thus is hard to read.

Version-Release number of selected component (if applicable):
5.8 alpha2

How reproducible:
Add playbook, order it via self service portal, watch output in services tab in self service portal.

Steps to Reproduce:
1. Create Ansible playbook service item
2. Order service item via self service
3. Watch output in "Services"

Actual results:
Output looks like shown in the attached screenshot

Expected results:
Output should respect line breaks and thus should be readable

Additional info:

Comment 3 Greg McCullough 2017-04-06 14:49:33 UTC
Bill, Please take a look and determine if this is a backend issue or needs to by handled by the UI teams.

Comment 4 Bill Wei 2017-04-06 17:10:12 UTC
Harpreet, please try the output with html format.

Comment 5 Harpreet Kataria 2017-04-06 18:03:56 UTC
https://github.com/ManageIQ/manageiq-ui-classic/pull/948

Comment 6 CFME Bot 2017-04-07 21:13:23 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/2f280f94a31c587a6cb18de1c738582a65766ded

commit 2f280f94a31c587a6cb18de1c738582a65766ded
Author:     Harpreet Kataria <hkataria>
AuthorDate: Thu Apr 6 14:00:48 2017 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Apr 7 16:55:38 2017 -0400

    changed stdout format to be html
    
    Added new JS controller to handle sanitize and render html on the screen.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1439273

 .../controllers/sanitize_and_render_controller.js  |  8 +++
 app/views/service/_svcs_show.html.haml             | 81 ++++++++++++----------
 2 files changed, 51 insertions(+), 38 deletions(-)
 create mode 100644 app/assets/javascripts/controllers/sanitize_and_render_controller.js