(airflow) branch main updated (f4c4519f89 -> e81cefb8a6)

2024-05-25 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from f4c4519f89 Add max_consecutive_failed_dag_runs in API spec (#39830) add e81cefb8a6 Remove unused backward compati

Re: [PR] Delete code marked for deletion in azure wasb task logging [airflow]

2024-05-25 Thread via GitHub
eladkal merged PR #39827: URL: https://github.com/apache/airflow/pull/39827 -- 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.

[PR] Bump docker version to 7.1.0 [airflow]

2024-05-25 Thread via GitHub
eladkal opened a new pull request, #39839: URL: https://github.com/apache/airflow/pull/39839 [docker 7.1.0](https://github.com/docker/docker-py/releases/tag/7.1.0) is released with the needed bugfix to update request package > Fixed an issue due to an update in the [requests](https:/

Re: [PR] Allow FTPHook to change port number [airflow]

2024-05-25 Thread via GitHub
eladkal commented on PR #39465: URL: https://github.com/apache/airflow/pull/39465#issuecomment-2132077581 @VelmiraPetkova can you please fix static checks? -- 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

Re: [PR] [Experiment] Use `importlib` import mode for `pytest` [airflow]

2024-05-25 Thread via GitHub
github-actions[bot] closed pull request #38594: [Experiment] Use `importlib` import mode for `pytest` URL: https://github.com/apache/airflow/pull/38594 -- 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 t

Re: [PR] Fix inital value for _batch_id attribute in livy operator [airflow]

2024-05-25 Thread via GitHub
github-actions[bot] commented on PR #38628: URL: https://github.com/apache/airflow/pull/38628#issuecomment-2131753717 This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for you

Re: [I] Improve KubernetesExecutor Observability [airflow]

2024-05-25 Thread via GitHub
github-actions[bot] commented on issue #39215: URL: https://github.com/apache/airflow/issues/39215#issuecomment-2131753503 This issue has been automatically marked as stale because it has been open for 14 days with no response from the author. It will be closed in next 7 days if no further

[I] Add docs. for development of integration tests [airflow]

2024-05-25 Thread via GitHub
shahar1 opened a new issue, #39838: URL: https://github.com/apache/airflow/issues/39838 ### What do you see as an issue? [Current docs](https://github.com/a0x8o/airflow/blob/master/TESTING.rst#airflow-integration-tests) refer to the enablement and usage of integration tests, but ther

Re: [PR] Restore providers' unit tests marked as integration tests [airflow]

2024-05-25 Thread via GitHub
shahar1 closed pull request #39768: Restore providers' unit tests marked as integration tests URL: https://github.com/apache/airflow/pull/39768 -- 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 sp

Re: [PR] Restore providers' unit tests marked as integration tests [airflow]

2024-05-25 Thread via GitHub
shahar1 commented on PR #39768: URL: https://github.com/apache/airflow/pull/39768#issuecomment-2131426956 Closing as I relocated the unit tests in #39831 (to avoid confusion) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Revert "Deprecation of AutoML services: Add deprecation warnings and raise exceptions for already deprecated ones" [airflow]

2024-05-25 Thread via GitHub
eladkal commented on PR #39753: URL: https://github.com/apache/airflow/pull/39753#issuecomment-2131416738 Now that https://github.com/apache/airflow/pull/39752 is merged. Can we close this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please l

[PR] Revamp Contributor's quick start [airflow]

2024-05-25 Thread via GitHub
RNHTTR opened a new pull request, #39837: URL: https://github.com/apache/airflow/pull/39837 closes: #39653 -- 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 unsubscrib

Re: [I] Metric for pool_scheduled [airflow]

2024-05-25 Thread via GitHub
rawwar commented on issue #39757: URL: https://github.com/apache/airflow/issues/39757#issuecomment-2131385548 @andreyolv , I think this will be available in the next release : https://github.com/apache/airflow/pull/38899 -- This is an automated message from the Apache Git Service. To resp

Re: [I] reattach_on_restart doesn't work properly as exepected [airflow]

2024-05-25 Thread via GitHub
paramjeet01 commented on issue #39791: URL: https://github.com/apache/airflow/issues/39791#issuecomment-2131358763 When `reattach_on_restart` is set to true, it is expected that if a worker pod dies, a new worker pod will adopt the task pod. However, due to the `on_kill` method called by `t

Re: [I] reattach_on_restart doesn't work properly as exepected [airflow]

2024-05-25 Thread via GitHub
RNHTTR commented on issue #39791: URL: https://github.com/apache/airflow/issues/39791#issuecomment-2131315216 If I understand correctly, when [`reattach_on_restart`](https://github.com/apache/airflow/blob/f4c4519f89fcfacf9ef3494f820e4138a2ec3d05/airflow/providers/cncf/kubernetes/operators/po

Re: [PR] Add `max_consecutive_failed_dag_runs` in API spec [airflow]

2024-05-25 Thread via GitHub
pankajastro merged PR #39830: URL: https://github.com/apache/airflow/pull/39830 -- 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...@airf

(airflow) branch main updated: Add max_consecutive_failed_dag_runs in API spec (#39830)

2024-05-25 Thread pankaj
This is an automated email from the ASF dual-hosted git repository. pankaj pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new f4c4519f89 Add max_consecutive_failed_dag_runs in A

Re: [I] Unrecognized field "max_consecutive_failed_dag_runs" in DAG model [airflow]

2024-05-25 Thread via GitHub
pankajastro closed issue #39814: Unrecognized field "max_consecutive_failed_dag_runs" in DAG model URL: https://github.com/apache/airflow/issues/39814 -- 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

Re: [PR] Remove redundant code from Dockerfile [airflow]

2024-05-25 Thread via GitHub
RNHTTR commented on PR #39833: URL: https://github.com/apache/airflow/pull/39833#issuecomment-2131307443 I think there's actually a pre-commit hook that builds these blocks ([see here](https://github.com/apache/airflow/blob/main/Dockerfile#L81-L86)). -- This is an automated message from t

Re: [I] Tasks with 'none_failed_min_one_success' trigger_rule skipping before Dynamic Task Group is fully expanded [airflow]

2024-05-25 Thread via GitHub
RNHTTR commented on issue #39801: URL: https://github.com/apache/airflow/issues/39801#issuecomment-2131306606 Here's a slightly simpler reproduction without nested task groups or branching: ```python from airflow.decorators import dag, task, task_group from pendulum import datet

[PR] Resolving ECS fargate deprecated warnings [airflow]

2024-05-25 Thread via GitHub
dirrao opened a new pull request, #39834: URL: https://github.com/apache/airflow/pull/39834 Resolving ECS fargate deprecated warnings -- 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

Re: [PR] Remove redundant code from Dockerfile [airflow]

2024-05-25 Thread via GitHub
boring-cyborg[bot] commented on PR #39833: URL: https://github.com/apache/airflow/pull/39833#issuecomment-2131274318 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors'

[PR] Remove redundant code from Dockerfile [airflow]

2024-05-25 Thread via GitHub
surawut-jirasaktavee opened a new pull request, #39833: URL: https://github.com/apache/airflow/pull/39833 ### Remove Redundant Code in Dockerfile This pull request removes a duplicated block of code in the Dockerfile. The redundant code was found in the section respons

Re: [PR] Add `mssql` integration tests [airflow]

2024-05-25 Thread via GitHub
shahar1 commented on code in PR #39831: URL: https://github.com/apache/airflow/pull/39831#discussion_r1614658686 ## scripts/ci/docker-compose/integration-mssql.yml: ## @@ -0,0 +1,47 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license a

Re: [PR] Add `mssql` integration tests [airflow]

2024-05-25 Thread via GitHub
shahar1 commented on code in PR #39831: URL: https://github.com/apache/airflow/pull/39831#discussion_r1614658686 ## scripts/ci/docker-compose/integration-mssql.yml: ## @@ -0,0 +1,47 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license a

[PR] Add `mssql` integration tests [airflow]

2024-05-25 Thread via GitHub
shahar1 opened a new pull request, #39831: URL: https://github.com/apache/airflow/pull/39831 closes: #38577 related: #39768 - Creates a Docker Compose integration yaml file to `mssql`. - Add an integration test to `mssql` hook. - Adds integration tests for:

Re: [PR] Exponential Backoff Not Functioning in BaseSensorOperator Reschedule Mode [airflow]

2024-05-25 Thread via GitHub
sanket2000 commented on code in PR #39823: URL: https://github.com/apache/airflow/pull/39823#discussion_r1614558902 ## airflow/sensors/base.py: ## @@ -322,7 +322,7 @@ def run_duration() -> float: raise AirflowRescheduleException(reschedule_date) els

[PR] Add `max_consecutive_failed_dag_runs` in API spec [airflow]

2024-05-25 Thread via GitHub
pankajastro opened a new pull request, #39830: URL: https://github.com/apache/airflow/pull/39830 closes: https://github.com/apache/airflow/issues/39814 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apac

(airflow) branch main updated (2b6f8ffc69 -> cae96630e1)

2024-05-25 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 2b6f8ffc69 Upgrade node version used to compile assets (#39824) add cae96630e1 Remove 'legacy' definition for `

Re: [PR] Remove 'legacy' definition for `CronDataIntervalTimetable` [airflow]

2024-05-25 Thread via GitHub
Taragolis merged PR #39780: URL: https://github.com/apache/airflow/pull/39780 -- 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...@airflo

Re: [PR] Exponential Backoff Not Functioning in BaseSensorOperator Reschedule Mode [airflow]

2024-05-25 Thread via GitHub
Taragolis commented on code in PR #39823: URL: https://github.com/apache/airflow/pull/39823#discussion_r1614507844 ## airflow/sensors/base.py: ## @@ -322,7 +322,7 @@ def run_duration() -> float: raise AirflowRescheduleException(reschedule_date) else

Re: [I] dag.test() hangs with deferrable operators if DAG is paused [airflow]

2024-05-25 Thread via GitHub
pankajastro commented on issue #28865: URL: https://github.com/apache/airflow/issues/28865#issuecomment-2131112936 Could you please verify if this issue still exists? It should be fixed in PR https://github.com/apache/airflow/pull/34585. -- This is an automated message from the Apache Git

Re: [PR] Fix DAG test command stuck in infinite loop [airflow]

2024-05-25 Thread via GitHub
pankajastro commented on PR #32000: URL: https://github.com/apache/airflow/pull/32000#issuecomment-2131112558 look like issue has been fixed PR https://github.com/apache/airflow/pull/34585 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] Fix DAG test command stuck in infinite loop [airflow]

2024-05-25 Thread via GitHub
pankajastro closed pull request #32000: Fix DAG test command stuck in infinite loop URL: https://github.com/apache/airflow/pull/32000 -- 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 com

Re: [PR] Fetch intermediate log async GKEStartPod [airflow]

2024-05-25 Thread via GitHub
pankajastro commented on PR #39348: URL: https://github.com/apache/airflow/pull/39348#issuecomment-2131081171 > @pankajastro Any ETA on this? Thanks CI is green now. Hopefully, we will merge soon. In the meantime, could you please test it once? -- This is an automated message from