-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
Error Report Failed Taking Screenshot #22326
Comments
Try running |
thanks. it works for me. |
Closing as this appears to be resolved. Please ping me here if that's not the case! Thanks @sfirke! |
Why is this closed as fixed if the solution is installing older version ? |
Most of the time people reporting errors in Alerts & Reports are better served by running the latest stable version (I wouldn't say "an older version", it's the latest release), since they want production-ready software. So when that's the solution, these issues are solved. As in this issue. That said, I think you are right that this will need to get fixed on the master branch. @rusackas What do you think? Should a new issue be opened for this? |
Solution is then not a fix for me, as I reverted to latest stable just because of this, and given that we based our recent work on latest master, as I think is recommended in the official docs, I have host of other problems now - missing stuff, jumping charts etc. I am considering abandoning email or reimplementing it differently just so i can revert to latest as it behaves much better. Currrent solution can not be quialified as a fix of any kind and thus this issue should be reopen, IMO. Not sure why would you want to reopen new issue for it too, that practice can only make it harder for people to follow what is going on and which issue is relevant and artificially inflates the number of issues. |
No problem re-opening this issue. I think i misinterpreted some of the thread a little bit. I think I saw some related issues that were marked as closed... I'll do a little digging to see if I can find them and link them here in case they help. |
Related things seem to warrant investigation: |
Sorry to hear that. That new users are directed to deploy |
Thanks for reopening this.
I understand your point, but on the other hand users can have fixes ASAP that way instead of waiting for official release and you get more "beta testers" that way... halthy master is the point. For instance, this problem would probably be discovered after the official release and now it can get fixed before... |
Could you share your whole setup with me ? - I'm running into the same problem and changing to 2.0.0-dev doesn't seem to work for me |
Had the same bug. But found a workaround) Final configuration (superset_config_docker.py):
One more note: png is working but csv still doesn't work with the issue Failed generating csv HTTP Error 407: Authorization Required Good luck! |
Hi @gonzo-soc, I used GECKODRIVER_VERSION=v0.29.0 for Linux-amd64 and it works fine. Is there any workaround? |
I'm experiencing this bug while testing reports using the 2.1.0rc2-dev image. |
I wouldn't call this a bug, more so the default config doesn't work with the 2.1.0rc2-dev image out of the box. You just need to override this config to make it work: https://github.com/apache/superset/blob/2.1.0rc2/superset/config.py#L1307 In fact, the comment in the file mentions that the arg should be removed if using firefox |
I tried specifying |
This should be fixed in 2.1.0rc3 |
hello, I am experiencing the same issue here, with |
is there an official fix for this?! Deploying thru helm chart, I am facing the same issue. I do helm pull on the repo, so I assume it pulls the latest version. Any suggestions? |
This was fixed with this PR: https://github.com/apache/superset/pull/23388/files . Specifically the line This functionality works on the latest official release of Superset, 2.1.0. If someone is getting |
I have pulled the latest docker image from "https://hub.docker.com/r/apache/superset/tags" so it must be the latest tag. Checked the link that you provided, I didn't have those apt installation in values.yaml file. I added them. Now i am getting "Failed taking a screenshot Message: 'chromedriver' executable needs to be in PATH". I check the config.py file in the worker pod and its "WEBDRIVER_OPTION_ARGS = ["--headless"]". |
Are you using |
i just pulled the
|
hey @asayeb
And in your
|
thank @amartincolville. can you please let me know which one is up to date. Docker or Kubernetes documentation? |
@asayeb same here. Struggling for last 2 days to make it work. :( |
@santosh-sahoo334 I followed the Kubernetes documentation: https://superset.apache.org/docs/installation/running-on-kubernetes . removed the reports I created and created new ones. csv, text and dashboard works fine. png still has issue for me. Follow the Kubernetes documentation if you are using helm deployment. a follow up question: Is there an option to send reports directly to a google drive instead of email? thanks @sfirke and @amartincolville for the help. |
A clear and concise description of what the bug is.
Error on scheduled report : Failed taking a screenshot Message: Argument --marionette can't be set via capabilities
Superset run on docker
How to reproduce the bug
Expected results
dashboard screenshot received by email or slack channel
what you expected to happen.
dashboard screenshot received by email or slack channel
Actual results
error Failed taking a screenshot Message: Argument --marionette can't be set via capabilities
The text was updated successfully, but these errors were encountered: