Re: [I] triggerer block_watchdog is crying "wolf!" [airflow]

2023-12-28 Thread via GitHub
potiuk commented on issue #35806: URL: https://github.com/apache/airflow/issues/35806#issuecomment-1871820163 Hey @dlesco - any progres with 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

[I] Empty Details tab after upgrading to Airflow 2.8.0 [airflow]

2023-12-28 Thread via GitHub
csp33 opened a new issue, #36481: URL: https://github.com/apache/airflow/issues/36481 ### Apache Airflow version 2.8.0 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? After upgrading to Airflow 2.8.0, I've found out tha

Re: [I] ERROR: could not queue task TaskInstanceKey [airflow]

2023-12-28 Thread via GitHub
github-actions[bot] closed issue #20047: ERROR: could not queue task TaskInstanceKey URL: https://github.com/apache/airflow/issues/20047 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [I] ERROR: could not queue task TaskInstanceKey [airflow]

2023-12-28 Thread via GitHub
github-actions[bot] commented on issue #20047: URL: https://github.com/apache/airflow/issues/20047#issuecomment-1871796066 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
amoghrajesh commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1438048766 ## CONTRIBUTING.rst: ## @@ -515,14 +647,13 @@ these guidelines: useful features from newer versions. - Adhere to guidelines for commit messages describe

[PR] Add branching based on mapped task group example to dynamic-task-mapping.rst [airflow]

2023-12-28 Thread via GitHub
RNHTTR opened a new pull request, #36480: URL: https://github.com/apache/airflow/pull/36480 Based on trying to solve [this stack overflow question](https://stackoverflow.com/questions/77730116/branching-not-working-in-airflow-as-expected/77730300#77730300), it seems impossible to reliably b

Re: [I] Status of testing Providers that were prepared on December 28, 2023 [airflow]

2023-12-28 Thread via GitHub
dirrao commented on issue #36467: URL: https://github.com/apache/airflow/issues/36467#issuecomment-1871728621 I have verified the following #36433 and #36431. They looks good to me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] added cli command to list auth managers under `airflow providers` [airflow]

2023-12-28 Thread via GitHub
Lee-W commented on code in PR #36445: URL: https://github.com/apache/airflow/pull/36445#discussion_r1437975093 ## tests/always/test_providers_manager.py: ## @@ -384,6 +384,11 @@ def test_notification(self): notification_class_names = list(provider_manager.notification)

Re: [PR] Deprecate AutoMLTrainModelOperator for Vision and Video [airflow]

2023-12-28 Thread via GitHub
Lee-W commented on code in PR #36473: URL: https://github.com/apache/airflow/pull/36473#discussion_r1437972926 ## tests/system/providers/google/cloud/automl/example_automl_video_intelligence_classification.py: ## @@ -73,73 +72,88 @@ schedule="@once", start_date=datetim

Re: [I] get_current_context not present in user_defined_macros [airflow]

2023-12-28 Thread via GitHub
messi-gao commented on issue #33217: URL: https://github.com/apache/airflow/issues/33217#issuecomment-1871690456 @ivan-afonichkin I found that this issue was fixed at 2.7.1 version.is there a way to fix this issue under 2.7.1 version? -- This is an automated message from the Apache Git Se

Re: [I] Requests/limits in custom-values.yaml ignored [airflow]

2023-12-28 Thread via GitHub
github-actions[bot] commented on issue #35497: URL: https://github.com/apache/airflow/issues/35497#issuecomment-1871636708 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [DRAFT] Unified database interface [airflow]

2023-12-28 Thread via GitHub
github-actions[bot] commented on PR #35598: URL: https://github.com/apache/airflow/pull/35598#issuecomment-1871636683 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] triggerer block_watchdog is crying "wolf!" [airflow]

2023-12-28 Thread via GitHub
github-actions[bot] commented on issue #35806: URL: https://github.com/apache/airflow/issues/35806#issuecomment-1871636670 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: [I] When setup airflow database backend (postgressql), webserver is close. [airflow]

2023-12-28 Thread via GitHub
github-actions[bot] commented on issue #35938: URL: https://github.com/apache/airflow/issues/35938#issuecomment-1871636651 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

Re: [I] When setup airflow database backend (postgressql), webserver is close. [airflow]

2023-12-28 Thread via GitHub
github-actions[bot] closed issue #35938: When setup airflow database backend (postgressql), webserver is close. URL: https://github.com/apache/airflow/issues/35938 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [I] Airflow Scheduler with Kubernetes Executor has errors in logs and stuck slots with no running tasks [airflow]

2023-12-28 Thread via GitHub
crabio commented on issue #36478: URL: https://github.com/apache/airflow/issues/36478#issuecomment-1871513362 Discussed in https://github.com/apache/airflow/discussions/35426 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[I] Airflow Scheduler with Kubernetes Executor has errors in logs and stuck slots with no running tasks [airflow]

2023-12-28 Thread via GitHub
crabio opened a new issue, #36478: URL: https://github.com/apache/airflow/issues/36478 ### Apache Airflow version 2.8.0 ### If "Other Airflow 2 version" selected, which one? 2.7.3 ### What happened? I found race condition between 2 schedulers. Schedul

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
ZuhairORZaki commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871496890 > In addition, the is no possible to create DAG ID with this `` name because it would failed on validation > > https://github.com/apache/airflow/blob/e9ba37bb58da0e3d6739ec06

Re: [PR] fix: Repair incorrect display of the deferrable operators in the documentation [airflow]

2023-12-28 Thread via GitHub
potiuk merged PR #36476: URL: https://github.com/apache/airflow/pull/36476 -- 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) branch main updated: fix: Repair defferable operator list in docs (#36476)

2023-12-28 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 7cd326ccbd fix: Repair defferable operator list in

(airflow) branch main updated: Improve generation of "Status of testing" issue (#36470)

2023-12-28 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 5d88f6f9a4 Improve generation of "Status of testing

Re: [PR] Improve generation of "Status of testing" issue [airflow]

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

Re: [I] Kubernetes Executor wastefully creates pods for EmptyOperator tasks [airflow]

2023-12-28 Thread via GitHub
potiuk closed issue #36463: Kubernetes Executor wastefully creates pods for EmptyOperator tasks URL: https://github.com/apache/airflow/issues/36463 -- 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

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
Taragolis commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871430550 In addition, the is no possible to create DAG ID with this `` name because it would failed on validation https://github.com/apache/airflow/blob/e9ba37bb58da0e3d6739ec063f7160f50

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on PR #36472: URL: https://github.com/apache/airflow/pull/36472#issuecomment-1871421861 First pass done :). Thanks @Lee-W, @amoghrajesh and @pankajkoti for the dilligence :) -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437836310 ## CONTRIBUTING.rst: ## @@ -515,14 +647,13 @@ these guidelines: useful features from newer versions. - Adhere to guidelines for commit messages described in

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437836310 ## CONTRIBUTING.rst: ## @@ -515,14 +647,13 @@ these guidelines: useful features from newer versions. - Adhere to guidelines for commit messages described in

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437835672 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guideli

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437835552 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437834726 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437835044 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437834458 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437834242 ## CONTRIBUTING.rst: ## @@ -453,6 +453,29 @@ Step 4: Prepare PR PR guidelines described in `pull request guidelines <#pull-request-guidelines>`_. Create Pull

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437833962 ## .asf.yaml: ## @@ -44,37 +44,47 @@ github: main: required_pull_request_reviews: required_approving_review_count: 1 +required_linear_hist

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437833516 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guideli

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437833370 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guideli

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437832940 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guideli

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437832615 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guideli

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437832216 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guideli

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437832024 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437831812 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437831676 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437831568 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437829131 ## CONTRIBUTING.rst: ## @@ -453,6 +453,29 @@ Step 4: Prepare PR PR guidelines described in `pull request guidelines <#pull-request-guidelines>`_. Create Pull

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437829336 ## CONTRIBUTING.rst: ## @@ -453,6 +453,29 @@ Step 4: Prepare PR PR guidelines described in `pull request guidelines <#pull-request-guidelines>`_. Create Pull

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437822958 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merging

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437820834 ## .asf.yaml: ## @@ -44,37 +44,47 @@ github: main: required_pull_request_reviews: required_approving_review_count: 1 +required_linear_hist

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437819923 ## .asf.yaml: ## @@ -44,37 +44,47 @@ github: main: required_pull_request_reviews: required_approving_review_count: 1 +required_linear_hist

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437817965 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guideli

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
potiuk commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871381217 > > NIT: can you please add a unit test showing the failure scenario? > > Seems the test failed because of using `re` module instead of `re2`. Yes. We require re2 globally in

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
amoghrajesh commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437793243 ## CONTRIBUTING.rst: ## @@ -453,6 +453,29 @@ Step 4: Prepare PR PR guidelines described in `pull request guidelines <#pull-request-guidelines>`_. Create

[PR] fix: Repair incorrect display of the deferrable operators in the documentation [airflow]

2023-12-28 Thread via GitHub
kacpermuda opened a new pull request, #36476: URL: https://github.com/apache/airflow/pull/36476 The current issue with the incorrect display of the list of deferrable operators in the documentation [here](https://airflow.apache.org/docs/apache-airflow-providers/core-extensio

Re: [I] During scheduler prcoess restart, the scheduler is launching the duplicate worker pod thought the worker pod already exists and in queued state [airflow]

2023-12-28 Thread via GitHub
dirrao commented on issue #35676: URL: https://github.com/apache/airflow/issues/35676#issuecomment-1871339541 Its not working in airflow 2.3.3 and Kubernetes provider 7.9.0. I verified latest airflow 2.3.3 and Kubernetes provider 7.9.0. It is working as expected. So, hence closing this tick

(airflow) branch constraints-main updated: Updating constraints. Github run id:7349138665

2023-12-28 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 4b0b598a2b Updating con

Re: [I] Kubernetes Executor wastefully creates pods for EmptyOperator tasks [airflow]

2023-12-28 Thread via GitHub
dirrao commented on issue #36463: URL: https://github.com/apache/airflow/issues/36463#issuecomment-1871324588 @buu-nguyen Can you share the airflow version and Kubernetes provider version where you are seeing this issue? I haven't seen this issue in the latest version. -- This is an a

Re: [PR] Add openlineage docs ext with list of supported operators and hooks [airflow]

2023-12-28 Thread via GitHub
kacpermuda commented on code in PR #36311: URL: https://github.com/apache/airflow/pull/36311#discussion_r1437757108 ## docs/exts/operators_and_hooks_ref.py: ## @@ -215,6 +215,136 @@ def _render_deferrable_operator_content(*, header_separator: str): return _render_template(

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
ZuhairORZaki commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871306528 > NIT: can you please add a unit test showing the failure scenario? Seems the test failed because of using `re` module instead of `re2`. -- This is an automated message fro

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
ZuhairORZaki commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871294907 > Have no idea how to reproduce it. > > In breeze this one link > > ``` > http://127.0.0.1:28080/confirm?task_id=runme_0&dag_id=%3Cimg%20src%3D%27x%27%20onerror%3D%

(airflow) branch main updated: Handle list like input objects in weavaite's 'create_or_replace_document_objects' hook method (#36475)

2023-12-28 Thread utkarsharma
This is an automated email from the ASF dual-hosted git repository. utkarsharma 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 0a741b2fe6 Handle list like input objects in

Re: [PR] Handle list like input objects in weavaite's 'create_or_replace_document_objects' hook method [airflow]

2023-12-28 Thread via GitHub
utkarsharma2 merged PR #36475: URL: https://github.com/apache/airflow/pull/36475 -- 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...@air

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
pankajkoti commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437734498 ## CONTRIBUTING.rst: ## @@ -464,9 +487,97 @@ Step 5: Pass PR Review Note that committers will use **Squash and Merge** instead of **Rebase and Merge** when merg

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
ZuhairORZaki commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871283818 > Have no idea how to reproduce it. > > In breeze this one link > > ``` > http://127.0.0.1:28080/confirm?task_id=runme_0&dag_id=%3Cimg%20src%3D%27x%27%20onerror%3D%

Re: [PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36470: URL: https://github.com/apache/airflow/pull/36470#discussion_r1437730829 ## dev/README_RELEASE_PROVIDER_PACKAGES.md: ## @@ -531,26 +531,38 @@ git push --set-upstream origin "${branch}" Create a GitHub issue with the content generated via

Re: [PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36470: URL: https://github.com/apache/airflow/pull/36470#discussion_r1437730566 ## dev/README_RELEASE_PROVIDER_PACKAGES.md: ## @@ -531,26 +531,38 @@ git push --set-upstream origin "${branch}" Create a GitHub issue with the content generated via

Re: [PR] Handle list like input objects [airflow]

2023-12-28 Thread via GitHub
utkarsharma2 commented on code in PR #36475: URL: https://github.com/apache/airflow/pull/36475#discussion_r1437722159 ## airflow/providers/weaviate/hooks/weaviate.py: ## @@ -21,7 +21,7 @@ import json import warnings from functools import cached_property -from typing import TY

Re: [PR] Handle list like input objects [airflow]

2023-12-28 Thread via GitHub
utkarsharma2 commented on code in PR #36475: URL: https://github.com/apache/airflow/pull/36475#discussion_r1437722159 ## airflow/providers/weaviate/hooks/weaviate.py: ## @@ -21,7 +21,7 @@ import json import warnings from functools import cached_property -from typing import TY

Re: [PR] Handle list like input objects [airflow]

2023-12-28 Thread via GitHub
pankajkoti commented on code in PR #36475: URL: https://github.com/apache/airflow/pull/36475#discussion_r1437721273 ## airflow/providers/weaviate/hooks/weaviate.py: ## @@ -21,7 +21,7 @@ import json import warnings from functools import cached_property -from typing import TYPE

Re: [PR] Handle list like input objects [airflow]

2023-12-28 Thread via GitHub
pankajkoti commented on code in PR #36475: URL: https://github.com/apache/airflow/pull/36475#discussion_r1437721273 ## airflow/providers/weaviate/hooks/weaviate.py: ## @@ -21,7 +21,7 @@ import json import warnings from functools import cached_property -from typing import TYPE

Re: [PR] Handle list like input objects [airflow]

2023-12-28 Thread via GitHub
utkarsharma2 commented on code in PR #36475: URL: https://github.com/apache/airflow/pull/36475#discussion_r1437720676 ## airflow/providers/weaviate/hooks/weaviate.py: ## @@ -21,7 +21,7 @@ import json import warnings from functools import cached_property -from typing import TY

Re: [PR] Handle list like input objects [airflow]

2023-12-28 Thread via GitHub
pankajastro commented on code in PR #36475: URL: https://github.com/apache/airflow/pull/36475#discussion_r1437719633 ## airflow/providers/weaviate/hooks/weaviate.py: ## @@ -21,7 +21,7 @@ import json import warnings from functools import cached_property -from typing import TYP

[PR] Handle list like input objects [airflow]

2023-12-28 Thread via GitHub
utkarsharma2 opened a new pull request, #36475: URL: https://github.com/apache/airflow/pull/36475 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidel

(airflow) branch main updated: Add utkarsharma2 to committers list (#36474)

2023-12-28 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 e3fb20d358 Add utkarsharma2 to committers lis

Re: [PR] Add utkarsharma2 to committers list [airflow]

2023-12-28 Thread via GitHub
hussein-awala merged PR #36474: URL: https://github.com/apache/airflow/pull/36474 -- 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] Add utkarsharma2 to committers list [airflow]

2023-12-28 Thread via GitHub
hussein-awala opened a new pull request, #36474: URL: https://github.com/apache/airflow/pull/36474 (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: [PR] Implement Google Analytics Admin (GA4) operators [airflow]

2023-12-28 Thread via GitHub
VladaZakharova commented on PR #36276: URL: https://github.com/apache/airflow/pull/36276#issuecomment-1871229653 Hi @potiuk ! Could you please take a look on these changes? We can't understand why there is this problem with spelling :D As a master of Sphinx, maybe you can share your id

[PR] Deprecate AutoMLTrainModelOperator for Vision and Video [airflow]

2023-12-28 Thread via GitHub
VladaZakharova opened a new pull request, #36473: URL: https://github.com/apache/airflow/pull/36473 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guid

Re: [PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36470: URL: https://github.com/apache/airflow/pull/36470#discussion_r1437682456 ## dev/README_RELEASE_PROVIDER_PACKAGES.md: ## @@ -531,26 +531,38 @@ git push --set-upstream origin "${branch}" Create a GitHub issue with the content generated via

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
Taragolis commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871198530 Have no idea how to reproduce it. In breeze this one link ``` http://127.0.0.1:28080/confirm?task_id=runme_0&dag_id=%3Cimg%20src%3D%27x%27%20onerror%3D%27alert%281%29%27%3

Re: [PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
Lee-W commented on code in PR #36472: URL: https://github.com/apache/airflow/pull/36472#discussion_r1437680615 ## CONTRIBUTING.rst: ## @@ -475,38 +586,59 @@ Pull Request Guidelines Before you submit a pull request (PR) from your forked repo, check that it meets these guidelin

Re: [PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
potiuk commented on code in PR #36470: URL: https://github.com/apache/airflow/pull/36470#discussion_r1437678832 ## dev/README_RELEASE_PROVIDER_PACKAGES.md: ## @@ -531,26 +531,38 @@ git push --set-upstream origin "${branch}" Create a GitHub issue with the content generated via

Re: [PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
potiuk commented on PR #36470: URL: https://github.com/apache/airflow/pull/36470#issuecomment-1871192408 > BTW we seems to have some differences between issue generation of providers and of core. In core it also generates the issue with link to the PR (calling for testing to both issue auth

[PR] Review and update our Pull Request review process and policy [airflow]

2023-12-28 Thread via GitHub
potiuk opened a new pull request, #36472: URL: https://github.com/apache/airflow/pull/36472 While applying result of the discussion about "resolving conversation" I have found that it would be good - while enabling it to test it to describe a bit better and explain some of the things we lea

[I] Task is retried when it is setting failed manually [airflow]

2023-12-28 Thread via GitHub
antoniocorralsierra opened a new issue, #36471: URL: https://github.com/apache/airflow/issues/36471 ### Apache Airflow version Other Airflow 2 version (please specify below) ### If "Other Airflow 2 version" selected, which one? 2.7.2 ### What happened? I hav

Re: [PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
eladkal commented on code in PR #36470: URL: https://github.com/apache/airflow/pull/36470#discussion_r1437653089 ## dev/README_RELEASE_PROVIDER_PACKAGES.md: ## @@ -531,26 +531,38 @@ git push --set-upstream origin "${branch}" Create a GitHub issue with the content generated vi

Re: [PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
amoghrajesh commented on code in PR #36470: URL: https://github.com/apache/airflow/pull/36470#discussion_r1437615651 ## dev/README_RELEASE_PROVIDER_PACKAGES.md: ## @@ -531,26 +531,38 @@ git push --set-upstream origin "${branch}" Create a GitHub issue with the content generate

(airflow) branch main updated: Use base aws classes in AWS Step Functions Operators/Sensors/Triggers (#36468)

2023-12-28 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 22294abf68 Use base aws classes in AWS Step Functi

Re: [PR] Use base aws classes in AWS Step Functions Operators/Sensors/Triggers [airflow]

2023-12-28 Thread via GitHub
eladkal merged PR #36468: URL: https://github.com/apache/airflow/pull/36468 -- 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] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
eladkal commented on PR #36470: URL: https://github.com/apache/airflow/pull/36470#issuecomment-1871117424 BTW we seems to have some differences between issue generation of providers and of core. In core it also generates the issue with link to the PR (calling for testing to both issue autho

Re: [I] Status of testing Providers that were prepared on December 28, 2023 [airflow]

2023-12-28 Thread via GitHub
Lee-W commented on issue #36467: URL: https://github.com/apache/airflow/issues/36467#issuecomment-1871099033 https://github.com/apache/airflow/pull/36417 and https://github.com/apache/airflow/pull/36363 also verified -- This is an automated message from the Apache Git Service. To respond

Re: [PR] Fix XSS weakness in airflow/www/views.py [airflow]

2023-12-28 Thread via GitHub
potiuk commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871096264 NIT: can you please add a unit test showing the failure scenario? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [I] Status of testing Providers that were prepared on December 28, 2023 [airflow]

2023-12-28 Thread via GitHub
Lee-W commented on issue #36467: URL: https://github.com/apache/airflow/issues/36467#issuecomment-1871047696 Tested our example DAGs with the following providers without encountering issues * apache-airflow-providers-amazon==8.15.0rc1 * apache-airflow-providers-cncf-kubernetes==7.1

Re: [I] Status of testing Providers that were prepared on December 28, 2023 [airflow]

2023-12-28 Thread via GitHub
Taragolis commented on issue #36467: URL: https://github.com/apache/airflow/issues/36467#issuecomment-1871045644 Check my changes: - https://github.com/apache/airflow/pull/36447 : Works as expected - https://github.com/apache/airflow/pull/36434 : Internal changes: comments added -

[PR] Improve generation of "Status of testing" issue [airflow]

2023-12-28 Thread via GitHub
potiuk opened a new pull request, #36470: URL: https://github.com/apache/airflow/pull/36470 This PR improves/simplifies the process of issue generation when provider package rc candidates are prepared for voting. It improves the commmand to generate the issue and makes it simpler (les

Re: [PR] Fix XSS weakness in method confirm [airflow]

2023-12-28 Thread via GitHub
boring-cyborg[bot] commented on PR #36469: URL: https://github.com/apache/airflow/pull/36469#issuecomment-1871028511 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

[PR] Fix XSS weakness in method confirm [airflow]

2023-12-28 Thread via GitHub
ZuhairORZaki opened a new pull request, #36469: URL: https://github.com/apache/airflow/pull/36469 While triaging your project, our bug fixing tool generated the following message- In file: `views.py`, there is a method that is vulnerable to XSS which can compromise any cookies, sessi

(airflow) branch main updated: Fix preparing rc candidates for providers (#36465)

2023-12-28 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 a3e5a971ed Fix preparing rc candidates for provider

Re: [PR] Fix preparing rc candidates for providers [airflow]

2023-12-28 Thread via GitHub
potiuk merged PR #36465: URL: https://github.com/apache/airflow/pull/36465 -- 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) branch main updated: Remove pypitest from the release process (#36466)

2023-12-28 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 c16b421872 Remove pypitest from the release p

Re: [PR] Remove pypitest from the release process [airflow]

2023-12-28 Thread via GitHub
hussein-awala merged PR #36466: URL: https://github.com/apache/airflow/pull/36466 -- 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] Use base aws classes in AWS Step Functions Operators/Sensors/Triggers [airflow]

2023-12-28 Thread via GitHub
Taragolis opened a new pull request, #36468: URL: https://github.com/apache/airflow/pull/36468 related: #35278 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-re

  1   2   >