-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
Send emails does not work -official documentation wrong? CHROME=>version incompatible ChromeDriver/Chrome FIREFOX=> 'geckodriver' executable needs to be in PATH #20843
Comments
@nigzak thanks for all the effort you put into filing this issue, and sorry it slipped through the cracks for so long. A lot of updates have been made in this area. Can you confirm whether or not this is still an issue? If it is, we'll do our best to repro/triage accordingly (CC @yousoph - happy to help track this with you as needed) |
Hi @rusackas I have added both driver in my container (still like this :)) and I am using currently CHROME variant. Official Documentation => https://superset.apache.org/docs/installation/alerts-reports/ This walk-through is not working because the versions are not matching and the use will fail. Minimum suggested fix: remove "_88" to use latest version of chromedriver.
exact code lines where are not matching on docu page:
With this fix currently chromedriver version works. |
Just opened a teensy PR with your suggestion. If that merges, it should close this issue. Thanks for the report and the update! |
Sending emails does not work - does not mind if firefox or chrome is used
How to reproduce the bug
create custom dockerfile with adding firefox or chrome (same issue if you only add one of them) and config
having redis active and reachable from docker
start docker
init image, load example and add a job for sending a report every minute
start celery
HINT: featureflag alert_reports is active
Expected results
email is sent
Actual results
error is shown in logfile, no email is sent
Official Documentation: https://superset.apache.org/docs/installation/alerts-reports/
HINT CHROME: the chromedriver version is using V88 (LATEST_RELEASE_88) and the chrome stable is using currently 103 => I already changed this and updated "LATEST_RELEASE_88" to "LATEST_RELEASE"
=> if I use it without "_88" the error with version incompatibility is not there anymore and it works
firefox
dockerfile
error
chrome LATEST_RELEASE_88 (from docu page, unchanged)
dockerfile
error
Environment
(please complete the following information):
Checklist
Make sure to follow these steps before submitting your issue - thank you!
Additional context
superset_config firefox
superset_config chrome
The text was updated successfully, but these errors were encountered: