elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.75k stars 8.15k forks source link

[Reporting] Full data table content must be contained in the pdf generated #17950

Closed elasticmachine closed 1 hour ago

elasticmachine commented 7 years ago

Original comment by @tbragin:

Right now only the preview of the data table shows up in the PDF report, because we essentially take a screenshot of the first page of the table. We should consider how to improve this.

Possibilities to discuss:

This is similar to the problem we have with saved search export LINK REDACTED

elasticmachine commented 7 years ago

Original comment by @tbragin:

cc: @kobelb

elasticmachine commented 7 years ago

Original comment by @tbragin:

We discussed this live and decided on the following approach in the immediate future:

Note, this is what Splunk does as well

Reason for not going with the other option:

mbarretta commented 6 years ago

also would include saved searches in this request: a saved search added to a dashboard will be truncated, even if the number of rows within the search is a relatively low number, like 20.

stacey-gammon commented 5 years ago

We discussed this a bit in the Sharing team planning week. Here is some of the notes from that session:

Saved Search and Data Tables: treat them the same.

tsullivan commented 5 years ago

@mbarretta

also would include saved searches in this request: a saved search added to a dashboard will be truncated, even if the number of rows within the search is a relatively low number, like 20.

For saved searches, there is a CSV export option in the "Share" menu when viewing the search in Discover.

There's a gap right now on being able to export CSV from a saved search when viewing it in the Dashboard context though, and we need to fix it.

Providing more convenient access to exporting CSV, for saved search and data table, is a general theme where we could see a lot of wins.

  1. Have a POST "Generation URL" to allow exporting data table (aggregated) visualization data to CSV
  2. Be able to export CSV from a saved search when viewing it in the Dashboard context
mbarretta commented 5 years ago

@tsullivan I appreciate that the CSV export allows saved search records to be exported, but not exporting everything seen in a dashboard breaks the "promise".

I've seen many dashboards that are essentially text-only: the preference is for raw data over graphics so that they can see everything in one place and avoid the extra click required to go from a visualization to the underlying data. In those cases, the dashboard is really more like a few Discover views glued together.

So yes, users could export the various data, import them into something like Excel, and then export to PDF from there, but I think it's a fair expectation for them to assume if there are 20 rows of records on a dashboard, they will be visible when that dashboard is exported.

Eniqmatic commented 5 years ago

+1 is there any more progress on this or when we will likely see it? We have a dashboard that is data tables with split tables based on how many security attacks have occured. But our PDF reports only show 1 page. Is there any temporary workarounds for this?

cmagdalena-racowireless commented 5 years ago

+1

tsullivan commented 5 years ago

@Eniqmatic The App team is actively working on doing a lot of heavy lifting around how the visualizations get their data, and the fruit of that effort will be that we can get data from a saved visualization on the server-side. That will lead into options on how to implement both of the items from the description:

There is a workaround to export CSV from an aggregated table visualization. It's from the UI, so unfortunately it would be hard to automate: you can click the Inspector for the visualization to see the underlying data. On that screen are export controls. See: https://www.elastic.co/guide/en/kibana/6.6/vis-inspector.html

Eniqmatic commented 5 years ago

@tsullivan Excellent sounds like what I am looking for. Hopefully the guys make good progress with that! Thanks for your response.

elasticmachine commented 4 years ago

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

mikeh688 commented 4 years ago

Enterprise users of Elastic need to be able to share pdf reports. The current limitation of only generating a pdf image for the first page of tabular data massively undermines the value of Elastic and KIbana in the eyes of these customers. It's fantastic that Kibana will do all the really funky new stuff (Lens etc) but, ultimately, if it cant deliver the basics, enterprise users wont adopt it because it doesnt meet their needs (mainstream adopters vs early adopters).

fbaligand commented 4 years ago

I fully agree with @mikeh688 Then, I have several users that ask me to have alerting with a Kibana table visualization as CSV attachment. And today, I can't address this need, that is very useful to be able to open data in Excel, and even, simply copy the text of the data table. Formatted data is necessary for me. Particularly to use number format locale. Microsoft Excel expects number format locale from current country. This is important for decimal numbers to us the good decimal separator. Same thing for dates.

reighnman commented 3 years ago

Same as above comments..

I ran into this issue with table data on a dashboard. I then tried just a pdf report at the table visualization, same issue.

I then saved the data table filters as a saved search to try a standalone csv report, but cannot do things like GROUP BY/aggregation/etc at the discovery layer that are necessary for this data.

Overall reporting still feels like a third-party addon or external cog, having to setup things like email configuration directly in the config. I would hope things are going in the same direction as alerting is going, with use of "connectors" managed internally and available to use with scheduled reports as well (in addition to fixing the issues with PDF generation on data tables).

elasticmachine commented 3 years ago

Pinging @elastic/kibana-app-services (Team:AppServices)

tsullivan commented 3 years ago

With SearchSource from @elastic/kibana-app-services available on the server side, it is now possible to re-create the query and tabify the search results using the reporting framework. The formatting of the data can easily(?) be the same in a report as it is seen in the browser. Once Reporting is able to access the raw table data, it could support CSV or PDF exports for the table data.

I think that is what has been needed all along. It has taken a long time to get here because it depended on having SearchSource available on the server-side.

mikeh688 commented 3 years ago

SearchSource sounds like something internal. So, your post sounds like great news and what does it mean in terms of something being available in release form that enables paged tables to be distributed in pdf reports? thanks Mike

tsullivan commented 3 years ago

@mikeh688 Hi, thanks for asking!

Feature:Reporting is owned by a larger team of devs that also maintain "SearchSource" which is a configuration scheme for searches that is becoming more and more ubiquitous in Kibana. The larger pool of resources and expertise means this can be worked on in the short term.

I have added this issue to a project board for App Services, and it will be discussed in a planning session so that an owner can take it on.

mikeh688 commented 3 years ago

Tim - thanks and great news that this is heading towards resolution.

fbaligand commented 3 years ago

As you say @tsullivan, most of my Kibana clients ask to be able to have a Data Table content sent by mail every day, in CSV format, as an attachment.

tornis commented 3 years ago

+1

tsullivan commented 3 years ago

This can be started after https://github.com/elastic/kibana/pull/88303

JamesB797 commented 3 years ago

It looks like this can be started now, as #88303 has been merged.

Being able to email daily reports from a lens visualization via watcher and a POST URL would solve an immediate business need for my company.

mikeh688 commented 3 years ago

Hi guys - great to see this moving forwards (and thanks). Is there any idea of a release ETA? thanks

tsullivan commented 2 years ago

Looks like there is a greater need for exporting CSV from table-based visualizations in an automation-friendly way. Therefore, we can close this issue in favor of https://github.com/elastic/kibana/issues/30982

mikeh688 commented 2 years ago

Can i challenge this being closed please? Customers who create dashboards and want to share (for example, management reports), require data table content to be contained in the pdf generated. The issue at the moment is that only the first page of the table is contained in the pdf output and this is very limiting.

tsullivan commented 2 years ago

Hi @mikeh688 I agree we still need an issue to provide what you wrote. This issue was a little vague because the description didn't put it as succinctly. I am going to re-open and edit the description to focus on "Customers who create dashboards... require data table content to be contained in the pdf generated"

The other issue is for allowing full CSV export from data table visualizations in an automation-friendly way, which solves the "full export" problem, but it isn't in PDF context.

mikeh688 commented 2 years ago

thanks

petrklapka commented 2 years ago

Thank you for contributing to this issue, however, we are closing this issue due to inactivity as part of a backlog grooming effort. If you believe this feature/bug should still be considered, please reopen with a comment.

ThomasEscobar commented 2 years ago

Hi @petrklapka, I stand with @mikeh688 in this case and would also oppose this being closed.

This is definitely still a requirement for us, and something we would love to see coming in the future. People are always interested in the dashboard export, and then think "ah..." when we tell them the limitations, especially for the data in their tables.

fbaligand commented 2 years ago

I’m also very interested by this feature!

mikeh688 commented 2 years ago

Q: when is a reporting tool not a reporting tool? A: when you cant share the data

Its quite embarrassing promoting KIbana as a reporting tool with lots of great features until a customer asks "can we share pdfs of the reports" and we have to answer "yes, but only the first page of any tabular data". It undermines the great features.

Please reopen the ticket

mbarretta commented 2 years ago

This is still an unmet requirement, per the last reopening. Reopening again.

petrklapka commented 2 years ago

Thanks for the feedback all! This is unfortunately the only way to dig out of our cruft of almost 900 issues. @mbarretta looks like you reopened it for me, so thanks. The squeaky wheels will get the kicks. Only a few hundred more to go!

mikeh688 commented 2 years ago

no problem and many thanks

elasticmachine commented 2 years ago

Pinging @elastic/kibana-presentation (Team:Presentation)

tsullivan commented 2 years ago

I chatted with @vadimkibana a bit about this today. Ultimately, the dashboard application will need to render itself to support a feature where tables are fully visible when a screenshot is taken, or when we get the PDF from Chromium. There needs to be two enhancements:

  1. In "preserve layout" mode (default for generating a PDF in dashboard), we can include additional pages at the end of the PDF for each table panel on the dashboard. Multiple pages would result in the PDF by having the Dashboard app provide multiple "locator params" - one for each page in the PDF. The concept of using multiple locator params in a report job is already used in Kibana for the Canvas application.
  2. In "print layout" mode, the custom CSS for screenshot layout, which is under the Dashboard domain, should have a selector for panels with tables inside of them, and ensure the height of those panels is not constrained and the panel doesn't show a scrollbar.

This outline is just to give context on why Vadim and I felt we need to tag the Presentation team on this issue

ThomThomson commented 1 year ago

@tsullivan I'm not sure it's feasible for us to build functionality into the "preserve layout" mode that appends pages onto the report for the contents of each table. When printing a dashboard, that mode is really more of a WYSIWYG sort of experience, where the layout of the dashboard grid is reproduced. I'm not convinced that we should introduce print specific changes to it.

The idea of ensuring the entirety of each panel is shown with no scroll bars in print mode is appealing, and something we could attempt.

scottf9 commented 1 year ago

Highly highly request this feature! So many times I get requests to schedule/build pdf reports and this severely limits what we can do.

mrin9san commented 1 year ago

Hi, we are platinum customers of Elastic and request this feature. Its not practical to simply open up elastic account in presentations and show reports. Anyways a report with screenshot of the first page is a useless feature.

gord-lyr commented 8 months ago

+1 on this enhancement. Moving from a competing SIEM product, and can't provide the same level of reporting to our customers.

petrklapka commented 8 months ago

Looping @osmanis, @kevinsweet and @rayafratkina - This feature ask has percolated up several times now here.

mikeh688 commented 8 months ago

we'd still like to see this feature; the requirement hasn't gone away (see earlier responses).

tsullivan commented 8 months ago

The feature added in https://github.com/elastic/kibana/pull/174511 (8.13) makes it possible to create a CSV report using data from an ES|QL query. It doesn't entirely satisfy these requirements, but it does for the first time make it possible to get a report of an aggregated search.

elasticmachine commented 1 month ago

Pinging @elastic/appex-sharedux (Team:SharedUX)

tsullivan commented 4 weeks ago

@elastic/kibana-presentation Hi, this issue pertains to the layout engine of the dashboard, so wouldn't really fall under the SharedUX team, which maintains the service of reporting. What I'd like to do next is add this to a new meta issue that collects other issues involving reporting where dashboard layout is factor of making the feature work better.

Update: created https://github.com/elastic/kibana/issues/191532

ThomThomson commented 4 weeks ago

This is a really interesting one. It's definitely the layout of Dashboard, but it's also the pagination of the table, etc. For instance, even if we limit to 500 rows - how do we do this? Are we calculating the exact pixel size of each row, rendering all rows in one page and spitting out a massive PDF of the Dashboard?

To be honest, I think this issue could be closed. Firing off a CSV export is likely a much better way to get all of the data from a table.

tsullivan commented 4 weeks ago

To be honest, I think this issue could be closed. Firing off a CSV export is likely a much better way to get all of the data from a table.

I agree with you, @ThomThomson. There is a ton of complexity under this and many open questions about how this could be implemented. If we close this we could instead focus on https://github.com/elastic/kibana/issues/30982, which about capturing aggregated table data (not a saved search, which is currently supported) in a CSV report.