[GitHub] [airflow] bryzgaloff commented on issue #29366: default_args feature incompatible with Dynamic Task Mapping

2023-02-15 Thread via GitHub
bryzgaloff commented on issue #29366: URL: https://github.com/apache/airflow/issues/29366#issuecomment-1432666361 Hi @m1racoli thank you for reporting an issue :+1: I have encountered the same limitation which was surprising to me. I agree that `default_args` should keep working for dynamic

[GitHub] [airflow] boring-cyborg[bot] commented on issue #29567: Problem init sesion in Apache Airflow

2023-02-15 Thread via GitHub
boring-cyborg[bot] commented on issue #29567: URL: https://github.com/apache/airflow/issues/29567#issuecomment-1432635225 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] leo-frison opened a new issue, #29567: Problem init sesion in Apache Airflow

2023-02-15 Thread via GitHub
leo-frison opened a new issue, #29567: URL: https://github.com/apache/airflow/issues/29567 ### Apache Airflow version 2.5.1 ### What happened Ooops! Something bad has happened. Airflow is used by many users, and it is very likely that others had similar problem

[GitHub] [airflow] lwyszomi commented on issue #29538: GoogleCampaignManagerReportSensor not working correctly on API Version V4

2023-02-15 Thread via GitHub
lwyszomi commented on issue #29538: URL: https://github.com/apache/airflow/issues/29538#issuecomment-1432611302 @eladkal we are working on some of deprecations, and this is also in our plans to fix soon. -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [airflow] arjunanan6 commented on issue #29318: Return error when requirements fail to install

2023-02-15 Thread via GitHub
arjunanan6 commented on issue #29318: URL: https://github.com/apache/airflow/issues/29318#issuecomment-1432557858 I would like to take this one, if no one else is working on it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] dstandish commented on pull request #29500: Move libjars parameter in Sqoop Hook to Hook parameter

2023-02-15 Thread via GitHub
dstandish commented on PR #29500: URL: https://github.com/apache/airflow/pull/29500#issuecomment-1432488524 I don't understand why this change. What's wrong with configuring in connection? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] dannikay commented on pull request #29503: Get rid of state in Apache Beam provider hook

2023-02-15 Thread via GitHub
dannikay commented on PR #29503: URL: https://github.com/apache/airflow/pull/29503#issuecomment-1432485664 Locally triggered "breeze static-checks --all-files --show-diff-on-failure --color always" and it passes with the new patch. -- This is an automated message from the Apache Git Servi

[GitHub] [airflow] josh-fell commented on pull request #29562: Fix docstring for EcsRunTaskOperator region_name -> region

2023-02-15 Thread via GitHub
josh-fell commented on PR #29562: URL: https://github.com/apache/airflow/pull/29562#issuecomment-1432431320 Nice! Congrats on your first contribution @deanmorin! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[airflow] branch main updated: Fix docstring for EcsRunTaskOperator region_name -> region (#29562)

2023-02-15 Thread joshfell
This is an automated email from the ASF dual-hosted git repository. joshfell 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 cadab59e8d Fix docstring for EcsRunTaskOperator r

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #29562: Fix docstring for EcsRunTaskOperator region_name -> region

2023-02-15 Thread via GitHub
boring-cyborg[bot] commented on PR #29562: URL: https://github.com/apache/airflow/pull/29562#issuecomment-1432430573 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] josh-fell closed issue #28910: Misnamed param in EcsRunTaskOperator

2023-02-15 Thread via GitHub
josh-fell closed issue #28910: Misnamed param in EcsRunTaskOperator URL: https://github.com/apache/airflow/issues/28910 -- 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 unsub

[GitHub] [airflow] josh-fell merged pull request #29562: Fix docstring for EcsRunTaskOperator region_name -> region

2023-02-15 Thread via GitHub
josh-fell merged PR #29562: URL: https://github.com/apache/airflow/pull/29562 -- 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

[GitHub] [airflow] josh-fell opened a new pull request, #29565: Handle deleting more than 256 blobs using `WasbHook.delete_file()`

2023-02-15 Thread via GitHub
josh-fell opened a new pull request, #29565: URL: https://github.com/apache/airflow/pull/29565 Currently users cannot delete more than 256 blobs in a single `WasbHook.delete_file()` call per limits of the [`ContainerClient.delete_blobs()` API](https://learn.microsoft.com/en-us/python/api/a

[GitHub] [airflow] josh-fell commented on a diff in pull request #29548: Syedahsn/ec2 create terminate operators

2023-02-15 Thread via GitHub
josh-fell commented on code in PR #29548: URL: https://github.com/apache/airflow/pull/29548#discussion_r1107926793 ## tests/providers/amazon/aws/operators/test_ec2.py: ## @@ -20,23 +20,83 @@ from moto import mock_ec2 from airflow.providers.amazon.aws.hooks.ec2 import EC2Hook

[GitHub] [airflow] josh-fell commented on a diff in pull request #29548: Syedahsn/ec2 create terminate operators

2023-02-15 Thread via GitHub
josh-fell commented on code in PR #29548: URL: https://github.com/apache/airflow/pull/29548#discussion_r1107926635 ## tests/providers/amazon/aws/operators/test_ec2.py: ## @@ -20,23 +20,83 @@ from moto import mock_ec2 from airflow.providers.amazon.aws.hooks.ec2 import EC2Hook

[GitHub] [airflow] josh-fell commented on a diff in pull request #29548: Syedahsn/ec2 create terminate operators

2023-02-15 Thread via GitHub
josh-fell commented on code in PR #29548: URL: https://github.com/apache/airflow/pull/29548#discussion_r1107926537 ## tests/providers/amazon/aws/operators/test_ec2.py: ## @@ -20,23 +20,83 @@ from moto import mock_ec2 from airflow.providers.amazon.aws.hooks.ec2 import EC2Hook

[GitHub] [airflow] josh-fell commented on a diff in pull request #29548: Syedahsn/ec2 create terminate operators

2023-02-15 Thread via GitHub
josh-fell commented on code in PR #29548: URL: https://github.com/apache/airflow/pull/29548#discussion_r1107924100 ## airflow/providers/amazon/aws/operators/ec2.py: ## @@ -116,3 +116,139 @@ def execute(self, context: Context): target_state="stopped", ch

[GitHub] [airflow] bbovenzi opened a new pull request, #29564: reorient grid buttons

2023-02-15 Thread via GitHub
bbovenzi opened a new pull request, #29564: URL: https://github.com/apache/airflow/pull/29564 Save some vertical space in the grid view by moving the nav bar above the dag run bar chart. - Auto refresh goes inside of the main grid nav, to the top right. - Task group expand/collapse goe

[GitHub] [airflow] bbovenzi opened a new pull request, #29563: Improve grid view height calculations

2023-02-15 Thread via GitHub
bbovenzi opened a new pull request, #29563: URL: https://github.com/apache/airflow/pull/29563 In https://github.com/apache/airflow/pull/29367 I address the issue of the grid view heights being too short simply by doubling the minimum height. This PR tries to come up with a better solution.

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #29562: Fix docstring for EcsRunTaskOperator region_name -> region

2023-02-15 Thread via GitHub
boring-cyborg[bot] commented on PR #29562: URL: https://github.com/apache/airflow/pull/29562#issuecomment-1432311253 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 Contribution

[GitHub] [airflow] deanmorin opened a new pull request, #29562: Fix docstring for EcsRunTaskOperator region_name -> region

2023-02-15 Thread via GitHub
deanmorin opened a new pull request, #29562: URL: https://github.com/apache/airflow/pull/29562 closes: #28910 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-gui

[airflow] branch constraints-main updated: Updating constraints. Github run id:4188977065

2023-02-15 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-main by this push: new decd4f1976 Updating con

[GitHub] [airflow] deanmorin commented on issue #28910: Misnamed param in EcsRunTaskOperator

2023-02-15 Thread via GitHub
deanmorin commented on issue #28910: URL: https://github.com/apache/airflow/issues/28910#issuecomment-1432244105 Yes sorry for the slow reply @vincbeck I can do this. I found an actual bug so I'll need to figure out the PR process anyway -- This is an automated message from the Apache Git

[GitHub] [airflow] takersk opened a new issue, #29561: CeleryKubernetesExecutor @task(queue="kubernetes")  not work

2023-02-15 Thread via GitHub
takersk opened a new issue, #29561: URL: https://github.com/apache/airflow/issues/29561 ### Apache Airflow version Other Airflow 2 version (please specify below) ### What happened value.yaml ``` config: celery_kubernetes_executor: kubernetes_queue: '

[airflow] branch main updated (95e26d5d6b -> eb46eeb33d)

2023-02-15 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 95e26d5d6b Breeze OpenTelemetry Integration (#29521) add eb46eeb33d Fixed a typo in `CONTRIBUTING.rst` (#29559)

[GitHub] [airflow] kaxil merged pull request #29559: Fixed a typo in CONTRIBUTING.rst

2023-02-15 Thread via GitHub
kaxil merged PR #29559: URL: https://github.com/apache/airflow/pull/29559 -- 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.ap

[GitHub] [airflow] KarshVashi opened a new pull request, #29559: Fixed a typo in CONTRIBUTING.rst

2023-02-15 Thread via GitHub
KarshVashi opened a new pull request, #29559: URL: https://github.com/apache/airflow/pull/29559 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more

[GitHub] [airflow] kristopherkane commented on pull request #29136: Dataproc batches

2023-02-15 Thread via GitHub
kristopherkane commented on PR #29136: URL: https://github.com/apache/airflow/pull/29136#issuecomment-1432096963 Is there anyone that can run this for merge? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [airflow] vincbeck commented on a diff in pull request #29409: Fix Rest API update user output

2023-02-15 Thread via GitHub
vincbeck commented on code in PR #29409: URL: https://github.com/apache/airflow/pull/29409#discussion_r1107746275 ## airflow/api_connexion/openapi/v1.yaml: ## @@ -2288,7 +2288,7 @@ paths: content: application/json: schema: -

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107741070 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] blag commented on a diff in pull request #29441: datasets, next_run_datasets, remove unnecessary timestamp filter

2023-02-15 Thread via GitHub
blag commented on code in PR #29441: URL: https://github.com/apache/airflow/pull/29441#discussion_r1107726702 ## airflow/www/views.py: ## @@ -3715,7 +3715,6 @@ def next_run_datasets(self, dag_id): DatasetEvent, and_(

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107722874 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107721105 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] ashb commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
ashb commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107718180 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Configuri

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107711173 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] Taragolis commented on issue #29459: Unexpected error when using Cloud Run jobs command in BashOperator

2023-02-15 Thread via GitHub
Taragolis commented on issue #29459: URL: https://github.com/apache/airflow/issues/29459#issuecomment-1432016262 It is just my assumption about `gcloud` I could be wrong. The main question are you have any problem with other tasks? -- This is an automated message from the Apache Git S

[GitHub] [airflow] Taragolis commented on a diff in pull request #28525: Add CloudRunExecuteJobOperator

2023-02-15 Thread via GitHub
Taragolis commented on code in PR #28525: URL: https://github.com/apache/airflow/pull/28525#discussion_r1107708315 ## tests/providers/google/cloud/hooks/test_cloud_run.py: ## @@ -0,0 +1,79 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107706397 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] Taragolis commented on a diff in pull request #28525: Add CloudRunExecuteJobOperator

2023-02-15 Thread via GitHub
Taragolis commented on code in PR #28525: URL: https://github.com/apache/airflow/pull/28525#discussion_r1107705856 ## tests/providers/google/cloud/hooks/test_cloud_run.py: ## @@ -0,0 +1,78 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] 1inuxoid commented on pull request #29452: Add support of a different AWS connection for DynamoDB

2023-02-15 Thread via GitHub
1inuxoid commented on PR #29452: URL: https://github.com/apache/airflow/pull/29452#issuecomment-1431992437 @eladkal @ferruzzi please have a look at the changes at your convenience. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [airflow] syun64 commented on pull request #28961: Emit DataDog statsd metrics with metadata tags

2023-02-15 Thread via GitHub
syun64 commented on PR #28961: URL: https://github.com/apache/airflow/pull/28961#issuecomment-1431989563 @potiuk @hussein-awala @uranusjr I've started working with backend folks to add the new metric tags to the backend to be able to read the soon to be published metrics... I was rem

[GitHub] [airflow] ashb commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
ashb commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107671833 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Configuri

[GitHub] [airflow] ashb commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
ashb commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107670859 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Configuri

[GitHub] [airflow] ashb commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
ashb commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107671487 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Configuri

[GitHub] [airflow] Aakcht commented on pull request #29347: SSH Provider: Add cmd_timeout to ssh connection extra

2023-02-15 Thread via GitHub
Aakcht commented on PR #29347: URL: https://github.com/apache/airflow/pull/29347#issuecomment-1431948430 @ferruzzi I'd say that this PR is about the timeout for waiting for command reply. For example, consider the command ```bash sleep 10 ``` (as actually used in the tests for

[GitHub] [airflow] boring-cyborg[bot] commented on issue #29557: Execution Timeout is not working properly on airflow 2.5.0

2023-02-15 Thread via GitHub
boring-cyborg[bot] commented on issue #29557: URL: https://github.com/apache/airflow/issues/29557#issuecomment-1431913769 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] deepaktripathi1997 opened a new issue, #29557: Execution Timeout is not working properly on airflow 2.5.0

2023-02-15 Thread via GitHub
deepaktripathi1997 opened a new issue, #29557: URL: https://github.com/apache/airflow/issues/29557 ### Apache Airflow version Other Airflow 2 version (please specify below) ### What happened We're experiencing occasional issues with tasks that have specified an 'executio

[GitHub] [airflow-client-python] eladkal commented on issue #28: Pools API broken using airflow 2.1.2

2023-02-15 Thread via GitHub
eladkal commented on issue #28: URL: https://github.com/apache/airflow-client-python/issues/28#issuecomment-1431860576 Cool. please let us know -- 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 th

[GitHub] [airflow] ferruzzi commented on pull request #29347: SSH Provider: Add cmd_timeout to ssh connection extra

2023-02-15 Thread via GitHub
ferruzzi commented on PR #29347: URL: https://github.com/apache/airflow/pull/29347#issuecomment-1431843459 I think I need a little more context here. It looks like this PR is adding the option to set the command timeout for the session rather than for each command. Is that correct? And

[GitHub] [airflow-client-python] byingyang commented on issue #28: Pools API broken using airflow 2.1.2

2023-02-15 Thread via GitHub
byingyang commented on issue #28: URL: https://github.com/apache/airflow-client-python/issues/28#issuecomment-1431837367 @eladkal We chose to call the API manually using the Requests lib for now, but we can check again soon. -- This is an automated message from the Apache Git Service. To

[airflow] branch main updated (08bec93e5c -> 95e26d5d6b)

2023-02-15 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 08bec93e5c Improve health checks in example docker-compose and clarify usage (#29408) add 95e26d5d6b Breeze OpenT

[GitHub] [airflow] potiuk merged pull request #29521: Breeze OpenTelemetry Integration

2023-02-15 Thread via GitHub
potiuk merged PR #29521: URL: https://github.com/apache/airflow/pull/29521 -- 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

[GitHub] [airflow] Taragolis commented on issue #29556: Different AWS ECS Operators use inner Sensor and do not propagate connection arguments

2023-02-15 Thread via GitHub
Taragolis commented on issue #29556: URL: https://github.com/apache/airflow/issues/29556#issuecomment-1431742629 Personally I do not have a time until next week for work on this issue. If someone could/want work on this issue, feel free to ask to assign this issue. -- This is an automa

[GitHub] [airflow] Taragolis opened a new issue, #29556: Different AWS ECS Operators use inner Sensor and do not propagate connection arguments

2023-02-15 Thread via GitHub
Taragolis opened a new issue, #29556: URL: https://github.com/apache/airflow/issues/29556 ### Apache Airflow Provider(s) amazon ### Versions of Apache Airflow Providers main/develop ### Apache Airflow version main/develop ### Operating System N

[GitHub] [airflow] john-jac commented on a diff in pull request #29511: Add Landing Times entry to UI docs

2023-02-15 Thread via GitHub
john-jac commented on code in PR #29511: URL: https://github.com/apache/airflow/pull/29511#discussion_r1107387375 ## docs/apache-airflow/ui.rst: ## @@ -160,6 +160,22 @@ DAG over many runs. +Landing Times +. +Airflow landing times are calculated from

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107382127 ## tests/providers/sqlite/hooks/test_sqlite.py: ## @@ -28,26 +29,32 @@ class TestSqliteHookConn: def setup_method(self): - -self.connection = Connect

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107378635 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107377992 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] feluelle commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
feluelle commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1107376763 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Confi

[GitHub] [airflow] vincbeck commented on a diff in pull request #29513: AIP-44 Migrate TaskInstance.check_and_change_state_before_execution to Internal API

2023-02-15 Thread via GitHub
vincbeck commented on code in PR #29513: URL: https://github.com/apache/airflow/pull/29513#discussion_r1107319000 ## airflow/models/taskinstance.py: ## @@ -1207,9 +1208,11 @@ def get_dagrun(self, session: Session = NEW_SESSION) -> DagRun: return dr +@staticmeth

[GitHub] [airflow] raj-wadhwa commented on issue #22132: Show average dag run duration

2023-02-15 Thread via GitHub
raj-wadhwa commented on issue #22132: URL: https://github.com/apache/airflow/issues/22132#issuecomment-1431594210 Hi @moreaupascal56 , I did work on this feature. But for now, using it locally. Can we connect to review the solution and then submit it? -- This is an automated mess

[GitHub] [airflow] boring-cyborg[bot] commented on issue #29555: Dag Fails while creating a Dynamic Tasks using airflow variables.

2023-02-15 Thread via GitHub
boring-cyborg[bot] commented on issue #29555: URL: https://github.com/apache/airflow/issues/29555#issuecomment-1431580060 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] sangrpatil2 opened a new issue, #29555: Dag Fails while creating a Dynamic Tasks using airflow variables.

2023-02-15 Thread via GitHub
sangrpatil2 opened a new issue, #29555: URL: https://github.com/apache/airflow/issues/29555 ### Apache Airflow version Other Airflow 2 version (please specify below) ### What happened We have dynamic Dag, where we have a few Task Groups and each of these contains dynamic

[GitHub] [airflow] vincbeck commented on pull request #28900: Convert DagFileProcessor.execute_callbacks to Internal API

2023-02-15 Thread via GitHub
vincbeck commented on PR #28900: URL: https://github.com/apache/airflow/pull/28900#issuecomment-1431575917 I see, thank you. I guess [this proposal](https://github.com/apache/airflow/pull/29513#discussion_r1106553896) will impact this PR if we decide to apply it, so let's wait on a decision

[GitHub] [airflow] michaelmicheal commented on a diff in pull request #29441: datasets, next_run_datasets, remove unnecessary timestamp filter

2023-02-15 Thread via GitHub
michaelmicheal commented on code in PR #29441: URL: https://github.com/apache/airflow/pull/29441#discussion_r1107288797 ## airflow/www/views.py: ## @@ -3715,7 +3715,6 @@ def next_run_datasets(self, dag_id): DatasetEvent, and_(

[GitHub] [airflow] josh-fell commented on issue #29553: Calendar error in Tutorial DAG

2023-02-15 Thread via GitHub
josh-fell commented on issue #29553: URL: https://github.com/apache/airflow/issues/29553#issuecomment-1431548587 Hi @bervel, this is a duplicate of #29227. Closing this issue, but please let us know if you are experiencing this issue with a different scenario. -- This is an automated mess

[GitHub] [airflow] josh-fell closed issue #29553: Calendar error in Tutorial DAG

2023-02-15 Thread via GitHub
josh-fell closed issue #29553: Calendar error in Tutorial DAG URL: https://github.com/apache/airflow/issues/29553 -- 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

[GitHub] [airflow] mesaverde0207 commented on issue #29459: Unexpected error when using Cloud Run jobs command in BashOperator

2023-02-15 Thread via GitHub
mesaverde0207 commented on issue #29459: URL: https://github.com/apache/airflow/issues/29459#issuecomment-1431534336 Thank you for the comment, and sorry for my late response. So if the gcloud CLI ended the original process and created a new one in this case, is this the correct behavior

[GitHub] [airflow] moreaupascal56 commented on issue #22132: Show average dag run duration

2023-02-15 Thread via GitHub
moreaupascal56 commented on issue #22132: URL: https://github.com/apache/airflow/issues/22132#issuecomment-1431509391 Hey ! I'd love to have this feature for a long time as well, if you want some help I can give it a shot as well -- This is an automated message from the Apache Git Service

[GitHub] [airflow] dstandish commented on pull request #29496: Fix log tailing issues with legacy log view

2023-02-15 Thread via GitHub
dstandish commented on PR #29496: URL: https://github.com/apache/airflow/pull/29496#issuecomment-1431493511 > Tested it locally. Looks good! Super, thank you -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [airflow-client-python] eladkal merged pull request #71: Bump stale bot version

2023-02-15 Thread via GitHub
eladkal merged PR #71: URL: https://github.com/apache/airflow-client-python/pull/71 -- 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-client-python] branch main updated: Bump stale bot version (#71)

2023-02-15 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-client-python.git The following commit(s) were added to refs/heads/main by this push: new 9979a40 Bump stale bot version (#71

[GitHub] [airflow] mobuchowski closed pull request #29528: newsfragments: add missing 29289 newsfragment

2023-02-15 Thread via GitHub
mobuchowski closed pull request #29528: newsfragments: add missing 29289 newsfragment URL: https://github.com/apache/airflow/pull/29528 -- 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 c

[GitHub] [airflow] mobuchowski commented on a diff in pull request #29528: newsfragments: add missing 29289 newsfragment

2023-02-15 Thread via GitHub
mobuchowski commented on code in PR #29528: URL: https://github.com/apache/airflow/pull/29528#discussion_r1107187260 ## newsfragments/29289.significant.rst: ## @@ -0,0 +1,12 @@ +Listener API, which previously used SqlAlchemy events to be notified about +TaskInstance state change

[GitHub] [airflow] eladkal opened a new pull request, #29554: [WIP] Update issue triage policy

2023-02-15 Thread via GitHub
eladkal opened a new pull request, #29554: URL: https://github.com/apache/airflow/pull/29554 Following mailing list thread [[Discussion] Set further policies for triaging issues](https://lists.apache.org/thread/k8sv9cgqjzsgqm7x0h3ysmkkcq4m1owq) as we agreed on the high level concept we can

[GitHub] [airflow] mhenc commented on pull request #28900: Convert DagFileProcessor.execute_callbacks to Internal API

2023-02-15 Thread via GitHub
mhenc commented on PR #28900: URL: https://github.com/apache/airflow/pull/28900#issuecomment-1431391228 Usually I test it in breeze. I run two sessions: `breeze shell` in one and in another terminal `docker exec -it /bin/bash` Then in first I simply run `airflow internal-api` to

[GitHub] [airflow] Taragolis commented on a diff in pull request #29496: Fix log tailing issues with legacy log view

2023-02-15 Thread via GitHub
Taragolis commented on code in PR #29496: URL: https://github.com/apache/airflow/pull/29496#discussion_r1107119708 ## airflow/www/views.py: ## @@ -1527,7 +1527,15 @@ def get_logs_with_metadata(self, session=None): ti = ( session.query(models.TaskInstance)

[GitHub] [airflow-client-python] eladkal opened a new pull request, #71: Bump stale bot version

2023-02-15 Thread via GitHub
eladkal opened a new pull request, #71: URL: https://github.com/apache/airflow-client-python/pull/71 Same as https://github.com/apache/airflow/pull/29491 -- 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

[airflow-client-python] 01/01: Bump stale bot version

2023-02-15 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a commit to branch staleupdate in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git commit 3ef9b993e3bf83c679fe9b78543b6926d34b5593 Author: Elad Kalif <45845474+elad...@users.noreply.github.c

[airflow-client-python] branch staleupdate created (now 3ef9b99)

2023-02-15 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a change to branch staleupdate in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git at 3ef9b99 Bump stale bot version This branch includes the following new commits: new 3ef9b9

[airflow] branch constraints-main updated: Updating constraints. Github run id:4183433160

2023-02-15 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-main by this push: new a979afe913 Updating con

[GitHub] [airflow] eladkal commented on issue #11911: BigQuery support for create or replace table

2023-02-15 Thread via GitHub
eladkal commented on issue #11911: URL: https://github.com/apache/airflow/issues/11911#issuecomment-1431300066 thanks for the clarification. @jkukul would you like to raise a PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [airflow] shaneikennedy opened a new issue, #11911: BigQuery support for create or replace table

2023-02-15 Thread via GitHub
shaneikennedy opened a new issue, #11911: URL: https://github.com/apache/airflow/issues/11911 **Description** BigQuery supports multiple [create table statements](https://cloud.google.com/bigquery/docs/reference/standard-sql/data-definition-language#create_table_statement), one of wh

[airflow-client-python] branch main updated: Prepare to rename master branch to main (#70)

2023-02-15 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-client-python.git The following commit(s) were added to refs/heads/main by this push: new 7fa6125 Prepare to rename master br

[GitHub] [airflow-client-python] eladkal merged pull request #70: Prepare to rename master branch to main

2023-02-15 Thread via GitHub
eladkal merged PR #70: URL: https://github.com/apache/airflow-client-python/pull/70 -- 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...@

[GitHub] [airflow] jkukul commented on issue #11911: BigQuery support for create or replace table

2023-02-15 Thread via GitHub
jkukul commented on issue #11911: URL: https://github.com/apache/airflow/issues/11911#issuecomment-1431290823 @eladkal `exists_ok` parameter doesn't allow to replace a table if it exists. It will make the operator simply not create a table if it already exists. It corresponds to `CREATE TAB

[GitHub] [airflow] aru-trackunit commented on pull request #29477: `GoogleDriveHook`: Add folder_id param to upload_file

2023-02-15 Thread via GitHub
aru-trackunit commented on PR #29477: URL: https://github.com/apache/airflow/pull/29477#issuecomment-1431263957 @shahar1 I pulled your code and only added `folder_id` assignments ``` file_upload = LocalFilesystemToGoogleDriveOperator( task_id="uploading-file" g

[GitHub] [airflow] kobethuwis commented on issue #29393: S3TaskHandler continuously returns "*** Falling back to local log" even if log_pos is provided when log not in s3

2023-02-15 Thread via GitHub
kobethuwis commented on issue #29393: URL: https://github.com/apache/airflow/issues/29393#issuecomment-1431253928 Also experiencing this issue 😐 -- 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 th

[airflow] branch main updated (f0d90497da -> 08bec93e5c)

2023-02-15 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 f0d90497da Change expose_hostname default to false (#29547) add 08bec93e5c Improve health checks in example doc

[GitHub] [airflow] Taragolis merged pull request #29408: Improve health checks in example docker-compose and clarify usage

2023-02-15 Thread via GitHub
Taragolis merged PR #29408: URL: https://github.com/apache/airflow/pull/29408 -- 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

Error while running github feature from .asf.yaml in airflow-client-python!

2023-02-15 Thread Apache Infrastructure
An error occurred while running github feature in .asf.yaml!: [GitHub] Request error with message: "Branch not found". (status code: 404)

Error while running github feature from .asf.yaml in airflow-client-python!

2023-02-15 Thread Apache Infrastructure
An error occurred while running github feature in .asf.yaml!: [GitHub] Request error with message: "Branch not found". (status code: 404)

[airflow-client-python] branch main created (now 43750ac)

2023-02-15 Thread humbedooh
This is an automated email from the ASF dual-hosted git repository. humbedooh pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git at 43750ac Add stale bot (#69) No new revisions were added by this update.

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29528: newsfragments: add missing 29289 newsfragment

2023-02-15 Thread via GitHub
ephraimbuddy commented on code in PR #29528: URL: https://github.com/apache/airflow/pull/29528#discussion_r1106985299 ## newsfragments/29289.significant.rst: ## @@ -0,0 +1,12 @@ +Listener API, which previously used SqlAlchemy events to be notified about +TaskInstance state chang

[GitHub] [airflow] ashb commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
ashb commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1106950234 ## tests/providers/sqlite/hooks/test_sqlite.py: ## @@ -28,26 +29,32 @@ class TestSqliteHookConn: def setup_method(self): - -self.connection = Connection(

[GitHub] [airflow] eladkal commented on pull request #29477: `GoogleDriveHook`: Add folder_id param to upload_file

2023-02-15 Thread via GitHub
eladkal commented on PR #29477: URL: https://github.com/apache/airflow/pull/29477#issuecomment-1431048541 @shahar1 can you check https://github.com/apache/airflow/issues/29552 I think it would be good to add the parameter to `LocalFilesystemToGoogleDriveOperator` -- This is an automated

[GitHub] [airflow] eladkal commented on issue #29552: Google provider doesn't let uploading file to a shared drive

2023-02-15 Thread via GitHub
eladkal commented on issue #29552: URL: https://github.com/apache/airflow/issues/29552#issuecomment-1431043747 I think https://github.com/apache/airflow/pull/29477 will adress this. Can you confirm / review the PR? -- This is an automated message from the Apache Git Service. To respond to

[airflow] branch constraints-main updated: Updating constraints. Github run id:4182022152

2023-02-15 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-main by this push: new bb05809097 Updating con

[GitHub] [airflow] moiseenkov commented on pull request #28336: Fixed hanged KubernetesPodOperator

2023-02-15 Thread via GitHub
moiseenkov commented on PR #28336: URL: https://github.com/apache/airflow/pull/28336#issuecomment-1431004729 @dimberman , @dstandish , @jedcunningham, hi. Could you please review my fixes? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] ashb commented on a diff in pull request #28721: Use connection URI in SqliteHook

2023-02-15 Thread via GitHub
ashb commented on code in PR #28721: URL: https://github.com/apache/airflow/pull/28721#discussion_r1106860977 ## docs/apache-airflow-providers-sqlite/connections/sqlite.rst: ## @@ -23,14 +23,22 @@ The SQLite connection type provides connection to a SQLite database. Configuri

  1   2   >