Re: [PR] Give `on_task_instance_failed` access to the error that caused the failure [airflow]

2024-03-20 Thread via GitHub
ephraimbuddy commented on PR #38155: URL: https://github.com/apache/airflow/pull/38155#issuecomment-2011336350 > > > This would be rather hacky. Likely better to add a new method `on_task_instanc_failed_with_error` > > > > > > surely possible, but it doesn't look very clean... And

Re: [I] Hook notification when you turn on/off (pause/unpause) a DAG from the UI [airflow]

2024-03-20 Thread via GitHub
Bowrna commented on issue #36516: URL: https://github.com/apache/airflow/issues/36516#issuecomment-2011325373 no i will have more time from this week and let me take a look at this @Harshith246 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] heartbeat recovery message [airflow]

2024-03-20 Thread via GitHub
Bowrna commented on PR #34457: URL: https://github.com/apache/airflow/pull/34457#issuecomment-2011247217 Test is failing in the health api call testcases. let me debug the issue and fix it by EOD -- This is an automated message from the Apache Git Service. To respond to the message, pleas

Re: [PR] Add `impersonation_scopes` to BigQuery [airflow]

2024-03-20 Thread via GitHub
ying-w commented on PR #38169: URL: https://github.com/apache/airflow/pull/38169#issuecomment-2011174045 I need some help figuring out how to write a test. I see that there is a cloud/operators/test_bigquery.py file with a `impersonation_chain` call but no `impersonation_chain` call in clou

Re: [PR] Fix test_auth.py [airflow]

2024-03-20 Thread via GitHub
Satoshi-Sh closed pull request #38359: Fix test_auth.py URL: https://github.com/apache/airflow/pull/38359 -- 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] Migrate to connexion v3 [airflow]

2024-03-20 Thread via GitHub
Satoshi-Sh commented on PR #37638: URL: https://github.com/apache/airflow/pull/37638#issuecomment-2011109441 I had the same issue with assertion code status 302 == 200. It seems the responses are after a user gets redirected to the targeted pages. I used the path to check if the redirect w

[PR] Fix test_auth.py [airflow]

2024-03-20 Thread via GitHub
Satoshi-Sh opened a new pull request, #38359: URL: https://github.com/apache/airflow/pull/38359 ## Updates > To avoid Error on "state_color_mapping" is undefined error I included `init_jinja_gloabals` in the minimal_app_for_api. > Assertion Error 200 == 302 on client_with_login

Re: [PR] feat: Add DbtCloudRetryJobOperator to retry failed dbt job [airflow]

2024-03-20 Thread via GitHub
andyguwc commented on PR #38001: URL: https://github.com/apache/airflow/pull/38001#issuecomment-2011079074 cc @Taragolis @hussein-awala @eladkal anyone can help review? Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

Re: [PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-20 Thread via GitHub
hussein-awala commented on PR #38358: URL: https://github.com/apache/airflow/pull/38358#issuecomment-2011014885 Need to test it before marking it as ready for review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] Add a migration script for encrypted trigger kwargs [airflow]

2024-03-20 Thread via GitHub
hussein-awala opened a new pull request, #38358: URL: https://github.com/apache/airflow/pull/38358 related: #38233 To avoid breaking the execution of created triggers after the upgrade to 2.9.0, we need to encrypt the `kwargs` during the migration process and decrypt them on downgrad

Re: [PR] Migrate to connexion v3 [airflow]

2024-03-20 Thread via GitHub
sudiptob2 commented on PR #37638: URL: https://github.com/apache/airflow/pull/37638#issuecomment-2010952631 Hi, @vincbeck Need a little bit of help here. As we are using `connexion_app.client` now, `anonymous_client` does not return `302`when we call `pool/delete/id` route. As a r

Re: [I] Wrong order of parameters in WebUI [airflow]

2024-03-20 Thread via GitHub
github-actions[bot] commented on issue #35944: URL: https://github.com/apache/airflow/issues/35944#issuecomment-2010947723 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

Re: [PR] k8s spark operator compatibility [airflow]

2024-03-20 Thread via GitHub
github-actions[bot] commented on PR #36268: URL: https://github.com/apache/airflow/pull/36268#issuecomment-2010947712 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] 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte (after upgrade to 2.8.2) [airflow]

2024-03-20 Thread via GitHub
github-actions[bot] commented on issue #37871: URL: https://github.com/apache/airflow/issues/37871#issuecomment-2010947659 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

Re: [PR] Update start.rst [airflow]

2024-03-20 Thread via GitHub
github-actions[bot] closed pull request #37098: Update start.rst URL: https://github.com/apache/airflow/pull/37098 -- 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: [PR] WIP/Idea: Pass task output as outlet to dataset trigger params [airflow]

2024-03-20 Thread via GitHub
boschglobal closed pull request #37888: WIP/Idea: Pass task output as outlet to dataset trigger params URL: https://github.com/apache/airflow/pull/37888 -- 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

Re: [PR] Fix EmrNotebookExecutionSensor broken test [airflow]

2024-03-20 Thread via GitHub
hussein-awala merged PR #38354: URL: https://github.com/apache/airflow/pull/38354 -- 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...@ai

(airflow) branch main updated: Fix EmrNotebookExecutionSensor broken test (#38354)

2024-03-20 Thread husseinawala
This is an automated email from the ASF dual-hosted git repository. husseinawala 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 952d04a3c1 Fix EmrNotebookExecutionSensor bro

Re: [PR] Activate ruff B007 to detect unused variables in the loop [airflow]

2024-03-20 Thread via GitHub
uranusjr commented on code in PR #38309: URL: https://github.com/apache/airflow/pull/38309#discussion_r1533064816 ## tests/providers/fab/auth_manager/test_security.py: ## @@ -997,7 +997,7 @@ def test_parent_dag_access_applies_to_subdag(app, security_manager, assert_user_

(airflow) branch constraints-2-9 updated: Updating constraints. Github run id:8364223672

2024-03-20 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-2-9 in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-2-9 by this push: new 5f41c8566b Updating const

[PR] Release notes for helm chart 1.13.1 [airflow]

2024-03-20 Thread via GitHub
jedcunningham opened a new pull request, #38356: URL: https://github.com/apache/airflow/pull/38356 (no comment) -- 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 unsubscri

Re: [I] triggerer : {serve_logs.py:103} WARNING - The signature of the request was wrong [airflow]

2024-03-20 Thread via GitHub
raphaelauv commented on issue #38352: URL: https://github.com/apache/airflow/issues/38352#issuecomment-2010886613 yes , thanks -- 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

Re: [I] triggerer : {serve_logs.py:103} WARNING - The signature of the request was wrong [airflow]

2024-03-20 Thread via GitHub
Taragolis commented on issue #38352: URL: https://github.com/apache/airflow/issues/38352#issuecomment-2010830392 > jwt.exceptions.InvalidSignatureError: Signature verification failed Did you set [`[webserver] secret_key`](https://airflow.apache.org/docs/apache-airflow/stable/configura

[PR] Fix EmrNotebookExecutionSensor broken test [airflow]

2024-03-20 Thread via GitHub
hussein-awala opened a new pull request, #38354: URL: https://github.com/apache/airflow/pull/38354 related: #38306 -- 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 unsubs

[I] triggerdagrunoperator wait_for_completion deferrable bugs [airflow]

2024-03-20 Thread via GitHub
raphaelauv opened a new issue, #38353: URL: https://github.com/apache/airflow/issues/38353 ### Apache Airflow version 2.8.3 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? using the triggerdagrunoperator with wait_for_c

[I] triggerer : {serve_logs.py:103} WARNING - The signature of the request was wrong [airflow]

2024-03-20 Thread via GitHub
raphaelauv opened a new issue, #38352: URL: https://github.com/apache/airflow/issues/38352 ### Apache Airflow version 2.8.3 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? ``` {serve_logs.py:103} WARNING - The signat

[PR] Fix deprecated apache.hive operators arguments in `MappedOperator` [airflow]

2024-03-20 Thread via GitHub
Taragolis opened a new pull request, #38351: URL: https://github.com/apache/airflow/pull/38351 similar to: https://github.com/apache/airflow/pull/38178 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/a

(airflow) branch main updated: Limit `cohere` to <5 (#38348)

2024-03-20 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis 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 3400dfe0be Limit `cohere` to <5 (#38348) 3400dfe

Re: [PR] Limit `cohere` to <5 [airflow]

2024-03-20 Thread via GitHub
Taragolis merged PR #38348: URL: https://github.com/apache/airflow/pull/38348 -- 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] Delete all old dag pages and redirect to grid view [airflow]

2024-03-20 Thread via GitHub
dstandish commented on PR #37988: URL: https://github.com/apache/airflow/pull/37988#issuecomment-2010728436 @bbovenzi i think this one really wants a news fragment. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

(airflow) branch main updated: Remove unused loop variable from airflow package (#38308)

2024-03-20 Thread husseinawala
This is an automated email from the ASF dual-hosted git repository. husseinawala 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 8e7b1add43 Remove unused loop variable from a

Re: [PR] Remove unused loop variable from airflow package [airflow]

2024-03-20 Thread via GitHub
hussein-awala merged PR #38308: URL: https://github.com/apache/airflow/pull/38308 -- 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...@ai

Re: [PR] Fix in decorators to log dag-id during delete dag-run action #36057 [airflow]

2024-03-20 Thread via GitHub
bbovenzi commented on PR #38192: URL: https://github.com/apache/airflow/pull/38192#issuecomment-2010693001 Do we use `AirflowModelView.delete` anywhere else? Or will `pk` always be a DagRun.id? -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Fix in decorators to log dag-id during delete dag-run action #36057 [airflow]

2024-03-20 Thread via GitHub
sachinp97 commented on PR #38192: URL: https://github.com/apache/airflow/pull/38192#issuecomment-2010676453 Updated the description to indicate the intent -- 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 t

Re: [PR] Limit `cohere` to <5 [airflow]

2024-03-20 Thread via GitHub
Taragolis commented on PR #38348: URL: https://github.com/apache/airflow/pull/38348#issuecomment-2010675015 - https://github.com/apache/airflow/issues/38349 - https://github.com/apache/airflow/issues/38350 -- This is an automated message from the Apache Git Service. To respond to the me

[I] Support `cohere` to 5+ [airflow]

2024-03-20 Thread via GitHub
Taragolis opened a new issue, #38349: URL: https://github.com/apache/airflow/issues/38349 ### Body Cohere v5.0.0 just released and seems like it might not work out of the box with [Cohere Provider ](https://airflow.apache.org/docs/apache-airflow-providers-cohere/stable/index.html)

Re: [PR] Limit `cohere` to <5 [airflow]

2024-03-20 Thread via GitHub
Taragolis commented on code in PR #38348: URL: https://github.com/apache/airflow/pull/38348#discussion_r1532881799 ## airflow/providers/cohere/hooks/cohere.py: ## @@ -53,7 +53,7 @@ def __init__( self.max_retries = max_retries @cached_property -def get_conn(se

Re: [PR] Limit `cohere` to <5 [airflow]

2024-03-20 Thread via GitHub
Taragolis commented on code in PR #38348: URL: https://github.com/apache/airflow/pull/38348#discussion_r1532881799 ## airflow/providers/cohere/hooks/cohere.py: ## @@ -53,7 +53,7 @@ def __init__( self.max_retries = max_retries @cached_property -def get_conn(se

[PR] Limit `cohere` to <5 [airflow]

2024-03-20 Thread via GitHub
Taragolis opened a new pull request, #38348: URL: https://github.com/apache/airflow/pull/38348 cohere 5.0.0 [just released](https://pypi.org/project/cohere/#history), and seems breaks mypy and most possible it is also brings some breaking changes ```console airflow/

Re: [I] "Queue up new tasks" equivalent in CLI / REST API [airflow]

2024-03-20 Thread via GitHub
qcha41 commented on issue #38341: URL: https://github.com/apache/airflow/issues/38341#issuecomment-2010644575 Duplicates of https://github.com/apache/airflow/issues/38342 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] NeptuneStopDbClusterOperator - Handle invalid cluster states [airflow]

2024-03-20 Thread via GitHub
Taragolis commented on code in PR #38287: URL: https://github.com/apache/airflow/pull/38287#discussion_r1532840938 ## airflow/providers/amazon/aws/operators/neptune.py: ## @@ -81,17 +85,86 @@ def __init__( self.delay = waiter_delay self.max_attempts = waiter_ma

(airflow) branch main updated: Revert CodeCell formatting changes (#38343)

2024-03-20 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 9f9ae33a8b Revert CodeCell formatting change

Re: [PR] Revert CodeCell formatting changes [airflow]

2024-03-20 Thread via GitHub
jedcunningham merged PR #38343: URL: https://github.com/apache/airflow/pull/38343 -- 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...@ai

[PR] Fix set deprecated amazon operators arguments in `MappedOperator` [airflow]

2024-03-20 Thread via GitHub
Taragolis opened a new pull request, #38346: URL: https://github.com/apache/airflow/pull/38346 similar to: https://github.com/apache/airflow/pull/38178 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/a

Re: [I] Add colour to logs [airflow]

2024-03-20 Thread via GitHub
romanzdk commented on issue #37443: URL: https://github.com/apache/airflow/issues/37443#issuecomment-2010551473 This would be super useful! I would keep info logs black though. Yellow warning and red error :) -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] Only show latest dataset event timestamp after last run [airflow]

2024-03-20 Thread via GitHub
jedcunningham commented on code in PR #38340: URL: https://github.com/apache/airflow/pull/38340#discussion_r1532783318 ## airflow/www/views.py: ## @@ -3695,8 +3695,16 @@ def next_run_datasets(self, dag_id): with create_session() as session: dag_model = DagM

(airflow-site) branch gh-pages updated (767736f2d5 -> 31ebeecf42)

2024-03-20 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch gh-pages in repository https://gitbox.apache.org/repos/asf/airflow-site.git discard 767736f2d5 Rewritten history to remove past gh-pages deployments new 31ebeecf42 Rewritten history to

Re: [PR] fix(google,log): Avoid log name overriding [airflow]

2024-03-20 Thread via GitHub
shahar1 commented on PR #38071: URL: https://github.com/apache/airflow/pull/38071#issuecomment-2010522117 > I updated the PR to print a warning when using the `name` constructor argument. However, my test seems to not pass... https://github.com/apache/airflow/actions/runs/8355631792/job/228

Re: [PR] Remove unused loop variable from airflow package [airflow]

2024-03-20 Thread via GitHub
hussein-awala commented on code in PR #38308: URL: https://github.com/apache/airflow/pull/38308#discussion_r1532772164 ## airflow/utils/db.py: ## @@ -90,7 +90,7 @@ "2.7.0": "405de8318b3a", "2.8.0": "10b52ebd31f7", "2.8.1": "88344c1d9134", -"2.9.0": "1fd5653699

[PR] Fix set deprecated slack operators arguments in `MappedOperator` [airflow]

2024-03-20 Thread via GitHub
Taragolis opened a new pull request, #38345: URL: https://github.com/apache/airflow/pull/38345 similar to: https://github.com/apache/airflow/pull/38178 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/a

Re: [PR] Migrate to connexion v3 [airflow]

2024-03-20 Thread via GitHub
vincbeck commented on PR #37638: URL: https://github.com/apache/airflow/pull/37638#issuecomment-2010475846 > After Jarek fixed the CI image, I'm able to see the same errors locally. > > I found "coroutine is not a callable error" on the Not Found(404) responses on `/auth/fab/v1` endpo

(airflow) branch main updated: Fix bash command formatting in docs (#38335)

2024-03-20 Thread vincbeck
This is an automated email from the ASF dual-hosted git repository. vincbeck 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 0f465c3c16 Fix bash command formatting in docs (#

Re: [PR] Fix bash command formatting in docs [airflow]

2024-03-20 Thread via GitHub
vincbeck merged PR #38335: URL: https://github.com/apache/airflow/pull/38335 -- 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

(airflow) branch main updated: Update dev/refresh_images.sh script with recent changes (#38331)

2024-03-20 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk 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 834917f58e Update dev/refresh_images.sh script with

Re: [PR] Migrate to connexion v3 [airflow]

2024-03-20 Thread via GitHub
Satoshi-Sh commented on PR #37638: URL: https://github.com/apache/airflow/pull/37638#issuecomment-2010465008 After Jarek fixed the CI image, I'm able to see the same errors locally. I found "coroutine is not a callable error" on the Not Found(404) responses on `/auth/fab/v1` endpoin

Re: [PR] Update dev/refresh_images.sh script with recent changes [airflow]

2024-03-20 Thread via GitHub
potiuk merged PR #38331: URL: https://github.com/apache/airflow/pull/38331 -- 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

(airflow-site) branch main updated: csp for video iframes fix retry in head-end (#984)

2024-03-20 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow-site.git The following commit(s) were added to refs/heads/main by this push: new ebacbf0773 csp for video iframes fix retry in

Re: [PR] Retry fixing CSP for video iframes in head-end [airflow-site]

2024-03-20 Thread via GitHub
potiuk merged PR #984: URL: https://github.com/apache/airflow-site/pull/984 -- 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.

(airflow) branch main updated: Adding max consecutive failed dag runs info in UI (#38229)

2024-03-20 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 513977690f Adding max consecutive failed dag runs

Re: [I] Adding max_consecutive_failed_dag_runs in UI for dag [airflow]

2024-03-20 Thread via GitHub
eladkal closed issue #38171: Adding max_consecutive_failed_dag_runs in UI for dag URL: https://github.com/apache/airflow/issues/38171 -- 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] Adding max consecutive failed dag runs info in UI [airflow]

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

Re: [PR] Replace to broad exceptions into the Core [airflow]

2024-03-20 Thread via GitHub
Taragolis closed pull request #38344: Replace to broad exceptions into the Core URL: https://github.com/apache/airflow/pull/38344 -- 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.

Re: [I] Support timezone-aware `time`s for `TimeSensor` and `TimeSensorAsync`. [airflow]

2024-03-20 Thread via GitHub
Taragolis commented on issue #38337: URL: https://github.com/apache/airflow/issues/38337#issuecomment-2010368889 > The timezone is completely ignored. Heh, that is what I exactly (or close) asked into the `#contributors` channel in [Slack](https://s.apache.org/airflow-slack) today: h

Re: [PR] NeptuneStopDbClusterOperator - Handle invalid cluster states [airflow]

2024-03-20 Thread via GitHub
ellisms commented on code in PR #38287: URL: https://github.com/apache/airflow/pull/38287#discussion_r1532635341 ## airflow/providers/amazon/aws/hooks/neptune.py: ## @@ -83,3 +89,32 @@ def get_cluster_status(self, cluster_id: str) -> str: :return: The status of the clus

Re: [PR] Remove unused loop variable from airflow package [airflow]

2024-03-20 Thread via GitHub
jedcunningham commented on code in PR #38308: URL: https://github.com/apache/airflow/pull/38308#discussion_r1532635320 ## airflow/utils/db.py: ## @@ -90,7 +90,7 @@ "2.7.0": "405de8318b3a", "2.8.0": "10b52ebd31f7", "2.8.1": "88344c1d9134", -"2.9.0": "1fd5653699

Re: [PR] Remove unused loop variable from airflow package [airflow]

2024-03-20 Thread via GitHub
jedcunningham commented on code in PR #38308: URL: https://github.com/apache/airflow/pull/38308#discussion_r1532634553 ## airflow/utils/db.py: ## @@ -90,7 +90,7 @@ "2.7.0": "405de8318b3a", "2.8.0": "10b52ebd31f7", "2.8.1": "88344c1d9134", -"2.9.0": "1fd5653699

Re: [PR] Remove unused loop variable from airflow package [airflow]

2024-03-20 Thread via GitHub
jedcunningham commented on code in PR #38308: URL: https://github.com/apache/airflow/pull/38308#discussion_r1532632207 ## airflow/utils/db.py: ## @@ -90,7 +90,7 @@ "2.7.0": "405de8318b3a", "2.8.0": "10b52ebd31f7", "2.8.1": "88344c1d9134", -"2.9.0": "1fd5653699

Re: [I] Status of testing of Apache Airflow 2.8.4rc1 [airflow]

2024-03-20 Thread via GitHub
potiuk commented on issue #38334: URL: https://github.com/apache/airflow/issues/38334#issuecomment-2010359144 Tested / verified all my changes. Particularly tested well the propagation of permissions (#38164) and the scheduler heartrate change (#37992) - in various scenarios and those fixes

[PR] Replace to broad exceptions into the Core [airflow]

2024-03-20 Thread via GitHub
Taragolis opened a new pull request, #38344: URL: https://github.com/apache/airflow/pull/38344 Replace extensive `raise Exception` by more generic and appropriate to the error. --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](h

(airflow) branch main updated: Revert Add TaskFail entries to Gantt chart (#38339)

2024-03-20 Thread bbovenzi
This is an automated email from the ASF dual-hosted git repository. bbovenzi 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 04c4b5c361 Revert Add TaskFail entries to Gantt c

Re: [PR] Revert Add TaskFail entries to Gantt chart [airflow]

2024-03-20 Thread via GitHub
bbovenzi merged PR #38339: URL: https://github.com/apache/airflow/pull/38339 -- 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

Re: [PR] NeptuneStopDbClusterOperator - Handle invalid cluster states [airflow]

2024-03-20 Thread via GitHub
ellisms commented on code in PR #38287: URL: https://github.com/apache/airflow/pull/38287#discussion_r1532625228 ## airflow/providers/amazon/aws/operators/neptune.py: ## @@ -81,17 +85,92 @@ def __init__( self.delay = waiter_delay self.max_attempts = waiter_max_

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
potiuk commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532619692 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes +

(airflow) branch main updated: Hide upstream dataset edges in dag graph (#38338)

2024-03-20 Thread bbovenzi
This is an automated email from the ASF dual-hosted git repository. bbovenzi 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 76295d3d25 Hide upstream dataset edges in dag gra

Re: [PR] Hide upstream dataset edges in dag graph [airflow]

2024-03-20 Thread via GitHub
bbovenzi merged PR #38338: URL: https://github.com/apache/airflow/pull/38338 -- 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

(airflow) branch v2-9-test updated (6b31b859ea -> 7f78b16870)

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git omit 6b31b859ea Update RELEASE_NOTES.rst new 7f78b16870 Update RELEASE_NOTES.rst This update added new r

(airflow) 01/01: Update RELEASE_NOTES.rst

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 7f78b16870f4844e4bdecbceffc9565b9caedc3a Author: Ephraim Anierobi AuthorDate: Wed Mar 20 17:00:37 2024 +010

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
potiuk commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532610461 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,249 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes +

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
ephraimbuddy commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532610390 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes

Re: [PR] Create AWS auth manager documentation. Part: setup identity center [airflow]

2024-03-20 Thread via GitHub
o-nikolas commented on code in PR #38273: URL: https://github.com/apache/airflow/pull/38273#discussion_r1532595636 ## docs/apache-airflow-providers-amazon/auth-manager/setup/identity-center.rst: ## @@ -0,0 +1,142 @@ + .. Licensed to the Apache Software Foundation (ASF) under one

[PR] Revert CodeCell formatting changes [airflow]

2024-03-20 Thread via GitHub
bbovenzi opened a new pull request, #38343: URL: https://github.com/apache/airflow/pull/38343 After we fixed the Audit Log extras. We don't need this change anymore. We also were missing important check to make sure extra actually was json, which older audit log extra wouldn't be. ---

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
jedcunningham commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532606784 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Change

(airflow) branch v2-9-test updated (a8f154d6ff -> 6b31b859ea)

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git omit a8f154d6ff Update RELEASE_NOTES.rst new 6b31b859ea Update RELEASE_NOTES.rst This update added new r

(airflow) 01/01: Update RELEASE_NOTES.rst

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 6b31b859ea217709a5912b349e6defef008eb83a Author: Ephraim Anierobi AuthorDate: Wed Mar 20 17:00:37 2024 +010

Re: [I] "Queue up new tasks" equivalent in CLI / REST API [airflow]

2024-03-20 Thread via GitHub
qcha41 closed issue #38341: "Queue up new tasks" equivalent in CLI / REST API URL: https://github.com/apache/airflow/issues/38341 -- 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.

[I] "Queue up new tasks" equivalent in CLI / REST API [airflow]

2024-03-20 Thread via GitHub
qcha41 opened a new issue, #38342: URL: https://github.com/apache/airflow/issues/38342 ### Description Suggestion to add a new command in the CLI / REST API to reproduce the "Queue up new tasks" action that can be trigger within the UI, with start_date and end_date option to run this

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
potiuk commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532601999 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes +

(airflow) 02/02: Update RELEASE_NOTES.rst

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit a8f154d6ffc4be0e88a674b08c397ebd6d46c192 Author: Ephraim Anierobi AuthorDate: Wed Mar 20 17:00:37 2024 +010

(airflow) 01/02: Update version to 2.9.0

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 7e46acb11c21bf944ad77b0293258fdb932197da Author: Ephraim Anierobi AuthorDate: Wed Mar 20 16:59:27 2024 +010

(airflow) branch v2-9-test updated (5995c74055 -> a8f154d6ff)

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git omit 5995c74055 Apply suggestions from code review omit fca671adec Update RELEASE_NOTES.rst omit b954d

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
potiuk commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532599531 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes +

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
potiuk commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532598229 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes +

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
ephraimbuddy commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532593841 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes

[I] "Queue up new tasks" equivalent in CLI / REST API [airflow]

2024-03-20 Thread via GitHub
qcha41 opened a new issue, #38341: URL: https://github.com/apache/airflow/issues/38341 ### Description Suggestion to add a new command in the CLI / REST API to reproduce the "Queue up new tasks" action that can be trigger within the UI, with start_date and end_date option to run this

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
eladkal commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532587396 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes +^^^

Re: [PR] Add check in AWS auth manager to check if the Amazon Verified Permissions schema is up to date [airflow]

2024-03-20 Thread via GitHub
vincbeck commented on code in PR #38333: URL: https://github.com/apache/airflow/pull/38333#discussion_r1532587019 ## airflow/providers/amazon/aws/auth_manager/avp/facade.py: ## @@ -222,6 +224,19 @@ def get_batch_is_authorized_single_result( ) raise AirflowExcep

Re: [PR] Create AWS auth manager documentation. Part: setup identity center [airflow]

2024-03-20 Thread via GitHub
vincbeck commented on PR #38273: URL: https://github.com/apache/airflow/pull/38273#issuecomment-2010304495 See the generated documentation below: - [Configure AWS IAM Identity Center — apache-airflow-providers-amazon Documentation.pdf](https://github.com/apache/airflow/files/14670907/

[PR] Only show latest dataset event timestamp after last run [airflow]

2024-03-20 Thread via GitHub
bbovenzi opened a new pull request, #38340: URL: https://github.com/apache/airflow/pull/38340 In the next run dataset modal, "Last Update" can be confusing since the dataset update events could be irrelevant to the next dag run. Therefore, we should only show the last update event that has

(airflow) branch v2-9-test updated (fca671adec -> 5995c74055)

2024-03-20 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git from fca671adec Update RELEASE_NOTES.rst add 5995c74055 Apply suggestions from code review No new revisi

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-03-20 Thread via GitHub
potiuk commented on code in PR #38325: URL: https://github.com/apache/airflow/pull/38325#discussion_r1532568842 ## RELEASE_NOTES.rst: ## @@ -21,6 +21,276 @@ .. towncrier release notes start +Airflow 2.9.0 (2024-03-28) +-- + +Significant Changes +

  1   2   3   >