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.

Sign up for GitHub

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

Jump to bottom

[Reporting] Report fails for a dashboard with non existing vis #32959

Closed
TomonoriSoejima opened this issue Mar 12, 2019 · 4 comments
Closed

[Reporting] Report fails for a dashboard with non existing vis #32959

TomonoriSoejima opened this issue Mar 12, 2019 · 4 comments
Assignees
@tsullivan
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead enhancement New value added to drive a business result

Comments

@TomonoriSoejima
Copy link
Contributor

When you generate a report for a dashboard and the dashboard contains at least 1 non existing vis., the generation fails with Max attempt reached error.

It would be good if the reporting can detect such invalid vis and notify the error before even attempting to generate the report.
@timroes timroes added enhancement New value added to drive a business result (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Mar 12, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@timroes timroes added Team:Stack Services and removed Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Jul 18, 2019
@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 Team:Reporting Services 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)

@tsullivan tsullivan self-assigned this Jan 15, 2020
@tsullivan tsullivan changed the title suggestion for better message when report generation fails. [Reporting] Report fails for a dashboard with non existing vis Jan 15, 2020
@tsullivan
Copy link
Member

This is possibly solved in #52344, but it's still worth taking a look.

@tsullivan
Copy link
Member

In #52344 we no longer fail a job if there is an error on the page. We capture the error and take a screenshot for debugging purposes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsullivan tsullivan

Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead enhancement New value added to drive a business result
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@timroes @tsullivan @bmcconaghy @elasticmachine @TomonoriSoejima