EpicFail4Ever opened a new issue, #65136:
URL: https://github.com/apache/airflow/issues/65136
### Under which category would you file this issue?
Providers
### Apache Airflow version
3.2.0
### What happened and how to reproduce it?
Hello,
I was waiting the PR #57032 to be merged with Sentry fix on the 3.x.
I'm currently testing it and it's not working as I expect.
**Settings:**
Start an instance with sentry setup :
```
AIRFLOW__SENTRY__SENTRY_ON=True
AIRFLOW__SENTRY__DSN=<dns>
AIRFLOW__SENTRY__DEBUG=True
AIRFLOW__SENTRY__ENVIRONMENT=local
```
Start a simple dag that raise an error :
```python
from airflow.sdk import dag, task
@task
def raise_error():
raise ValueError("This is a test error")
@dag()
def sentry_raise():
raise_error()
sentry_raise()
```
The task ends with an error but this is not catch in the sentry session.
### What you think should happen instead?
I expected to the the error catch and send to sentry at the end of the task
run.
### Operating System
Debian GNU/Linux 12 (bookworm)
### Deployment
Docker-Compose
### Apache Airflow Provider(s)
_No response_
### Versions of Apache Airflow Providers
apache-airflow 3.2.0
apache-airflow-core 3.2.0
apache-airflow-providers-celery 3.17.2
apache-airflow-providers-common-compat 1.14.2
apache-airflow-providers-common-io 1.7.2
apache-airflow-providers-common-sql 1.34.0
apache-airflow-providers-elasticsearch 6.5.1
apache-airflow-providers-fab 3.6.0
apache-airflow-providers-mysql 6.5.1
apache-airflow-providers-postgres 6.6.2
apache-airflow-providers-smtp 2.4.4
apache-airflow-providers-standard 1.12.2
apache-airflow-task-sdk 1.2.0
### Official Helm Chart version
Not Applicable
### Kubernetes Version
_No response_
### Helm Chart configuration
_No response_
### Docker Image customizations
_No response_
### Anything else?
- It seems to comes from the [task Run Sentry decorator
function](https://github.com/apache/airflow/blob/3.2.0/task-sdk/src/airflow/sdk/execution_time/sentry/configured.py#L132)
that expect to catch an error raise.
- Then the run function is catching any error in the run and returns it in
the third tuple item.
So, error are never raised and nothing is never sent to sentry... 🤗
### Are you willing to submit PR?
- [x] Yes I am willing to submit a PR!
### Code of Conduct
- [x] I agree to follow this project's [Code of
Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
