Re: [PR] Added Feature: retry strategy parameter to Amazon AWS Batch Operator [airflow]

2023-11-22 Thread via GitHub
ferruzzi commented on code in PR #35789: URL: https://github.com/apache/airflow/pull/35789#discussion_r1402826636 ## airflow/providers/amazon/aws/operators/batch.py: ## @@ -201,6 +204,7 @@ def __init__( self.scheduling_priority_override = scheduling_priority_override

Re: [PR] Added Feature: retry strategy parameter to Amazon AWS Batch Operator [airflow]

2023-11-22 Thread via GitHub
ferruzzi commented on code in PR #35789: URL: https://github.com/apache/airflow/pull/35789#discussion_r1402798016 ## airflow/providers/amazon/aws/operators/batch.py: ## @@ -201,6 +204,7 @@ def __init__( self.scheduling_priority_override = scheduling_priority_override

Re: [PR] Added Feature: retry strategy parameter to Amazon AWS Batch Operator [airflow]

2023-11-22 Thread via GitHub
ferruzzi commented on code in PR #35789: URL: https://github.com/apache/airflow/pull/35789#discussion_r1402798016 ## airflow/providers/amazon/aws/operators/batch.py: ## @@ -201,6 +204,7 @@ def __init__( self.scheduling_priority_override = scheduling_priority_override

Re: [PR] Added Feature: retry strategy parameter to Amazon AWS Batch Operator [airflow]

2023-11-22 Thread via GitHub
o-nikolas commented on PR #35789: URL: https://github.com/apache/airflow/pull/35789#issuecomment-1823621645 Looks like this change has broken the example_batch system test: https://aws-mwaa.github.io/#/open-source/system-tests/dashboard.html ``` File "/opt/airflow/airflow/provide

Re: [PR] Added Feature: retry strategy parameter to Amazon AWS Batch Operator [airflow]

2023-11-22 Thread via GitHub
potiuk commented on PR #35789: URL: https://github.com/apache/airflow/pull/35789#issuecomment-1822398356 Nice indeed :) -- 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 u

Re: [PR] Added Feature: retry strategy parameter to Amazon AWS Batch Operator [airflow]

2023-11-22 Thread via GitHub
boring-cyborg[bot] commented on PR #35789: URL: https://github.com/apache/airflow/pull/35789#issuecomment-1822398241 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.

Re: [PR] Added Feature: retry strategy parameter to Amazon AWS Batch Operator [airflow]

2023-11-22 Thread via GitHub
potiuk merged PR #35789: URL: https://github.com/apache/airflow/pull/35789 -- 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-unsubscr...@airflow.a