Manageiq-ui-classic: Print or Export to PDF for saved reports is missing data

[h-kataria]

Downloading a report to pdf has empty report, Only report headers are visible.

works fine for list of hosts and host summary, it looks like only an issue with saved reports

[martinpovolny]

@h-kataria : I need some help understanding what is the expected behavior here.

• when the PDF download is requested, should the existing report result be downloaded or shall a new report run be initialized and a new report result created and displayed?

I am asking because currently a new reprot results is created but then there's code that explicitly asks for the last run time of the OLD (previous) report result.

That OLD last run time is then presented together with the NEW data.

To me that looks like multiple people touching this area "fixing" conflicting BZs ;-) A mess.

I was trying to find some meaning in this when I probably caused this regressing. But I'd prefer to find some order in this rather than "just fix" this issue.

WDYT?

[martinpovolny]

Might be that I broke this by using the NEW report result for the data. But then why is the task being executed? Why don't we directly display the old report restult w/o firing any task?

[martinpovolny]

Might be that previously @skateman did changes when removing the PDF generator and using HTML to generate the PDF in the browser. May be he kept the running of the report.

Before that time it was actually needed to run the report to get the PDF results. Because only the HTML lines where saved, not the PDF.

And even before that someone "fixed" the "wrong" time of run of the report (because he/she was expecting the old date, but the PDF got generated with a new date)....

Never mind the history, I need to understand what is the expected "correct" behavior ;-)

[martinpovolny]

Ok, according to our f2f meeting I will proceed with removing the execution of the report and making sure that the very same data that was displayed on the screen is being exported to PDF.

That means removing the wait_for_task logic from this path and having just one report_result_id to deal with -- the old, existing one.

[epwinchell]

@h-kataria I think this can be closed. Thanks.