GitHub user dosubot[bot] added a comment to the discussion: Weird behavior and errors in Alert&Reports with playwright.
<!-- Answer -->
Yes, that's correct—the screenshot generation logic (including image =
self.get_screenshot(user=user, window_size=window_size)) always runs if a
screenshot is needed, regardless of your ALERT_REPORTS_EXECUTORS setting. The
executor config (like FixedExecutor("admin")) only determines which user
context is used for the task; it doesn't change or bypass the screenshot call
itself. The user argument passed to get_screenshot will be the one specified by
the executor, but the screenshotting mechanism is otherwise agnostic to the
executor policy. This design ensures the report runs with the right
permissions, but doesn't alter the screenshot logic itself.
[Reference](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset/utils/screenshots.py)
<!-- Dosu Comment Footer -->
*To reply, just mention [@dosu](https://go.dosu.dev/dosubot).*
---
<sup>How did I do?
[Good](https://app.dosu.dev/response-feedback/753dff69-af01-4f85-8cae-de8dfd692d0e?feedback_type=great_response)
|
[Irrelevant](https://app.dosu.dev/response-feedback/753dff69-af01-4f85-8cae-de8dfd692d0e?feedback_type=irrelevant_answer)
|
[Incorrect](https://app.dosu.dev/response-feedback/753dff69-af01-4f85-8cae-de8dfd692d0e?feedback_type=incorrect_sources)
|
[Verbose](https://app.dosu.dev/response-feedback/753dff69-af01-4f85-8cae-de8dfd692d0e?feedback_type=too_verbose)
|
[Hallucination](https://app.dosu.dev/response-feedback/753dff69-af01-4f85-8cae-de8dfd692d0e?feedback_type=hallucination)
| [Report
🐛](https://app.dosu.dev/response-feedback/753dff69-af01-4f85-8cae-de8dfd692d0e?feedback_type=bug_report)
|
[Other](https://app.dosu.dev/response-feedback/753dff69-af01-4f85-8cae-de8dfd692d0e?feedback_type=other)</sup> [](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=github)&
nbsp;[](https://go.dosu.dev/discord-bot) [](https://twitter.com/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/discussions/34297)
GitHub link:
https://github.com/apache/superset/discussions/34297#discussioncomment-13873859
----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
