You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by po...@apache.org on 2020/11/16 00:23:08 UTC

[airflow] branch v1-10-test updated: Sentry >= 0.19.0 introduces a breaking change

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch v1-10-test
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/v1-10-test by this push:
     new d689979  Sentry >= 0.19.0 introduces a breaking change
d689979 is described below

commit d689979383ca7613963b11252f76602757915d7c
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Mon Nov 16 00:42:39 2020 +0100

    Sentry >= 0.19.0 introduces a breaking change
---
 Dockerfile.ci | 2 +-
 setup.py      | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/Dockerfile.ci b/Dockerfile.ci
index 5578bc0..aa426ef 100644
--- a/Dockerfile.ci
+++ b/Dockerfile.ci
@@ -250,7 +250,7 @@ ENV AIRFLOW_CONSTRAINTS_URL=${AIRFLOW_CONSTRAINTS_URL}
 
 # By changing the CI build epoch we can force reinstalling Airflow from the current master
 # It can also be overwritten manually by setting the AIRFLOW_CI_BUILD_EPOCH environment variable.
-ARG AIRFLOW_CI_BUILD_EPOCH="4"
+ARG AIRFLOW_CI_BUILD_EPOCH="5"
 ENV AIRFLOW_CI_BUILD_EPOCH=${AIRFLOW_CI_BUILD_EPOCH}
 
 ARG AIRFLOW_PRE_CACHED_PIP_PACKAGES="true"
diff --git a/setup.py b/setup.py
index 64bedb7..0f1ace5 100644
--- a/setup.py
+++ b/setup.py
@@ -373,8 +373,10 @@ sendgrid = [
     'sendgrid>=5.2.0,<6',
 ]
 sentry = [
+    # Sentry SDK 0.19.0 introduces a breaking change
+    # https://github.com/getsentry/sentry-python/issues/920#issuecomment-727658274
     'blinker>=1.1',
-    'sentry-sdk>=0.8.0',
+    'sentry-sdk>=0.8.0, <0.19.0',
 ]
 slack = [
     'slackclient>=1.0.0,<2.0.0',