New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle soft fail for s3keysensor #1161
Conversation
76ba793
to
9488397
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1161 +/- ##
=======================================
Coverage 98.58% 98.58%
=======================================
Files 90 90
Lines 5368 5377 +9
=======================================
+ Hits 5292 5301 +9
Misses 76 76
☔ View full report in Codecov by Sentry. |
d01fad0
to
77eac18
Compare
b4cd2a0
to
1317ab8
Compare
1317ab8
to
807585c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we paste a screenshot of a DAG run when the sensor failed and the task was marked skipped. Looks good to me otherwise. Have a minor comment wrt. to the exception message in the case of poke.
Merging this but feel free to raise any concerns I will address them separately |
if soft_fail is set to true then raise AirflowSkipException. We may need to replicate this to other sensor too
closes: #1149