[GitHub] [airflow] vedantlodha commented on a diff in pull request #29228: Fix dag run trigger with a note.

2023-01-29 Thread via GitHub
vedantlodha commented on code in PR #29228: URL: https://github.com/apache/airflow/pull/29228#discussion_r1090270676 ## airflow/api_connexion/endpoints/dag_run_endpoint.py: ## @@ -319,6 +319,15 @@ def post_dag_run(*, dag_id: str, session: Session = NEW_SESSION) -> APIResponse:

[GitHub] [airflow] uranusjr commented on a diff in pull request #29203: docs: Add trigger process missing from Airflow docker

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29203: URL: https://github.com/apache/airflow/pull/29203#discussion_r1090262772 ## docs/apache-airflow/howto/docker-compose/index.rst: ## @@ -68,6 +68,7 @@ This file contains several service definitions: task instances once their dependencies

[GitHub] [airflow] uranusjr commented on a diff in pull request #29203: docs: Add trigger process missing from Airflow docker

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29203: URL: https://github.com/apache/airflow/pull/29203#discussion_r1090262772 ## docs/apache-airflow/howto/docker-compose/index.rst: ## @@ -68,6 +68,7 @@ This file contains several service definitions: task instances once their dependencies

[GitHub] [airflow] uranusjr commented on a diff in pull request #29191: Add Documentation for notification feature extension

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29191: URL: https://github.com/apache/airflow/pull/29191#discussion_r1090260866 ## docs/apache-airflow/howto/notifications.rst: ## @@ -0,0 +1,34 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more contributor license

[GitHub] [airflow] uranusjr commented on a diff in pull request #29191: Add Documentation for notification feature extension

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29191: URL: https://github.com/apache/airflow/pull/29191#discussion_r1090260306 ## docs/apache-airflow/howto/notifications.rst: ## @@ -0,0 +1,34 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more contributor license

[GitHub] [airflow] uranusjr commented on a diff in pull request #29191: Add Documentation for notification feature extension

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29191: URL: https://github.com/apache/airflow/pull/29191#discussion_r1090259219 ## docs/apache-airflow/howto/notifications.rst: ## @@ -0,0 +1,34 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more contributor license

[GitHub] [airflow] uranusjr commented on a diff in pull request #29191: Add Documentation for notification feature extension

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29191: URL: https://github.com/apache/airflow/pull/29191#discussion_r1090258298 ## docs/apache-airflow/howto/notifications.rst: ## @@ -0,0 +1,34 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more contributor license

[GitHub] [airflow] uranusjr commented on a diff in pull request #29191: Add Documentation for notification feature extension

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29191: URL: https://github.com/apache/airflow/pull/29191#discussion_r1090257368 ## docs/apache-airflow-providers/core-extensions/notifications.rst: ## @@ -0,0 +1,26 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29228: Fix dag run trigger with a note.

2023-01-29 Thread via GitHub
ephraimbuddy commented on code in PR #29228: URL: https://github.com/apache/airflow/pull/29228#discussion_r1090219642 ## airflow/api_connexion/endpoints/dag_run_endpoint.py: ## @@ -319,6 +319,15 @@ def post_dag_run(*, dag_id: str, session: Session = NEW_SESSION) -> APIResponse:

[GitHub] [airflow-client-python] ephraimbuddy opened a new pull request, #67: Update the signing command

2023-01-29 Thread via GitHub
ephraimbuddy opened a new pull request, #67: URL: https://github.com/apache/airflow-client-python/pull/67 -amor was used instead of --amor -- 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 spec

[airflow-client-python] 01/01: Update the signing command

2023-01-29 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch update-sign in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git commit 0c063c264913348593c5a86a34493d65e5724618 Author: Ephraim Anierobi AuthorDate: Mon Jan 30 07

[airflow-client-python] branch update-sign created (now 0c063c2)

2023-01-29 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch update-sign in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git at 0c063c2 Update the signing command This branch includes the following new commits:

[GitHub] [airflow-client-go] ephraimbuddy opened a new pull request, #37: Update the signing command

2023-01-29 Thread via GitHub
ephraimbuddy opened a new pull request, #37: URL: https://github.com/apache/airflow-client-go/pull/37 -amor was used instead of --amor. Also added the executable bit to dev/sign.sh Thanks, @potiuk for the `-amor/--amor` explanation! -- This is an automated message from the Ap

[airflow-client-go] 01/01: Update the signing command

2023-01-29 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch update-sign in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git commit a140baa68c0c6ee565cb17fe1e70ae3a58144131 Author: Ephraim Anierobi AuthorDate: Mon Jan 30 07:04:

[airflow-client-go] branch update-sign updated (fb75341 -> a140baa)

2023-01-29 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch update-sign in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git discard fb75341 Update the signing command new a140baa Update the signing command This update

[GitHub] [airflow] amarlearning commented on a diff in pull request #29203: docs: Add trigger process missing from Airflow docker

2023-01-29 Thread via GitHub
amarlearning commented on code in PR #29203: URL: https://github.com/apache/airflow/pull/29203#discussion_r1090204810 ## docs/apache-airflow/howto/docker-compose/index.rst: ## @@ -68,6 +68,7 @@ This file contains several service definitions: task instances once their dependen

[airflow-client-go] 01/01: Update the signing command

2023-01-29 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch update-sign in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git commit fb753419abcc0b5bc6f3604ce3c3b2fdbff63c69 Author: Ephraim Anierobi AuthorDate: Mon Jan 30 07:04:

[airflow-client-go] branch update-sign created (now fb75341)

2023-01-29 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch update-sign in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git at fb75341 Update the signing command This branch includes the following new commits: ne

[GitHub] [airflow] raagn08 commented on issue #19670: Configure return value logging for a DecoratedOperator

2023-01-29 Thread via GitHub
raagn08 commented on issue #19670: URL: https://github.com/apache/airflow/issues/19670#issuecomment-1408041504 Hi, apologies if this is a redundant question but is there a workaround for now to disable XCOM/Taskflow returned values in logs? Or is there an existing solution that I might be u

[GitHub] [airflow] vedantlodha opened a new pull request, #29228: Fix dag run trigger with a note.

2023-01-29 Thread via GitHub
vedantlodha opened a new pull request, #29228: URL: https://github.com/apache/airflow/pull/29228 This PR allows dag run trigger with a note, which, at the moment gives a 400. Closes: #28825 --- **^ Add meaningful description above** Read the **[Pull Request Gu

[GitHub] [airflow] uranusjr commented on a diff in pull request #29206: Add non login-password auth support for SimpleHttpOpeator

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29206: URL: https://github.com/apache/airflow/pull/29206#discussion_r1090188302 ## airflow/providers/http/hooks/http.py: ## @@ -95,6 +98,11 @@ def get_conn(self, headers: dict[Any, Any] | None = None) -> requests.Session: self.

[GitHub] [airflow] uranusjr commented on pull request #27626: Fix an error that occur when Admin accesses to `Your Profile`.

2023-01-29 Thread via GitHub
uranusjr commented on PR #27626: URL: https://github.com/apache/airflow/pull/27626#issuecomment-1408019041 I think (some of) the code change should go into `_show` instead. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [airflow] uranusjr commented on a diff in pull request #27758: Enable trigger logging in webserver

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #27758: URL: https://github.com/apache/airflow/pull/27758#discussion_r1090182988 ## airflow/executors/base_executor.py: ## @@ -355,14 +355,15 @@ def execute_async( """ raise NotImplementedError() -def get_task_log(self, ti

[GitHub] [airflow] uranusjr commented on a diff in pull request #27758: Enable trigger logging in webserver

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #27758: URL: https://github.com/apache/airflow/pull/27758#discussion_r1090177715 ## airflow/providers/amazon/aws/log/cloudwatch_task_handler.py: ## @@ -65,6 +67,7 @@ def set_context(self, ti): self.handler = watchtower.CloudWatchLogHandl

[GitHub] [airflow] uranusjr commented on a diff in pull request #27758: Enable trigger logging in webserver

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #27758: URL: https://github.com/apache/airflow/pull/27758#discussion_r1090177715 ## airflow/providers/amazon/aws/log/cloudwatch_task_handler.py: ## @@ -65,6 +67,7 @@ def set_context(self, ti): self.handler = watchtower.CloudWatchLogHandl

[GitHub] [airflow] uranusjr commented on a diff in pull request #28571: Simplify log context setting / propagation handling

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #28571: URL: https://github.com/apache/airflow/pull/28571#discussion_r1090159560 ## airflow/utils/log/logging_mixin.py: ## @@ -32,15 +32,22 @@ ANSI_ESCAPE = re.compile(r"\x1B[@-_][0-?]*[ -/]*[@-~]") -# Private: A sentinel objects +# Private:

[GitHub] [airflow] uranusjr commented on a diff in pull request #29116: Add colors in help outputs of Airfow CLI commands #28789

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29116: URL: https://github.com/apache/airflow/pull/29116#discussion_r1090148795 ## airflow/cli/cli_parser.py: ## @@ -2201,46 +2202,56 @@ def _remove_dag_id_opt(command: ActionCommand): DAG_CLI_DICT: dict[str, CLICommand] = {sp.name: sp for sp i

[GitHub] [airflow] uranusjr commented on a diff in pull request #29203: docs: Add trigger process missing from Airflow docker

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29203: URL: https://github.com/apache/airflow/pull/29203#discussion_r1090147116 ## docs/apache-airflow/howto/docker-compose/index.rst: ## @@ -68,6 +68,7 @@ This file contains several service definitions: task instances once their dependencies

[GitHub] [airflow] uranusjr commented on a diff in pull request #29203: docs: Add trigger process missing from Airflow docker

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29203: URL: https://github.com/apache/airflow/pull/29203#discussion_r1090147116 ## docs/apache-airflow/howto/docker-compose/index.rst: ## @@ -68,6 +68,7 @@ This file contains several service definitions: task instances once their dependencies

[GitHub] [airflow] uranusjr commented on a diff in pull request #29203: docs: Add trigger process missing from Airflow docker

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29203: URL: https://github.com/apache/airflow/pull/29203#discussion_r1090147116 ## docs/apache-airflow/howto/docker-compose/index.rst: ## @@ -68,6 +68,7 @@ This file contains several service definitions: task instances once their dependencies

[GitHub] [airflow] uranusjr commented on a diff in pull request #29058: Add command to export purged records to CSV files

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1090143225 ## airflow/utils/db_cleanup.py: ## @@ -123,6 +126,15 @@ def _check_for_rows(*, query: Query, print_rows=False): return num_entities +def _dump_db(*, target_

[GitHub] [airflow] uranusjr commented on a diff in pull request #29058: Add command to export purged records to CSV files

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1090142830 ## airflow/utils/db_cleanup.py: ## @@ -370,3 +382,19 @@ def run_cleanup( session=session, ) session.commit() + + +@provide

[GitHub] [airflow] uranusjr commented on a diff in pull request #29058: Add command to export purged records to CSV files

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1090142651 ## airflow/cli/commands/db_command.py: ## @@ -207,3 +207,14 @@ def cleanup_tables(args): confirm=not args.yes, skip_archive=args.skip_archive,

[GitHub] [airflow] uranusjr commented on a diff in pull request #28846: Updated app to support configuring the caching hash method for FIPS

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #28846: URL: https://github.com/apache/airflow/pull/28846#discussion_r1090136984 ## airflow/www/extensions/init_cache.py: ## @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

[GitHub] [airflow] uranusjr commented on a diff in pull request #28846: Updated app to support configuring the caching hash method for FIPS

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #28846: URL: https://github.com/apache/airflow/pull/28846#discussion_r1090136514 ## airflow/compat/hashlib_wrapper.py: ## @@ -0,0 +1,45 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

[GitHub] [airflow] uranusjr commented on a diff in pull request #28846: Updated app to support configuring the caching hash method for FIPS

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #28846: URL: https://github.com/apache/airflow/pull/28846#discussion_r1090135796 ## airflow/compat/hashlib_wrapper.py: ## @@ -0,0 +1,45 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

[GitHub] [airflow] uranusjr commented on a diff in pull request #28846: Updated app to support configuring the caching hash method for FIPS

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #28846: URL: https://github.com/apache/airflow/pull/28846#discussion_r1090135941 ## airflow/config_templates/config.yml: ## @@ -1536,6 +1536,13 @@ webserver: type: boolean example: ~ default: "False" +caching_hash_method:

[GitHub] [airflow] uranusjr commented on a diff in pull request #28846: Updated app to support configuring the caching hash method for FIPS

2023-01-29 Thread via GitHub
uranusjr commented on code in PR #28846: URL: https://github.com/apache/airflow/pull/28846#discussion_r1090135796 ## airflow/compat/hashlib_wrapper.py: ## @@ -0,0 +1,45 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

[GitHub] [airflow] uranusjr commented on issue #29084: Add `max_active_tis_per_dagrun` for Dynamic Task Mapping

2023-01-29 Thread via GitHub
uranusjr commented on issue #29084: URL: https://github.com/apache/airflow/issues/29084#issuecomment-1407916817 I can see how this is different from DAG-level configuration, but am still unsure why DAG-level configurations can’t satisfy the need. Is there a practical use case this would be

[GitHub] [airflow] timecommunication opened a new issue, #29227: page /calendar bug ?

2023-01-29 Thread via GitHub
timecommunication opened a new issue, #29227: URL: https://github.com/apache/airflow/issues/29227 ### Apache Airflow version 2.5.1 ### What happened /calendar page give a problem, here is the capture ![屏幕截图 2023-01-30 093116](https://user-images.githubuser

[GitHub] [airflow] yxiao1996 commented on issue #28847: Graph UI: Add Filter Downstream & Filter DownStream & Upstream

2023-01-29 Thread via GitHub
yxiao1996 commented on issue #28847: URL: https://github.com/apache/airflow/issues/28847#issuecomment-1407814430 put up a PR: https://github.com/apache/airflow/pull/29226 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [airflow] github-actions[bot] commented on issue #28362: Airflow URL returning 404 when using Prefix

2023-01-29 Thread github-actions
github-actions[bot] commented on issue #28362: URL: https://github.com/apache/airflow/issues/28362#issuecomment-1407813670 This issue has been automatically marked as stale because it has been open for 30 days with no response from the author. It will be closed in next 7 days if no further

[GitHub] [airflow] yxiao1996 opened a new pull request, #29226: Implement new filtering options in graph view

2023-01-29 Thread via GitHub
yxiao1996 opened a new pull request, #29226: URL: https://github.com/apache/airflow/pull/29226 ### Description This change implements new task filtering options in graph view: * filtering downstream tasks of a task * filtering upstream and downstream tasks of a task

[GitHub] [airflow] yxiao1996 commented on issue #28847: Graph UI: Add Filter Downstream & Filter DownStream & Upstream

2023-01-29 Thread via GitHub
yxiao1996 commented on issue #28847: URL: https://github.com/apache/airflow/issues/28847#issuecomment-1407789904 thanks for fixing. -- 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 co

[airflow-client-python] branch master updated: Update .rat-excludes (#66)

2023-01-29 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git The following commit(s) were added to refs/heads/master by this push: new aee2e6b Update .rat-excludes (#6

[GitHub] [airflow-client-python] potiuk merged pull request #66: Update .rat-excludes

2023-01-29 Thread via GitHub
potiuk merged PR #66: URL: https://github.com/apache/airflow-client-python/pull/66 -- 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...@a

[GitHub] [airflow] michaelmicheal commented on issue #29162: Expand Dataset Integration Capabilities

2023-01-29 Thread via GitHub
michaelmicheal commented on issue #29162: URL: https://github.com/apache/airflow/issues/29162#issuecomment-1407773011 @jalajk24 Before working on it, I want to see whether or not @potiuk and others think of this feature, to see whether there is appetite for this change. Once we have a game

[GitHub] [airflow] potiuk commented on a diff in pull request #29225: Fix dev-mode start-airflow in Breeze

2023-01-29 Thread via GitHub
potiuk commented on code in PR #29225: URL: https://github.com/apache/airflow/pull/29225#discussion_r1090049220 ## dev/breeze/src/airflow_breeze/utils/run_utils.py: ## @@ -460,7 +465,9 @@ def run_compile_www_assets( "--all-files", "--verbose", ] -get_c

[GitHub] [airflow] pierrejeambrun commented on a diff in pull request #29225: Fix dev-mode start-airflow in Breeze

2023-01-29 Thread via GitHub
pierrejeambrun commented on code in PR #29225: URL: https://github.com/apache/airflow/pull/29225#discussion_r1090048551 ## dev/breeze/src/airflow_breeze/utils/run_utils.py: ## @@ -460,7 +465,9 @@ def run_compile_www_assets( "--all-files", "--verbose", ] -

[GitHub] [airflow] pierrejeambrun commented on a diff in pull request #29225: Fix dev-mode start-airflow in Breeze

2023-01-29 Thread via GitHub
pierrejeambrun commented on code in PR #29225: URL: https://github.com/apache/airflow/pull/29225#discussion_r1090048551 ## dev/breeze/src/airflow_breeze/utils/run_utils.py: ## @@ -460,7 +465,9 @@ def run_compile_www_assets( "--all-files", "--verbose", ] -

[GitHub] [airflow] pierrejeambrun commented on a diff in pull request #29225: Fix dev-mode start-airflow in Breeze

2023-01-29 Thread via GitHub
pierrejeambrun commented on code in PR #29225: URL: https://github.com/apache/airflow/pull/29225#discussion_r1090048551 ## dev/breeze/src/airflow_breeze/utils/run_utils.py: ## @@ -460,7 +465,9 @@ def run_compile_www_assets( "--all-files", "--verbose", ] -

[GitHub] [airflow] Taragolis commented on pull request #29220: Change reference codecov badge to main branch

2023-01-29 Thread via GitHub
Taragolis commented on PR #29220: URL: https://github.com/apache/airflow/pull/29220#issuecomment-1407763874 > It shows 36% even though our main seems to be at 88%. But 🤷 I think it happen because only Integration finished for particular run, usual it finished first -- This is an au

[GitHub] [airflow-client-python] pierrejeambrun opened a new pull request, #66: Update .rat-excludes

2023-01-29 Thread via GitHub
pierrejeambrun opened a new pull request, #66: URL: https://github.com/apache/airflow-client-python/pull/66 While checking licence for `2.5.1rc1` we still have a complaint on `.openapi-generator-ignore`. This also adds the same comment we have on [airflow](https://github.com/apache/a

[GitHub] [airflow] potiuk commented on pull request #29225: Fix dev-mode start-airflow in Breeze

2023-01-29 Thread via GitHub
potiuk commented on PR #29225: URL: https://github.com/apache/airflow/pull/29225#issuecomment-1407763667 Also fixed the message printed (properly show where the output of asset compilation is logged) -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [airflow] jalajk24 commented on issue #29162: Expand Dataset Integration Capabilities

2023-01-29 Thread via GitHub
jalajk24 commented on issue #29162: URL: https://github.com/apache/airflow/issues/29162#issuecomment-1407763159 Can I work on this? I am pretty new to open source -- 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-client-python] branch update-rat-excludes created (now c3f3004)

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a change to branch update-rat-excludes in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git at c3f3004 Update .rat-excludes This branch includes the following new commits:

[airflow-client-python] 01/01: Update .rat-excludes

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a commit to branch update-rat-excludes in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git commit c3f300482e07436b4a76c3e67a8ea1ea3b991f60 Author: pierrejeambrun AuthorDate: Sun Jan

[airflow-client-go] branch main updated: Add .rat-excludes also remove wrong LICENCE file (#36)

2023-01-29 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-client-go.git The following commit(s) were added to refs/heads/main by this push: new 68e8f3c Add .rat-excludes also remove wr

[GitHub] [airflow-client-go] potiuk merged pull request #36: Add .rat-excludes also remove wrong LICENCE file

2023-01-29 Thread via GitHub
potiuk merged PR #36: URL: https://github.com/apache/airflow-client-go/pull/36 -- 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...@airfl

[GitHub] [airflow-client-go] pierrejeambrun opened a new pull request, #36: Add .rat-excludes also remove wrong LICENCE file

2023-01-29 Thread via GitHub
pierrejeambrun opened a new pull request, #36: URL: https://github.com/apache/airflow-client-go/pull/36 Some files needs to be excluded from the licence check. This adds the `.rat-excludes` file. This also remove the incorrecte `LICNESE` file. (There is already a proper LICENSE file

[airflow-client-go] branch add-rat-excludes created (now 03c76da)

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a change to branch add-rat-excludes in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git at 03c76da Add .rat-excludes also remove wrong LICENCE file This branch includes the follo

[airflow-client-go] 01/01: Add .rat-excludes also remove wrong LICENCE file

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a commit to branch add-rat-excludes in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git commit 03c76daff85216786160beb2815d666ebd0a03a2 Author: pierrejeambrun AuthorDate: Sun Jan 29 21:1

svn commit: r59647 - in /dev/airflow/clients/go: ./ 2.5.0rc1/ 2.5.0rc1/apache-airflow-client-2.5.0-source.tar.gz 2.5.0rc1/apache-airflow-client-2.5.0-source.tar.gz.asc 2.5.0rc1/apache-airflow-client-2

2023-01-29 Thread pierrejeambrun
Author: pierrejeambrun Date: Sun Jan 29 19:50:12 2023 New Revision: 59647 Log: Add artifacts for Apache Airflow Go Client 2.5.0rc1 Added: dev/airflow/clients/go/ dev/airflow/clients/go/2.5.0rc1/ dev/airflow/clients/go/2.5.0rc1/apache-airflow-client-2.5.0-source.tar.gz (with props)

[GitHub] [airflow] potiuk commented on issue #28847: Graph UI: Add Filter Downstream & Filter DownStream & Upstream

2023-01-29 Thread via GitHub
potiuk commented on issue #28847: URL: https://github.com/apache/airflow/issues/28847#issuecomment-1407753819 > probably unrelated, `breeze start-airflow --dev-mode` looks broken for me with error message saying asset compilation fail. Without the `--dev-mode` flag it runs alright. F

[GitHub] [airflow] potiuk opened a new pull request, #29225: Fix dev-mode start-airflow in Breeze

2023-01-29 Thread via GitHub
potiuk opened a new pull request, #29225: URL: https://github.com/apache/airflow/pull/29225 The recent asset compilation fix #29185 broke --dev-mode flag in Breeze. The start-airflow command checks if the output file is successfully deleted, but in case of dev-mode - it is never deleted bec

[GitHub] [airflow] hussein-awala opened a new pull request, #29224: add output format arg for `cli.dags.trigger`

2023-01-29 Thread via GitHub
hussein-awala opened a new pull request, #29224: URL: https://github.com/apache/airflow/pull/29224 closes: #29178 --- Add output format argument for the command `dags.trigger` to simplify parsing the created `DagRun` info. -- This is an automated message from the Apache Git

[airflow] branch main updated (0a9a11acb7 -> f8c1410a0b)

2023-01-29 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 0a9a11acb7 Fix MySQL integrations tests run (#29213) add f8c1410a0b pre-commit: fix `files` path values (#29221)

[GitHub] [airflow] potiuk merged pull request #29221: pre-commit: fix `files` path values

2023-01-29 Thread via GitHub
potiuk merged PR #29221: URL: https://github.com/apache/airflow/pull/29221 -- 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-client-go] branch main updated: Add github release to release process (#35)

2023-01-29 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-client-go.git The following commit(s) were added to refs/heads/main by this push: new de7017f Add github release to release pr

[GitHub] [airflow-client-go] potiuk merged pull request #35: Add github release to release process

2023-01-29 Thread via GitHub
potiuk merged PR #35: URL: https://github.com/apache/airflow-client-go/pull/35 -- 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...@airfl

[airflow-client-python] branch master updated: Add github release to release process (#65)

2023-01-29 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git The following commit(s) were added to refs/heads/master by this push: new b94ca53 Add github release to re

[GitHub] [airflow-client-python] potiuk merged pull request #65: Add github release to release process

2023-01-29 Thread via GitHub
potiuk merged PR #65: URL: https://github.com/apache/airflow-client-python/pull/65 -- 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...@a

[GitHub] [airflow] potiuk opened a new pull request, #29223: Split installation of sdist providers into parallel chunks

2023-01-29 Thread via GitHub
potiuk opened a new pull request, #29223: URL: https://github.com/apache/airflow/pull/29223 Sdist provider installation takes a lot of time because pip cannot parallelise the sdist package building. But we still want to test all our provider's installation as sdist packages. This can

[GitHub] [airflow] hussein-awala commented on issue #29202: Trigger rule one_success with dynamic task mapping

2023-01-29 Thread via GitHub
hussein-awala commented on issue #29202: URL: https://github.com/apache/airflow/issues/29202#issuecomment-1407741460 For me it is not a bug, where with `one_success` trigger rule, we execute the task as soon as at least one parent succeeds, and we do not wait for all parents to be done.

[GitHub] [airflow] yxiao1996 commented on issue #28847: Graph UI: Add Filter Downstream & Filter DownStream & Upstream

2023-01-29 Thread via GitHub
yxiao1996 commented on issue #28847: URL: https://github.com/apache/airflow/issues/28847#issuecomment-1407737246 coded the filter downstream functionality, looks like this: original graph ![Screenshot_20230129_102657](https://user-images.githubusercontent.com/27112484/215348002-605

[GitHub] [airflow] yxiao1996 commented on issue #28847: Graph UI: Add Filter Downstream & Filter DownStream & Upstream

2023-01-29 Thread via GitHub
yxiao1996 commented on issue #28847: URL: https://github.com/apache/airflow/issues/28847#issuecomment-1407735538 probably unrelated, `breeze start-airflow --dev-mode` looks broken for me with error message saying asset compilation fail. Without the `--dev-mode` flag it runs alright.

[GitHub] [airflow] shyft-mike commented on a diff in pull request #28846: Updated app to support configuring the caching hash method for FIPS

2023-01-29 Thread via GitHub
shyft-mike commented on code in PR #28846: URL: https://github.com/apache/airflow/pull/28846#discussion_r1090024726 ## airflow/config_templates/config.yml: ## @@ -271,8 +270,8 @@ core: default: "0" default_task_retry_delay: description: | - The number

[airflow] branch constraints-main updated: Updating constraints. Build id:

2023-01-29 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 b29c01cc25 Updating con

[airflow-client-go] branch add-github-release-to-process updated (cd97ff2 -> 1c8d5a9)

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a change to branch add-github-release-to-process in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git from cd97ff2 Add github release to release process add 1c8d5a9 Add the exe

[GitHub] [airflow] jbampton opened a new pull request, #29221: pre-commit: fix `files` path values

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

[GitHub] [airflow-client-go] pierrejeambrun opened a new pull request, #35: Add github release to release process

2023-01-29 Thread via GitHub
pierrejeambrun opened a new pull request, #35: URL: https://github.com/apache/airflow-client-go/pull/35 As suggested by @potiuk, creating an actual release on github from the tag would be great. This change adds this step to the release process. -- This is an automated message from

[airflow-client-go] 01/01: Add github release to release process

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a commit to branch add-github-release-to-process in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git commit cd97ff2d22e313c8b42991fc6a8c8feb93ff6366 Author: pierrejeambrun AuthorDate: Su

[airflow-client-go] branch add-github-release-to-process created (now cd97ff2)

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a change to branch add-github-release-to-process in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git at cd97ff2 Add github release to release process This branch includes the fol

[GitHub] [airflow-client-python] pierrejeambrun opened a new pull request, #65: Add github release to release process

2023-01-29 Thread via GitHub
pierrejeambrun opened a new pull request, #65: URL: https://github.com/apache/airflow-client-python/pull/65 As suggested by @potiuk, creating an actual release on github from the tag would be great. This change adds this step to the release process. -- This is an automated message

[airflow-client-python] branch add-github-release-to-process created (now 69d7709)

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a change to branch add-github-release-to-process in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git at 69d7709 Add github release to release process This branch includes the

[airflow-client-python] 01/01: Add github release to release process

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a commit to branch add-github-release-to-process in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git commit 69d7709fddea59a4a4a3e5283d1ca83d088621e5 Author: pierrejeambrun AuthorDate

[GitHub] [airflow-client-go] pierrejeambrun commented on pull request #29: Generate client version 2.5.0

2023-01-29 Thread via GitHub
pierrejeambrun commented on PR #29: URL: https://github.com/apache/airflow-client-go/pull/29#issuecomment-1407721402 Very good idea, there is a step like this for airflow but not for clients. I will update the release processes :+1: -- This is an automated message from the Apache Git Ser

[GitHub] [airflow-client-go] potiuk commented on pull request #29: Generate client version 2.5.0

2023-01-29 Thread via GitHub
potiuk commented on PR #29: URL: https://github.com/apache/airflow-client-go/pull/29#issuecomment-1407718406 Also. I think it would be great to create actual GitHub releases from the tags (for both Python and Go client). -> it's super easy (create release from tag) and we might want to add

[airflow-client-go] annotated tag 2.5.0rc1 updated (80e9a79 -> 3aabd54)

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a change to annotated tag 2.5.0rc1 in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git *** WARNING: tag 2.5.0rc1 was modified! *** from 80e9a79 (commit) to 3aabd54 (tag) ta

[GitHub] [airflow-client-go] pierrejeambrun commented on pull request #29: Generate client version 2.5.0

2023-01-29 Thread via GitHub
pierrejeambrun commented on PR #29: URL: https://github.com/apache/airflow-client-go/pull/29#issuecomment-1407717084 I just added the `CHANGELOG.md` file to start tracking changes. According to the project history, last release was https://github.com/apache/airflow-client-go/pull/23 o

[airflow-client-go] branch generate-client-2.5.0 updated (80e9a79 -> 751f3da)

2023-01-29 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun pushed a change to branch generate-client-2.5.0 in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git from 80e9a79 Version 2.5.0 add 751f3da Add CHANGELOG.md No new revisions were add

[airflow] branch main updated (1910c5750c -> 0a9a11acb7)

2023-01-29 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 1910c5750c Change reference codecov badge to main branch (#29220) add 0a9a11acb7 Fix MySQL integrations tests run

[GitHub] [airflow] potiuk merged pull request #29213: Fix MySQL integrations tests run

2023-01-29 Thread via GitHub
potiuk merged PR #29213: URL: https://github.com/apache/airflow/pull/29213 -- 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] potiuk commented on pull request #29220: Change reference codecov badge to main branch

2023-01-29 Thread via GitHub
potiuk commented on PR #29220: URL: https://github.com/apache/airflow/pull/29220#issuecomment-1407711743 It shows 36% even though our main seems to be at 88%. But :shrug: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[airflow] branch main updated: Change reference codecov badge to main branch (#29220)

2023-01-29 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 1910c5750c Change reference codecov badge to main b

[GitHub] [airflow] potiuk merged pull request #29220: Change reference codecov badge to main branch

2023-01-29 Thread via GitHub
potiuk merged PR #29220: URL: https://github.com/apache/airflow/pull/29220 -- 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: pre-commit: add hook to sort `installed_providers.txt` (#29219)

2023-01-29 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 84235a433f pre-commit: add hook to sort `installed_

[GitHub] [airflow] potiuk merged pull request #29219: pre-commit: add hook to sort `installed_providers.txt`

2023-01-29 Thread via GitHub
potiuk merged PR #29219: URL: https://github.com/apache/airflow/pull/29219 -- 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] potiuk commented on pull request #29219: pre-commit: add hook to sort `installed_providers.txt`

2023-01-29 Thread via GitHub
potiuk commented on PR #29219: URL: https://github.com/apache/airflow/pull/29219#issuecomment-1407711273 Static checks passed. Merging. -- 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 specifi

[airflow-client-go] branch main updated: Sync main with master (#33)

2023-01-29 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-client-go.git The following commit(s) were added to refs/heads/main by this push: new 8c7e5f1 Sync main with master (#33) 8c7e

  1   2   >