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] Blank reports for tag clouds visualizations #101021

Closed
bhavyarm opened this issue May 31, 2021 · 18 comments · Fixed by #100017
Closed

[Reporting] Blank reports for tag clouds visualizations #101021

bhavyarm opened this issue May 31, 2021 · 18 comments · Fixed by #100017
Labels
blocker bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead impact:critical This issue should be addressed immediately due to a critical level of impact on the product. loe:medium Medium Level of Effort regression Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@bhavyarm
Copy link
Contributor

Kibana version: 7.13.1 BC2

Elasticsearch version: 7.13.1 BC2

Server OS version: darwin_x86_64

Browser version: chrome latest

Browser OS version: OS X

Original install method (e.g. download page, yum, from source, etc.): from staging

Describe the bug: If you do reporting on tag clouds they are empty. Both for png and pdf

  1. Go to visualize
  2. Pick your tag cloud visualization
  3. click share and do either PDF or png reporting

Kibana prints empty reports
Screen Shot 2021-05-31 at 4 59 42 PM
@bhavyarm bhavyarm added bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:AppServices labels May 31, 2021
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

@bhavyarm
Copy link
Contributor Author

Same on sample data dashboards

@bhavyarm
Copy link
Contributor Author

bhavyarm commented Jun 1, 2021

Hey team this is a regression from 7.12.1. Thanks!

@bhavyarm
Copy link
Contributor Author

bhavyarm commented Jun 1, 2021

Same issue in 7.13.0. So not a regression?

@marius-dr
Copy link
Member

Broken on master too.

@exalate-issue-sync exalate-issue-sync bot changed the title Blank reports for tag clouds visualizations [Reporting] Blank reports for tag clouds visualizations Jun 1, 2021
@exalate-issue-sync exalate-issue-sync bot added impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort loe:medium Medium Level of Effort and removed loe:small Small Level of Effort labels Jun 1, 2021
@bhavyarm bhavyarm added blocker Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Jun 2, 2021
@bhavyarm
Copy link
Contributor Author

bhavyarm commented Jun 2, 2021

Same when you print reports on tag cloud in visualize library

@tsullivan
Copy link
Member

@elastic/kibana-app can you look into this? Something special about the tag cloud visualization is breaking it with reporting.

@tsullivan
Copy link
Member

While trying to look into this using the Chromium debugger option [1], I ran into this bug: #101233

[1] xpack.reporting.capture.browser.chromium.inspect: true

@flash1293
Copy link
Contributor

@stratoula Did anything change for tag clouds In 7.12.1?

@stratoula
Copy link
Contributor

stratoula commented Jun 3, 2021 •
edited
Loading

@flash1293 nope, the last PR that changed something on tagcloud was #84107 and was merged on 7.11.0. Have we tested it on 7.11.0 ?

@stratoula
Copy link
Contributor

I just tested it on 7.11.2 and it generates the report correctly

@stratoula
Copy link
Contributor

The good news are that I have just tested this PR #100017 which changes the implementation of tagcloud and it seems that the report is generated. So this will close soonish!

@stratoula stratoula mentioned this issue Jun 8, 2021
Merged
5 tasks
@bhavyarm bhavyarm reopened this Jul 15, 2021
@bhavyarm
Copy link
Contributor Author

This bug is back on 7.13.4 on cloud -
Screen Shot 2021-07-15 at 2 45 06 PM

@bhavyarm
Copy link
Contributor Author

bhavyarm commented Jul 15, 2021 •
edited
Loading

On local you can see its printing but layout is off -
Screen Shot 2021-07-15 at 2 57 13 PM

@tsullivan
Copy link
Member

@stratoula has anything changed for this visualization type?

@stratoula
Copy link
Contributor

@tsullivan @bhavyarm I cant replicate it.
image

The new tagcloud implementation, which solves this bug, will be released in 7.14.0. 7.13 unfortunately still has this bug but we cant backport it.

I have checked it both in master and in 7.14 cloud (I think BC2).

@bhavyarm
Copy link
Contributor Author

Right that makes sense. I was confused. Thanks @stratoula closing it!

@exalate-issue-sync exalate-issue-sync bot added impact:critical This issue should be addressed immediately due to a critical level of impact on the product. and removed impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. labels Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocker bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead impact:critical This issue should be addressed immediately due to a critical level of impact on the product. loe:medium Medium Level of Effort regression Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Tagcloud] Replaces current implementation with elastic-charts stratoula/kibana
6 participants
@tsullivan @flash1293 @marius-dr @bhavyarm @elasticmachine @stratoula