Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reporting: add a "Remove" action in "Reporting" Management app #26725

Closed
fbaligand opened this issue Dec 5, 2018 · 12 comments · Fixed by #60077
Closed

Reporting: add a "Remove" action in "Reporting" Management app #26725

fbaligand opened this issue Dec 5, 2018 · 12 comments · Fixed by #60077
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead (Deprecated) Team:Reporting Services Use Team:SharedUX and one of the Feature:Reporting:* labels instead. enhancement New value added to drive a business result

Comments

@fbaligand
Copy link
Contributor

Currently, when we go to "Management" app, and then to "Reporting" section, in the reports list, we don't have a "remove" button in "Actions" column.
This is quite annoying, when you generate a report (CSV, PDF), download it, and want to remove it immediately, because report is not useful anymore.

This would be great to add a "Remove" action available in reports list section.

image

@tsullivan
Copy link
Member

@stacey-gammon do you know if there was ever a reason we didn't just add this?

Only reason I can think of to not do it, is the user privileges are likely read-only for the .reporting index.

A workaround is to use something like Curator to clean up old data. Curator is for deleting entire indices, though, and it doesn't know if the generated reports have been downloaded. That suggestion is probably more to address really, really old reports.

@fbaligand
Copy link
Contributor Author

Yes, there is curator, and planned in 6.6, there will be Elasticsearch Index Lifecycle Management API that will allow to remove a full index automatically.
But this issue is not about removing a full (and old) index, but just one report.
It would be very helpful!

@kindsun kindsun added enhancement New value added to drive a business result (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead labels Dec 5, 2018
@stacey-gammon
Copy link
Contributor

@tsullivan - the only reason I delayed implementing is that I really wanted reporting to use the saved object client interface, and use that to delete the documents. This would get us all the auth stuff for free, and the @elastic/kibana-security team really wants everyone to go through that interfaces. Also then we could use index migrations on the reporting index.

The task manager is coming soon too, and so there was an open question of where these task results would be stored - in the task manager, or still in the separate .reporting index? We decided to still keep the reports in the .reporting index because of risk of bloating the task manager, but it was also an open question at the time.

So, given all these things in flux, it felt like it'd be a hack to throw it in now. That being said, maybe it's worth a hack just to get it in there sooner. 🤔

@fbaligand
Copy link
Contributor Author

Well, if migration to saved objects is planned for 6.6 release, then this fix is useless.
But if it is planned for 6.7 or even 7.0, then I think it is a quick win very useful!
Especially since there is currently no way to remove reports from Kibana UI.

@tsullivan
Copy link
Member

@stacey-gammon totally agree on using the saved objects API. My understanding is that would do the work for us to ensure a user could only delete their own report job documents.

About the Task Manager topic, I replied over here: #23632 (comment)

@fbaligand
Copy link
Contributor Author

Is there a pending PR "migrate Reporting section to Saved Objects" ?

@tsullivan
Copy link
Member

@fbaligand no PR up yet. The details are in this issue: #21689

@fbaligand
Copy link
Contributor Author

This feature seems to not be planned for Kibana 6.6. No ?
So the “remove” button in “Reporting” section may be relevant, waiting this feature being released.

@lexdevelop
Copy link

Still no action on this :(

@tsullivan tsullivan self-assigned this Jul 8, 2019
@tsullivan tsullivan removed their assignment Sep 30, 2019
@douniwan5788
Copy link

Need this !

@bmcconaghy bmcconaghy added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) and removed Team:Stack Services labels Dec 12, 2019
@bmcconaghy bmcconaghy added (Deprecated) Team:Reporting Services Use Team:SharedUX and one of the Feature:Reporting:* labels instead. and removed Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Dec 20, 2019
@elasticmachine
Copy link
Contributor

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

@iamhowardtheduck
Copy link

This has a been a major ask of a customer of mine. The usage of DevTools to delete reports over-complicates what should be a rather simple task.

Need this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead (Deprecated) Team:Reporting Services Use Team:SharedUX and one of the Feature:Reporting:* labels instead. enhancement New value added to drive a business result
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants