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

PLAYWRIGHT_REPORTS_AND_THUMBNAILS grey output for dashboards #29324

Open
3 tasks done
kevinremoue opened this issue Jun 21, 2024 · 4 comments
Open
3 tasks done

PLAYWRIGHT_REPORTS_AND_THUMBNAILS grey output for dashboards #29324

kevinremoue opened this issue Jun 21, 2024 · 4 comments
Labels
alert-reports Namespace | Anything related to the Alert & Reports feature

Comments

@kevinremoue
Copy link

kevinremoue commented Jun 21, 2024

Bug description

I have the following flags on superset 3.1.x

FEATURE_FLAGS = {
    "DASHBOARD_RBAC": True,
    "ENABLE_TEMPLATE_PROCESSING": True,
    "GLOBAL_ASYNC_QUERIES": True,
    "HORIZONTAL_FILTER_BAR": True,
    "DRILL_BY": True,
    "DRILL_TO_DETAIL": True,
    "ALERT_REPORTS": True,
    "PLAYWRIGHT_REPORTS_AND_THUMBNAILS": True
    }

Using Selenium I had issue with map GL components so I activated the playwright flag.

Since I have activated playwright I am able to render single chart only in reports.
If I trigger reports on whole dashboard I got the following output:
image

No logs for playwright as the report is considered successful.

How to reproduce the bug

  1. Setup playwright integration
  2. Go to reports
  3. Create a report with a dashboard by email
  4. Check email content

Screenshots/recordings

No response

Superset version

3.1.3

Python version

3.9

Node version

16

Browser

Chrome

Additional context

No response

Checklist

  • I have searched Superset docs and Slack and didn't find a solution to my problem.
  • I have searched the GitHub issue tracker and didn't find a similar bug report.
  • I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.
@dosubot dosubot bot added the alert-reports Namespace | Anything related to the Alert & Reports feature label Jun 21, 2024
Copy link

dosubot bot commented Jun 21, 2024

Hello @kevinremoue! I'm here to help you with your Superset issue. I can assist with bug solving, answer questions, and guide you on becoming a contributor. Let's work on resolving your concern together while we wait for a human maintainer.

Enabling the PLAYWRIGHT_REPORTS_AND_THUMBNAILS feature in Apache Superset version 3.1.3 results in blank chart areas with a placeholder link when generating reports for an entire dashboard because the feature requires the installation of the "playwright" pip package. Additionally, a headless browser must be installed and properly configured. If these components are not installed or configured correctly, the feature will not function properly, leading to issues such as blank chart areas.

Moreover, the WebDriverPlaywright class in the superset/utils/webdriver.py file indicates that the Playwright integration involves several steps, including authenticating the browser context, waiting for elements to load, and handling potential errors. If any of these steps fail, it could result in blank chart areas. Specifically, the code waits for chart containers to draw and for loading elements to disappear. If these elements do not load correctly, the screenshot will not capture the charts, leading to blank areas [1][2][3][4].

To continue talking to Dosu, mention @dosu.

@rajivpatki
Copy link

I was just about to post this.

For me the output is blank unless I remove some elements from view in the dashboard. ie: if the dashboard is too long, it will generate a blank thumbnail, but if I put in a 2 tabs and hide an entire 2 rows of chart inside the second tab, the thumbnail is rendered properly.

I'm using playwright

@rusackas
Copy link
Member

Seems like a duplicate of #29326. We should probably close one of these in favor of the other.

@rajivpatki
Copy link

Seems like a duplicate of #29326. We should probably close one of these in favor of the other.

I wasn't sure if they were duplicates. I do face a similar issue with Slack Reports for long dashboards but some which don't render the Thumbnail manage to render the Report without issues. So I thought the problem might be different for these two issues

@rusackas rusackas mentioned this issue Jun 28, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alert-reports Namespace | Anything related to the Alert & Reports feature
Projects
None yet
Development

No branches or pull requests

3 participants