[GitHub] [airflow] eejbyfeldt opened a new pull request #16744: Fix unchecked indexing in _build_metrics

[GitBox <gi...@apache.org>]

eejbyfeldt opened a new pull request #16744:
URL: https://github.com/apache/airflow/pull/16744


   I am not sure if this can happened during regular use of the cli, but when
   running test cases `sys.argv` can be that args passed to the pytest. When this
   is the case it is possible for argv to only contain a single element.
   
   For the test case I was adding the resulting `IndexError` was caught and
   ignored here:
   https://github.com/apache/airflow/blob/main/airflow/task/task_runner/standard_task_runner.py#L87
   meaning it was very hard to uncover the cause of the exception.
   


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

[GitHub] [airflow] potiuk merged pull request #16744: Fix unchecked indexing in _build_metrics

[GitBox <gi...@apache.org>]

potiuk merged pull request #16744:
URL: https://github.com/apache/airflow/pull/16744


   


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