search

Yelp / elastalert

Easy & Flexible Alerting With ElasticSearch
https://elastalert.readthedocs.org
Apache License 2.0
7.99k stars 1.73k forks source link

Upstream pull #3188

Closed Oliver-Sellwood closed 3 years ago

Oliver-Sellwood commented 3 years ago

https://github.com/Yelp/elastalert/pull/3161 - no, ci error https://github.com/Yelp/elastalert/pull/3141 - no, ci error https://github.com/Yelp/elastalert/pull/3017 - no, breaks tests https://github.com/Yelp/elastalert/pull/2881 - bad object https://github.com/Yelp/elastalert/pull/2858 - yes https://github.com/Yelp/elastalert/pull/2759 (this one actually happened to me) - yes https://github.com/Yelp/elastalert/pull/2750 - no, undocumented https://github.com/Yelp/elastalert/pull/2653 - bad object https://github.com/jertel/elastalert2/pull/157 - yes

tests are breaking in master anyway so I couldn't test locally if the pr's with CI errors actually break tests.

Would it be a good idea to push changes from this fork to https://github.com/jertel/elastalert2 and let them handle maintenance?

Oliver-Sellwood commented 3 years ago

@synhershko