[PR] [airflow-ctl/v0-1-test] Fix a bad retry env var taking down the whole airflowctl CLI (#72429) [airflow]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] opened a new pull request, #72663:
URL: https://github.com/apache/airflow/pull/72663

   The three retry knobs are coerced with int() at module scope, and every
   airflowctl invocation imports that module, so a value the operator never
   meant to set - an empty string from `docker run -e VAR`, an unexpanded CI
   template variable, a typo - left them unable to run even `--help` to find
   out what they had broken. A negative wait was worse still: it parsed
   cleanly and then surfaced intermittently as a sleep-length error from deep
   inside the retry layer, nowhere near its cause.
   (cherry picked from commit 1be286ecfe49f3be3ab842dff5cadba497d7ceaf)
   
   Co-authored-by: Y-C <ea...@gmail.com>
   Co-authored-by: Eason09053360 <18...@users.noreply.github.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] [airflow-ctl/v0-1-test] Fix a bad retry env var taking down the whole airflowctl CLI (#72429) [airflow]

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk merged PR #72663:
URL: https://github.com/apache/airflow/pull/72663


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