Tomeshwari-02 opened a new pull request, #69904:
URL: https://github.com/apache/airflow/pull/69904

   ## What changed
   
   `HttpSensor` now passes its configured `response_error_codes_allowlist` into 
`HttpSensorTrigger` when running in deferrable mode.
   
   `HttpSensorTrigger` now serializes that allowlist and uses it when deciding 
whether an HTTP error should keep waiting. Non-allowlisted HTTP errors are no 
longer silently retried in a tight loop.
   
   Closes #69903
   
   ## Why
   
   The synchronous `HttpSensor` path honored custom allowlisted status codes 
such as `503`, but the deferrable trigger path only retried `404`. This made 
the same sensor configuration behave differently after deferral.
   
   ## Tests
   
   - `python -m py_compile 
providers/http/src/airflow/providers/http/sensors/http.py 
providers/http/src/airflow/providers/http/triggers/http.py 
providers/http/tests/unit/http/sensors/test_http.py 
providers/http/tests/unit/http/triggers/test_http.py`
   - `git diff --check`
   
   I could not run the focused pytest file in this local shell because the 
Airflow test dependencies are not installed (`tests_common` first, then 
`time_machine`).


-- 
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]

Reply via email to