Re: [PR] Enable none default ftp port [airflow]

2024-04-27 Thread via GitHub
kerlion commented on PR #39048: URL: https://github.com/apache/airflow/pull/39048#issuecomment-2081334434 > Can you please add unit test to cover this change? Sorry, I do know know how to add unit test, but I tested it in my PROD env. -- This is an automated message from the Apache

[PR] add deferrable support to `DatabricksNotebookOperator` [airflow]

2024-04-27 Thread via GitHub
rawwar opened a new pull request, #39295: URL: https://github.com/apache/airflow/pull/39295 related: #39178 This PR intends to make `DatabricksNotebookOperator` deferrable -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] Update Databricks provider to depend on databricks-sql-connector >= 3.1.0 [airflow]

2024-04-27 Thread via GitHub
sunchao commented on issue #39274: URL: https://github.com/apache/airflow/issues/39274#issuecomment-2081272243 Thanks @Taragolis ! Yes, looks like this depends on https://github.com/apache/airflow/issues/28723 -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Don't clear downstream tasks when marked status is failure and downstream is True [airflow]

2024-04-27 Thread via GitHub
github-actions[bot] closed pull request #23079: Don't clear downstream tasks when marked status is failure and downstream is True URL: https://github.com/apache/airflow/pull/23079 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[PR] fix: sqa deprecations for airflow providers [airflow]

2024-04-27 Thread via GitHub
dondaum opened a new pull request, #39293: URL: https://github.com/apache/airflow/pull/39293 related: #28723 fix deprecations for SQLAlchemy 2.0 for Airflow providers. **FAB** SQLAlchemy 2.0 is changing the behavior when an object is being merged into a

Re: [I] Create a Cloud Storage Operator that could return a list of objects in a folder [airflow]

2024-04-27 Thread via GitHub
RNHTTR commented on issue #39290: URL: https://github.com/apache/airflow/issues/39290#issuecomment-2081158158 You can just edit this Issue to request that this Operator be added to the list of operators in the docs, I think. -- This is an automated message from the Apache Git Service. To

Re: [PR] Warn users when viewing docs for an older version of Airflow [airflow-site]

2024-04-27 Thread via GitHub
RNHTTR closed pull request #872: Warn users when viewing docs for an older version of Airflow URL: https://github.com/apache/airflow-site/pull/872 -- 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] Warn users when viewing docs for an older version of Airflow [airflow-site]

2024-04-27 Thread via GitHub
RNHTTR commented on PR #872: URL: https://github.com/apache/airflow-site/pull/872#issuecomment-2081157504 Nope :\ -- 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

(airflow) branch v2-9-test updated: Copy menu_item href for nav bar (#39282)

2024-04-27 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/v2-9-test by this push: new cddaf23e59 Copy menu_item href

(airflow) branch main updated: Copy menu_item href for nav bar (#39282)

2024-04-27 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham 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 25f901a963 Copy menu_item href for nav bar

Re: [PR] Copy menu_item href for nav bar [airflow]

2024-04-27 Thread via GitHub
jedcunningham merged PR #39282: URL: https://github.com/apache/airflow/pull/39282 -- 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:

(airflow) branch main updated (c946fc3f0b -> 04caa6eceb)

2024-04-27 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 c946fc3f0b `TriggerDagRunOperator` depreacte `exection_date` in favor of `logical_date` (#39285) add 04caa6eceb

Re: [PR] Update docstring `LivyOperator` retry_args and deferrable docs [airflow]

2024-04-27 Thread via GitHub
boring-cyborg[bot] commented on PR #39266: URL: https://github.com/apache/airflow/pull/39266#issuecomment-2081003609 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

Re: [PR] Update docstring `LivyOperator` retry_args and deferrable docs [airflow]

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

Re: [PR] Expose AWS IAM missing param in Hashicorp secret [airflow]

2024-04-27 Thread via GitHub
ReadytoRocc commented on code in PR #38536: URL: https://github.com/apache/airflow/pull/38536#discussion_r1581853068 ## airflow/providers/hashicorp/_internal_client/vault_client.py: ## @@ -318,15 +321,36 @@ def _auth_azure(self, _client: hvac.Client) -> None: )

(airflow) branch main updated: `TriggerDagRunOperator` depreacte `exection_date` in favor of `logical_date` (#39285)

2024-04-27 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal 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 c946fc3f0b `TriggerDagRunOperator` depreacte

Re: [I] Align TriggerDagRunOperator with logical date terminology [airflow]

2024-04-27 Thread via GitHub
eladkal closed issue #26916: Align TriggerDagRunOperator with logical date terminology URL: https://github.com/apache/airflow/issues/26916 -- 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

Re: [PR] added logical_date parameter [airflow]

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

Re: [I] Bug in Extras in Add Connection , while writing few words are hidden due to margin and padding [airflow]

2024-04-27 Thread via GitHub
pateash commented on issue #37583: URL: https://github.com/apache/airflow/issues/37583#issuecomment-2080822317 @Ram-tripath could you please share some screenshots of the issue you are facing? -- This is an automated message from the Apache Git Service. To respond to the message, please

(airflow) branch v2-9-test updated: Move significant note for past release to release notes (#39283)

2024-04-27 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/v2-9-test by this push: new 368ff13c13 Move significant note

(airflow) branch main updated (60b17bb78a -> 8dfdc3a0a8)

2024-04-27 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 60b17bb78a Improve helm chart annotation building script (#39286) add 8dfdc3a0a8 Move significant note for

Re: [PR] Move significant newsfragment for 2.9.0 to release notes [airflow]

2024-04-27 Thread via GitHub
jedcunningham merged PR #39283: URL: https://github.com/apache/airflow/pull/39283 -- 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:

(airflow) branch main updated (48c98bc1d4 -> 60b17bb78a)

2024-04-27 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 48c98bc1d4 Update Hashicorp AWS assume role auth docs (#39287) add 60b17bb78a Improve helm chart

Re: [PR] Improve helm chart annotation building script [airflow]

2024-04-27 Thread via GitHub
jedcunningham merged PR #39286: URL: https://github.com/apache/airflow/pull/39286 -- 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:

Re: [I] KubernetesPodOperator callback example from Doc doesn't work [airflow]

2024-04-27 Thread via GitHub
boring-cyborg[bot] commented on issue #39291: URL: https://github.com/apache/airflow/issues/39291#issuecomment-2080656538 Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for

[I] KubernetesPodOperator callback example from Doc doesn't work [airflow]

2024-04-27 Thread via GitHub
owler opened a new issue, #39291: URL: https://github.com/apache/airflow/issues/39291 ### What do you see as an issue? https://airflow.apache.org/docs/apache-airflow-providers-cncf-kubernetes/stable/operators.html#id13 The problem may related to the None value for api_version

Re: [PR] fix: use `sqlalchemy_url` property in `get_uri` for postgresql provider [airflow]

2024-04-27 Thread via GitHub
rawwar commented on code in PR #38831: URL: https://github.com/apache/airflow/pull/38831#discussion_r1581814020 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -113,6 +114,18 @@ def schema(self): def schema(self, value): self.database = value +

Re: [PR] fix: use `sqlalchemy_url` property in `get_uri` for postgresql provider [airflow]

2024-04-27 Thread via GitHub
rawwar commented on code in PR #38831: URL: https://github.com/apache/airflow/pull/38831#discussion_r1581812721 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -113,6 +114,18 @@ def schema(self): def schema(self, value): self.database = value +

Re: [I] Create a Cloud Storage Operator that could return a list of objects in a folder [airflow]

2024-04-27 Thread via GitHub
lopezvit commented on issue #39290: URL: https://github.com/apache/airflow/issues/39290#issuecomment-2080457087 Ok, after a bit of googleing I found that it exists!!!

[I] Create a Cloud Storage Operator that could return a list of objects in a folder [airflow]

2024-04-27 Thread via GitHub
lopezvit opened a new issue, #39290: URL: https://github.com/apache/airflow/issues/39290 ### Description Create a new operator inside of `airflow.providers.google.cloud.operators.gcs` that, given a pattern/preffix would return a list of files in said folder, similar to the

Re: [I] [Bug]: Papermill Provider installed via pip seems empty [airflow]

2024-04-27 Thread via GitHub
deramos commented on issue #39281: URL: https://github.com/apache/airflow/issues/39281#issuecomment-2080436529 @Taragolis the message I posted earlier shows that `papermill‘ doesn't exist in the airflow.providers namespace even after pip install. I think you may have missed it. -- This

Re: [I] Rendering custom map index before task is run. [airflow]

2024-04-27 Thread via GitHub
tomrutter commented on issue #39118: URL: https://github.com/apache/airflow/issues/39118#issuecomment-2080433343 Happy to be assigned, but if someone has a clear idea, happy for that to go ahead too. I’m guessing the big issue with rendering on the webserver is picking up the

Re: [I] KubernetesPodOperator duplicating logs when interrupted [airflow]

2024-04-27 Thread via GitHub
gbonazzoli commented on issue #39236: URL: https://github.com/apache/airflow/issues/39236#issuecomment-2080427612 @raphaelauv with version 8.1.1 the problem is still present. It seems that now is allways getting "_Pod docker-java-w2ade41b log read interrupted but container base

Re: [PR] Warn users when viewing docs for an older version of Airflow [airflow-site]

2024-04-27 Thread via GitHub
eladkal commented on PR #872: URL: https://github.com/apache/airflow-site/pull/872#issuecomment-2080412865 @RNHTTR are you still working on this? -- 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

(airflow) branch main updated: Update Hashicorp AWS assume role auth docs (#39287)

2024-04-27 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 48c98bc1d4 Update Hashicorp AWS assume role auth

Re: [PR] Update Hashicorp AWS assume role auth docs [airflow]

2024-04-27 Thread via GitHub
pankajastro merged PR #39287: URL: https://github.com/apache/airflow/pull/39287 -- 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:

Re: [PR] Updates to Teradata Provider [airflow]

2024-04-27 Thread via GitHub
eladkal commented on code in PR #39217: URL: https://github.com/apache/airflow/pull/39217#discussion_r1581740750 ## airflow/providers/teradata/transfers/azure_blob_to_teradata.py: ## @@ -0,0 +1,105 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[PR] Update Hashicorp AWS assume role auth docs [airflow]

2024-04-27 Thread via GitHub
pankajastro opened a new pull request, #39287: URL: https://github.com/apache/airflow/pull/39287 Fix: https://github.com/apache/airflow/pull/39279#discussion_r1581736661 --- **^ Add meaningful description above** Read the **[Pull Request