Bug 1447940

Summary: Formatting of Provider summary PDF file generated from provider summary page is very broken
Product: Red Hat CloudForms Management Engine Reporter: Jan Krocil <jkrocil>
Component: UI - OPSAssignee: Eric Winchell <ewinchel>
Status: CLOSED CURRENTRELEASE QA Contact: Jan Krocil <jkrocil>
Severity: medium Docs Contact:
Priority: high    
Version: 5.7.0CC: dclarizi, ewinchel, hkataria, jhardy, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: Regression, TestOnly, ZStream
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1460815 1465081 (view as bug list) Environment:
Last Closed: 2018-03-06 14:53:06 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: 1460815, 1465081    
Attachments:
Description Flags
Broken summary PDF none

Description Jan Krocil 2017-05-04 09:32:45 UTC
Created attachment 1276204 [details]
Broken summary PDF

Description of problem:
SSIA

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

How reproducible:
Always

Steps to Reproduce:
1. Add a provider (EC2 in my case)
2. Go to it's summary page
3. Click the "Download summary in PDF format" button

Actual results:
See attached file

Expected results:
Formatting is correct - with tables and graphics, not just text

Additional info:

Comment 2 Jan Krocil 2017-05-04 09:52:47 UTC
This is also broken in 5.7.2.1 but it used to work in 5.6.z. Changing the "Version" to "5.7.0" and adding Regression keyword.

Comment 5 CFME Bot 2017-05-11 20:03:32 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/7c50807e91a5d291382bc969df4865a6fa789770

commit 7c50807e91a5d291382bc969df4865a6fa789770
Author:     Eric Hayes <eric>
AuthorDate: Thu May 11 11:39:18 2017 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu May 11 12:02:11 2017 -0700

    Fix missing extension
    
    https://github.com/ManageIQ/manageiq/pull/14793
    https://bugzilla.redhat.com/show_bug.cgi?id=1447940

 app/controllers/application_controller/report_downloads.rb | 2 +-
 spec/controllers/ems_common_controller_spec.rb             | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

Comment 6 CFME Bot 2017-05-12 11:37:03 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/46719a32a8acb6136e046107b242f7c36124f52b

commit 46719a32a8acb6136e046107b242f7c36124f52b
Author:     Eric Hayes <eric>
AuthorDate: Tue Apr 18 09:06:56 2017 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu May 11 11:14:45 2017 -0700

    Find correct stylesheet path in plugins
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1447940

 lib/pdf_generator.rb | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

Comment 8 CFME Bot 2017-06-02 03:06:37 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/1bd4255b3914728f0d82763f8d633950ea82524b

commit 1bd4255b3914728f0d82763f8d633950ea82524b
Author:     Eric Hayes <eric>
AuthorDate: Mon May 15 08:33:30 2017 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu Jun 1 14:34:51 2017 -0400

    Compile and write stylesheets for PDF rendering
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1447940

 lib/asset_writer.rb                       | 33 +++++++++++++++++++++++++++++
 lib/pdf_generator.rb                      | 11 ++--------
 lib/pdf_generator/prince_pdf_generator.rb | 11 +++++++++-
 spec/lib/pdf_generator_spec.rb            | 35 +++++++++++++++++++++++++++++++
 4 files changed, 80 insertions(+), 10 deletions(-)
 create mode 100644 lib/asset_writer.rb
 create mode 100644 spec/lib/pdf_generator_spec.rb

Comment 20 Jan Krocil 2017-10-30 13:28:07 UTC
Verified fixed in 5.9.0.4 - 5.9.0.4.20171024163837_ef71ea6; formatting is now correct.

Works only for cloud providers. There is a separate bug for the button missing in infra providers.