You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/17 14:18:58 UTC

[GitHub] [airflow] vincbeck opened a new pull request, #25766: Add breeze parameter test-timeout to override pytest timeouts

vincbeck opened a new pull request, #25766:
URL: https://github.com/apache/airflow/pull/25766

   ### Notes
   When you run the command `breeze testing tests ....`, a default timeout of 60 seconds is set to pytest and there is no way to override. We need to override this value to run long running system tests.
   
   ### Testing
   - breeze testing tests --help
   - breeze testing tests tests/system/providers/amazon/aws/example_batch.py --system amazon --test-timeout 0
   - breeze testing tests tests/system/providers/amazon/aws/example_batch.py --system amazon
   
   @potiuk 


-- 
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 commented on pull request #25766: Add breeze parameter test-timeout to override pytest timeouts

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #25766:
URL: https://github.com/apache/airflow/pull/25766#issuecomment-1218124578

   LGTM. But the static checks will fail because the help images regeneration. Seems you did not have pre-commit set (highly recommended). But you can force it manually via `breeze setup regenerate-command-images`.


-- 
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 #25766: Add breeze parameter test-timeout to override pytest timeouts

Posted by GitBox <gi...@apache.org>.
potiuk merged PR #25766:
URL: https://github.com/apache/airflow/pull/25766


-- 
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] vincbeck commented on pull request #25766: Add breeze parameter test-timeout to override pytest timeouts

Posted by GitBox <gi...@apache.org>.
vincbeck commented on PR #25766:
URL: https://github.com/apache/airflow/pull/25766#issuecomment-1218431523

   Transient error here as well


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