[GitHub] [airflow] maxcountryman commented on pull request #7232: [AIRFLOW-6569] Flush pending Sentry exceptions before exiting forked process

2020-05-22 Thread GitBox
maxcountryman commented on pull request #7232: URL: https://github.com/apache/airflow/pull/7232#issuecomment-632730483 It seems like the tests that are failing are unrelated to this patch and seem to be the same tests that are also failing on master. What's blocking merging this in

[GitHub] [airflow] maxcountryman commented on pull request #7232: [AIRFLOW-6569] Flush pending Sentry exceptions before exiting forked process

2020-05-18 Thread GitBox
maxcountryman commented on pull request #7232: URL: https://github.com/apache/airflow/pull/7232#issuecomment-630555724 I believe the bug this patch addresses is actively preventing us from receiving alerts in production.