You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "github-actions[bot] (via GitHub)" <gi...@apache.org> on 2026/04/13 16:42:02 UTC
[PR] [v3-2-test] Fix CI Slack notifications never firing recovery alerts (#65119) [airflow]
github-actions[bot] opened a new pull request, #65164:
URL: https://github.com/apache/airflow/pull/65164
The artifact lookup for the previous notification state was using
'gh api' with -f parameters, which makes the CLI default to POST
instead of GET. The artifacts endpoint returns 404 on POST, so the
script always treated each run as the first one with no prior state.
That meant the 'all tests passing' recovery notification could never
fire, since it requires the previous state to contain failures.
Force --method GET on the artifact lookup and add unit tests covering
both the GET requirement and the determine_action state machine.
(cherry picked from commit 35e002a6005d39ba3de283a72a5f9970f8912184)
Co-authored-by: Jarek Potiuk <ja...@potiuk.com>
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [v3-2-test] Fix CI Slack notifications never firing recovery alerts (#65119) [airflow]
Posted by "eladkal (via GitHub)" <gi...@apache.org>.
eladkal merged PR #65164:
URL: https://github.com/apache/airflow/pull/65164
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [v3-2-test] Fix CI Slack notifications never firing recovery alerts (#65119) [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #65164:
URL: https://github.com/apache/airflow/pull/65164#issuecomment-4243854471
ah :) thankis !
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org