8d5dcc5784
fix(report): fix last_eval_dttm sort and more tests ( #12121 )
...
* fix(report): fix last_eval_dttm sort and more tests
* remove unnecessary permissions and split code path
* remove SIP_34_ALERTS_UI
* disabling an alert that is working will turn it to not triggered
2020-12-21 19:07:30 +00:00
7369039713
tests: Parameterize some tests in alerts_tests.py ( #11313 )
...
* SO-1099 Parameterize some tests in alerts_tests.py
* Update alerts_tests.py
Remove commented out line
* Update alerts_tests.py
Reformatted using black
2020-10-19 23:05:04 -07:00
cb3f649a7f
chore: simplify alerting data model to leverage a single class ( #11179 )
...
* Collapse alerting models into a single one
Fixing upgrade migration & tests
* Address feedback
Co-authored-by: bogdan kyryliuk <bogdankyryliuk@dropbox.com>
2020-10-09 19:20:12 -07:00
b2fdf12f62
Support jinja templates ( #11008 )
...
Co-authored-by: bogdan kyryliuk <bogdankyryliuk@dropbox.com>
2020-09-24 10:29:44 -07:00
801fb4063c
Allow empty observations in alerting ( #10939 )
...
Co-authored-by: bogdan kyryliuk <bogdankyryliuk@dropbox.com>
2020-09-21 08:48:02 -07:00
c3f1720456
fix: superset alerting misc fixes ( #10891 )
...
* Superset alerting misc fixes
* Test 0 threshold
Co-authored-by: bogdan kyryliuk <bogdankyryliuk@dropbox.com>
2020-09-16 11:09:11 -07:00
ac2937a6c5
fix: use nullpool in the celery workers ( #10819 )
...
* Use nullpool in the celery workers
* Address comments
Co-authored-by: bogdan kyryliuk <bogdankyryliuk@dropbox.com>
2020-09-10 13:29:57 -07:00
54ae3b044f
fix: add validator information to email/slack alerts ( #10762 )
...
* added validator info to alerts
* adjusted format of messages
* added nits
Co-authored-by: Jason Davis <@dropbox.com>
2020-09-03 08:15:05 -07:00
b59f6b1833
feat: refractored SQL-based alerting framework ( #10605 )
...
* added new tables for alerting refractor
* reformatted inheritance structure
* added workflow for updated framework
* added suggested changes
* cleaned up changes
* added obervations to alert table to enable view
* added comments
* added requested changes
* fix tests
* added styling changes
* mypy
* added requested changes
* updated operator logic
* requested changes, 1 validator, styling changes
* refactored tests
* fix test alert workflow
* fixed create_alert in test
Co-authored-by: Jason Davis <@dropbox.com>
2020-09-01 13:36:02 -07:00
c0bb86d0d6
feat: slack integration for SQL-based alerts ( #10566 )
...
* add slack functionality
* deleted unused variable
* updated test
* black
* fix rebase
* added nits
* added slack no screenshot integration
* isort
* added namedtuple for screenshot
* added test
* fix precommit
Co-authored-by: Jason Davis <@dropbox.com>
2020-08-11 11:15:31 -07:00
8b9292ed05
fix: add retry to SQL-based alerting celery task ( #10542 )
...
* added retry and minimized sqlalchemy object lives
* pylint
* added try catch
* adjusted naming
* added scoped session
* update tests for dbsession
* added requested changes
* nit todo
Co-authored-by: Jason Davis <@dropbox.com>
2020-08-10 10:20:43 -07:00
fd2d1c58c5
Revert "chore: Cleanup database sessions ( #10427 )" ( #10537 )
...
This reverts commit 7645fc85c3
2020-08-06 15:33:48 -07:00
9c5b0e1c86
fix: refactored SQL-based alerts to not pass sqlalchemy objects as args ( #10506 )
...
* refractored alerting to not pass sqlalchemy obj as args
* updated to pass only alert id as arg
* used object id instead of argument
* updated alerts_tests.py to reflect change
Co-authored-by: Jason Davis <@dropbox.com>
2020-08-04 09:52:32 -07:00
7645fc85c3
chore: Cleanup database sessions ( #10427 )
...
Co-authored-by: John Bodley <john.bodley@airbnb.com>
2020-07-30 23:07:56 -07:00
7a329c25e9
feat: add test email functionality to SQL-based email alerts ( #10476 )
...
* added test email functionality
* formatting changes
* more formatting
* applied requested changes
* mypy
Co-authored-by: Jason Davis <@dropbox.com>
2020-07-30 13:27:22 -07:00
d065633c81
fix: Implement updates to SQL-based email alerts ( #10454 )
...
* implemented updates to alerting
* fixed imports and teardown
* changed unittest to pytest conventions
* add app_context to tests
Co-authored-by: Jason Davis <@dropbox.com>
2020-07-28 16:48:42 -07:00
Daniel Vaz Gaspar
hnle
Bogdan
Jason Davis
Erik Ritter
John Bodley