[GitHub] [airflow] Jorricks opened a new pull request #21555: Add queue button to click-on-DagRun interface.

2022-02-13 Thread GitBox
Jorricks opened a new pull request #21555: URL: https://github.com/apache/airflow/pull/21555 In our use-case for Airflow, we often add tasks after the DAG has been created and had a first couple successful runs. This means, we often want successful DagRuns to be re-queued, such

[GitHub] [airflow] KevinYang21 edited a comment on pull request #18675: New Tree View

2022-02-13 Thread GitBox
KevinYang21 edited a comment on pull request #18675: URL: https://github.com/apache/airflow/pull/18675#issuecomment-1038732956 Thank you @bbovenzi for the work. I'm all for a more powerful UI and appreciate every bit of excellent effort you guys made/are making. We already have some

[GitHub] [airflow] KevinYang21 commented on pull request #18675: New Tree View

2022-02-13 Thread GitBox
KevinYang21 commented on pull request #18675: URL: https://github.com/apache/airflow/pull/18675#issuecomment-1038732956 Thank you @bbovenzi for the work. I'm all for a more powerful UI and appreciate every bit of excellent effort you guys made/are making. We already have some great

[GitHub] [airflow] ferruzzi commented on a change in pull request #21523: Update EKS sample DAGs and docs

2022-02-13 Thread GitBox
ferruzzi commented on a change in pull request #21523: URL: https://github.com/apache/airflow/pull/21523#discussion_r805539267 ## File path: airflow/providers/amazon/aws/example_dags/example_eks_templated.py ## @@ -49,50 +44,53 @@ """ with DAG( -

[GitHub] [airflow] github-actions[bot] commented on pull request #21553: Add documentation for RC3 release of providers for Jan 2022

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #21553: URL: https://github.com/apache/airflow/pull/21553#issuecomment-1038700837 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not

[GitHub] [airflow] david30907d commented on a change in pull request #21542: (bigquery-hook): forgot to pass engine_kwargs in

2022-02-13 Thread GitBox
david30907d commented on a change in pull request #21542: URL: https://github.com/apache/airflow/pull/21542#discussion_r805308216 ## File path: airflow/providers/google/cloud/hooks/bigquery.py ## @@ -163,6 +163,8 @@ def get_sqlalchemy_engine(self, engine_kwargs=None):

[GitHub] [airflow] github-actions[bot] commented on pull request #21541: Add str to xcom_pull task_ids typing

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #21541: URL: https://github.com/apache/airflow/pull/21541#issuecomment-1038067146 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and

[GitHub] [airflow] github-actions[bot] commented on issue #10061: Toggle buttons fire wrong URLs when deploying in "subdirectory"

2022-02-13 Thread GitBox
github-actions[bot] commented on issue #10061: URL: https://github.com/apache/airflow/issues/10061#issuecomment-1038487569 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

[GitHub] [airflow] potiuk commented on pull request #21145: enter the shell breeze2 environment

2022-02-13 Thread GitBox
potiuk commented on pull request #21145: URL: https://github.com/apache/airflow/pull/21145#issuecomment-1037260947 -- 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

[GitHub] [airflow] subkanthi closed pull request #20754: Fix numeric param deserialization

2022-02-13 Thread GitBox
subkanthi closed pull request #20754: URL: https://github.com/apache/airflow/pull/20754 -- 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:

[GitHub] [airflow] github-actions[bot] commented on pull request #21552: Contributors quickstart clarification

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #21552: URL: https://github.com/apache/airflow/pull/21552#issuecomment-1038462050 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might

[GitHub] [airflow] github-actions[bot] commented on issue #12782: Github Oauth issue

2022-02-13 Thread GitBox
github-actions[bot] commented on issue #12782: URL: https://github.com/apache/airflow/issues/12782#issuecomment-1038487546 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

[GitHub] [airflow] eladkal commented on issue #21443: Status of testing Providers that were prepared on February 09, 2022

2022-02-13 Thread GitBox
eladkal commented on issue #21443: URL: https://github.com/apache/airflow/issues/21443#issuecomment-1038038364 > Ah . Cool that you checked. I will exclude trino from the release too and release those (and rc3 will include trino as well). I think @david30907d meant #19508 not

[GitHub] [airflow] eladkal commented on a change in pull request #21523: Update EKS sample DAGs and docs

2022-02-13 Thread GitBox
eladkal commented on a change in pull request #21523: URL: https://github.com/apache/airflow/pull/21523#discussion_r805323142 ## File path: airflow/providers/amazon/aws/example_dags/example_eks_templated.py ## @@ -49,50 +44,53 @@ """ with DAG( -

[GitHub] [airflow] david30907d commented on issue #21443: Status of testing Providers that were prepared on February 09, 2022

2022-02-13 Thread GitBox
david30907d commented on issue #21443: URL: https://github.com/apache/airflow/issues/21443#issuecomment-1037968774 -- 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

[GitHub] [airflow] potiuk commented on issue #21545: Add Go to docker images

2022-02-13 Thread GitBox
potiuk commented on issue #21545: URL: https://github.com/apache/airflow/issues/21545#issuecomment-1038044791 -- 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

[GitHub] [airflow] github-actions[bot] closed issue #10061: Toggle buttons fire wrong URLs when deploying in "subdirectory"

2022-02-13 Thread GitBox
github-actions[bot] closed issue #10061: URL: https://github.com/apache/airflow/issues/10061 -- 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:

[GitHub] [airflow] potiuk commented on issue #13231: One ClientInfo instance in Google provider

2022-02-13 Thread GitBox
potiuk commented on issue #13231: URL: https://github.com/apache/airflow/issues/13231#issuecomment-1038365124 Assigned you @pierrejeambrun -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #21542: (bigquery-hook): forgot to pass engine_kwargs in

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #21542: URL: https://github.com/apache/airflow/pull/21542#issuecomment-1038016260 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not

[GitHub] [airflow] subkanthi closed pull request #20860: Branch python operator decorator

2022-02-13 Thread GitBox
subkanthi closed pull request #20860: URL: https://github.com/apache/airflow/pull/20860 -- 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:

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #21552: Contributors quickstart clarification

2022-02-13 Thread GitBox
boring-cyborg[bot] commented on pull request #21552: URL: https://github.com/apache/airflow/pull/21552#issuecomment-1038456376 -- 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

[GitHub] [airflow] pierrejeambrun commented on issue #15797: some issues with example dags

2022-02-13 Thread GitBox
pierrejeambrun commented on issue #15797: URL: https://github.com/apache/airflow/issues/15797#issuecomment-1038201514 -- 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

[GitHub] [airflow] mik-laj commented on issue #21545: Add Go to docker images

2022-02-13 Thread GitBox
mik-laj commented on issue #21545: URL: https://github.com/apache/airflow/issues/21545#issuecomment-1038471499 I think we should add the guide to the "Recipes" section in `docker-stack` documentation package. https://airflow.apache.org/docs/docker-stack/recipes.html We already have

[GitHub] [airflow] potiuk edited a comment on pull request #19571: :pencil: (providers_google) add a location check

2022-02-13 Thread GitBox
potiuk edited a comment on pull request #19571: URL: https://github.com/apache/airflow/pull/19571#issuecomment-1038351148 > @eladkal sorry for the late reply, I finally figure out how to rebase sweat_smile (i'm git noob) As of last week you can EASILY do rebase with the GitHub UI:

[GitHub] [airflow] potiuk commented on pull request #21552: Contributors quickstart clarification

2022-02-13 Thread GitBox
potiuk commented on pull request #21552: URL: https://github.com/apache/airflow/pull/21552#issuecomment-1038462118 Nice! Thanks. Small things like that are the most important and it's cool to see first-time contribution that helps others to make first-time contribution :) -- This is an

[GitHub] [airflow] turbaszek commented on a change in pull request #21470: Create Auto ML operators for Vertex AI service

2022-02-13 Thread GitBox
turbaszek commented on a change in pull request #21470: URL: https://github.com/apache/airflow/pull/21470#discussion_r805326443 ## File path: airflow/providers/google/cloud/operators/vertex_ai/auto_ml.py ## @@ -0,0 +1,707 @@ +# +# Licensed to the Apache Software Foundation

[GitHub] [airflow] pierrejeambrun commented on pull request #20386: Add support for BeamGoPipelineOperator

2022-02-13 Thread GitBox
pierrejeambrun commented on pull request #20386: URL: https://github.com/apache/airflow/pull/20386#issuecomment-1038010070 Hello @potiuk, I hope you are doing well :smile: The CI has failed on `Tests / Helm Chart; KubernetesExecutor` (1 test didn't pass) but it does not seem

[GitHub] [airflow] potiuk merged pull request #21530: Added template_ext = ('.json') to databricks operators #18925

2022-02-13 Thread GitBox
potiuk merged pull request #21530: URL: https://github.com/apache/airflow/pull/21530 -- 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:

[GitHub] [airflow] github-actions[bot] commented on pull request #19683: In DAG dependency detector, use class type instead of class name, related #19582

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #19683: URL: https://github.com/apache/airflow/pull/19683#issuecomment-1038487475 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

[GitHub] [airflow] potiuk merged pull request #20733: Add Audit Log View to Dag View

2022-02-13 Thread GitBox
potiuk merged pull request #20733: URL: https://github.com/apache/airflow/pull/20733 -- 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:

[GitHub] [airflow] pateash removed a comment on issue #21443: Status of testing Providers that were prepared on February 09, 2022

2022-02-13 Thread GitBox
pateash removed a comment on issue #21443: URL: https://github.com/apache/airflow/issues/21443#issuecomment-1037409252 > k int @potiuk, **apache-airflow-providers-GitHub** tested and working as expected.

[GitHub] [airflow] besenthil commented on issue #21377: Databricks: add SQL endpoint operators

2022-02-13 Thread GitBox
besenthil commented on issue #21377: URL: https://github.com/apache/airflow/issues/21377#issuecomment-1037626693 @dinowernli Is there a test stub that I can use for testing the API call? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] subkanthi commented on pull request #21077: Kubernetes decorator

2022-02-13 Thread GitBox
subkanthi commented on pull request #21077: URL: https://github.com/apache/airflow/pull/21077#issuecomment-1038336832 > ``` > airflow/example_dags/example_kubernetes_decorator.py:32:6: F811 redefinition of unused 'dag' from line 25 >

[GitHub] [airflow] github-actions[bot] closed issue #12782: Github Oauth issue

2022-02-13 Thread GitBox
github-actions[bot] closed issue #12782: URL: https://github.com/apache/airflow/issues/12782 -- 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:

[GitHub] [airflow] github-actions[bot] commented on pull request #18938: Jinja templates should be rendered in dict keys

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #18938: URL: https://github.com/apache/airflow/pull/18938#issuecomment-1037598181 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

[GitHub] [airflow] eladkal commented on pull request #21322: fixed K8S connection creation issue when kubeconfig or kubeconfig_path is empty string

2022-02-13 Thread GitBox
eladkal commented on pull request #21322: URL: https://github.com/apache/airflow/pull/21322#issuecomment-1037951899 @enniomorricone123 Why did you close the PR? was it by mistake? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] potiuk commented on issue #21443: Status of testing Providers that were prepared on February 09, 2022

2022-02-13 Thread GitBox
potiuk commented on issue #21443: URL: https://github.com/apache/airflow/issues/21443#issuecomment-1037340722 -- 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

[GitHub] [airflow] boring-cyborg[bot] commented on issue #21545: Add Go to docker images

2022-02-13 Thread GitBox
boring-cyborg[bot] commented on issue #21545: URL: https://github.com/apache/airflow/issues/21545#issuecomment-1038042549 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] pierrejeambrun edited a comment on issue #15797: some issues with example dags

2022-02-13 Thread GitBox
pierrejeambrun edited a comment on issue #15797: URL: https://github.com/apache/airflow/issues/15797#issuecomment-1038226126 @eladkal Just looked into it, you are right. All of them are updated and should not raise warnings. I think we can close this issue. -- This is an

[GitHub] [airflow] potiuk closed pull request #21546: Allow to switch easily between Bullseye and Buster debian versions

2022-02-13 Thread GitBox
potiuk closed pull request #21546: URL: https://github.com/apache/airflow/pull/21546 -- 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:

[GitHub] [airflow] potiuk commented on pull request #21546: Allow to switch easily between Bullseye and Buster debian versions

2022-02-13 Thread GitBox
potiuk commented on pull request #21546: URL: https://github.com/apache/airflow/pull/21546#issuecomment-1038059734 cc: @Bowrna - you will need to also add `--debian-version` switch to `Breeze2` when we merge this one -- This is an automated message from the Apache Git Service. To

[GitHub] [airflow] sungpeo commented on a change in pull request #21538: py files doesn't have to be checked is_zipfiles in process_files

2022-02-13 Thread GitBox
sungpeo commented on a change in pull request #21538: URL: https://github.com/apache/airflow/pull/21538#discussion_r805325566 ## File path: airflow/models/dagbag.py ## @@ -284,7 +284,10 @@ def process_file(self, filepath, only_if_updated=True, safe_mode=True):

[GitHub] [airflow] eladkal commented on pull request #19571: :pencil: (providers_google) add a location check

2022-02-13 Thread GitBox
eladkal commented on pull request #19571: URL: https://github.com/apache/airflow/pull/19571#issuecomment-1038138295 @david30907d can you resolve conflicts and rebase so we can merge this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] pierrejeambrun commented on issue #13231: One ClientInfo instance in Google provider

2022-02-13 Thread GitBox
pierrejeambrun commented on issue #13231: URL: https://github.com/apache/airflow/issues/13231#issuecomment-1038270203 Hi @eladkal, Thank you for pointing this issue. I would be glad to work on it :) -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] pierrejeambrun edited a comment on issue #13231: One ClientInfo instance in Google provider

2022-02-13 Thread GitBox
pierrejeambrun edited a comment on issue #13231: URL: https://github.com/apache/airflow/issues/13231#issuecomment-1038270203 Hi @eladkal, Thank you for pointing me to this issue. I would be glad to work on it :) -- This is an automated message from the Apache Git Service.

[GitHub] [airflow] pateash edited a comment on issue #21443: Status of testing Providers that were prepared on February 09, 2022

2022-02-13 Thread GitBox
pateash edited a comment on issue #21443: URL: https://github.com/apache/airflow/issues/21443#issuecomment-1037411412 -- 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

[GitHub] [airflow] eladkal closed issue #15797: some issues with example dags

2022-02-13 Thread GitBox
eladkal closed issue #15797: URL: https://github.com/apache/airflow/issues/15797 -- 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:

[GitHub] [airflow] Bowrna commented on a change in pull request #21145: enter the shell breeze2 environment

2022-02-13 Thread GitBox
Bowrna commented on a change in pull request #21145: URL: https://github.com/apache/airflow/pull/21145#discussion_r805174798 ## File path: dev/breeze/src/airflow_breeze/shell/enter_shell.py ## @@ -0,0 +1,176 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [airflow] potiuk merged pull request #21542: (bigquery-hook): forgot to pass engine_kwargs in

2022-02-13 Thread GitBox
potiuk merged pull request #21542: URL: https://github.com/apache/airflow/pull/21542 -- 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:

[GitHub] [airflow] eladkal commented on issue #15797: some issues with example dags

2022-02-13 Thread GitBox
eladkal commented on issue #15797: URL: https://github.com/apache/airflow/issues/15797#issuecomment-1038208589 -- 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

[GitHub] [airflow] thcidale0808 commented on pull request #21223: Added Feature: Create Pools from Env Vars.

2022-02-13 Thread GitBox
thcidale0808 commented on pull request #21223: URL: https://github.com/apache/airflow/pull/21223#issuecomment-1037443067 @potiuk , please find below a proposed solution and answers to your questions: ## Solution: Create a state file and a pool config sync cli and api to sync

[GitHub] [airflow] potiuk edited a comment on pull request #20386: Add support for BeamGoPipelineOperator

2022-02-13 Thread GitBox
potiuk edited a comment on pull request #20386: URL: https://github.com/apache/airflow/pull/20386#issuecomment-1038029380 I am doing some more changes in the images, so I might add Go to the image. Can you please add an issue about it @pierrejeambrun and CC: me ? -- This is an automated

[GitHub] [airflow] potiuk closed pull request #21520: Add mssql-cli to devel extra in Airflow

2022-02-13 Thread GitBox
potiuk closed pull request #21520: URL: https://github.com/apache/airflow/pull/21520 -- 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:

[GitHub] [airflow] potiuk closed issue #19570: get_records() don't work out for BigQueryHook

2022-02-13 Thread GitBox
potiuk closed issue #19570: URL: https://github.com/apache/airflow/issues/19570 -- 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:

[GitHub] [airflow] potiuk merged pull request #20386: Add support for BeamGoPipelineOperator

2022-02-13 Thread GitBox
potiuk merged pull request #20386: URL: https://github.com/apache/airflow/pull/20386 -- 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:

[GitHub] [airflow] potiuk edited a comment on pull request #21520: Add mssql-cli to devel extra in Airflow

2022-02-13 Thread GitBox
potiuk edited a comment on pull request #21520: URL: https://github.com/apache/airflow/pull/21520#issuecomment-1037334254 Hey @ephraimbuddy - I found a way of adding the mssql-cli without impacting our dependencies (I am using pipx to install it). @uranusjr - re pipx - is it often

[GitHub] [airflow] potiuk closed issue #20283: New operator for Go SDK in Apache Beam

2022-02-13 Thread GitBox
potiuk closed issue #20283: URL: https://github.com/apache/airflow/issues/20283 -- 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:

[GitHub] [airflow] potiuk merged pull request #19571: :pencil: (providers_google) add a location check

2022-02-13 Thread GitBox
potiuk merged pull request #19571: URL: https://github.com/apache/airflow/pull/19571 -- 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:

[GitHub] [airflow] Acehaidrey commented on pull request #20733: Add Audit Log View to Dag View

2022-02-13 Thread GitBox
Acehaidrey commented on pull request #20733: URL: https://github.com/apache/airflow/pull/20733#issuecomment-1037384199 Thank you so much! Glad can help and had a lot of help from Sam + Brent here. I'll start the discussion in the slack about the improvements discussed. -- This is an

[GitHub] [airflow] potiuk commented on pull request #20733: Add Audit Log View to Dag View

2022-02-13 Thread GitBox
potiuk commented on pull request #20733: URL: https://github.com/apache/airflow/pull/20733#issuecomment-1037216950 > Finally thank you @potiuk it is good :) Do I need to do anything else for this to be merged? Nope. Just merged! Great job. Audit log view has been a highly requested

[GitHub] [airflow] potiuk merged pull request #21552: Contributors quickstart clarification

2022-02-13 Thread GitBox
potiuk merged pull request #21552: URL: https://github.com/apache/airflow/pull/21552 -- 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:

[GitHub] [airflow] potiuk commented on pull request #21520: Add mssql-cli to devel extra in Airflow

2022-02-13 Thread GitBox
potiuk commented on pull request #21520: URL: https://github.com/apache/airflow/pull/21520#issuecomment-1037334254 Hey @ephraimbuddy - I found a way of adding the mssql-cli without impacting our dependencies (I am using pipx to install it). @uranusjr - re pipx - is it often the

[GitHub] [airflow] potiuk closed issue #21443: Status of testing Providers that were prepared on February 09, 2022

2022-02-13 Thread GitBox
potiuk closed issue #21443: URL: https://github.com/apache/airflow/issues/21443 -- 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:

[GitHub] [airflow] pateash commented on issue #21443: Status of testing Providers that were prepared on February 09, 2022

2022-02-13 Thread GitBox
pateash commented on issue #21443: URL: https://github.com/apache/airflow/issues/21443#issuecomment-1037409252 -- 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

[GitHub] [airflow] potiuk edited a comment on pull request #20860: Branch python operator decorator

2022-02-13 Thread GitBox
potiuk edited a comment on pull request #20860: URL: https://github.com/apache/airflow/pull/20860#issuecomment-1038394132 -- 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.

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #21530: Added template_ext = ('.json') to databricks operators #18925

2022-02-13 Thread GitBox
boring-cyborg[bot] commented on pull request #21530: URL: https://github.com/apache/airflow/pull/21530#issuecomment-1037217272 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk commented on pull request #20386: Add support for BeamGoPipelineOperator

2022-02-13 Thread GitBox
potiuk commented on pull request #20386: URL: https://github.com/apache/airflow/pull/20386#issuecomment-1038027380 -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #21538: py files doesn't have to be checked is_zipfiles in process_files

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #21538: URL: https://github.com/apache/airflow/pull/21538#issuecomment-1037266688 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and

[GitHub] [airflow-site] potiuk merged pull request #544: Add documentation for packages - 2022-02-09

2022-02-13 Thread GitBox
potiuk merged pull request #544: URL: https://github.com/apache/airflow-site/pull/544 -- 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:

[GitHub] [airflow] sungpeo commented on pull request #21538: py files doesn't have to be checked is_zipfiles in process_files

2022-02-13 Thread GitBox
sungpeo commented on pull request #21538: URL: https://github.com/apache/airflow/pull/21538#issuecomment-1037900877 -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #18931: Add clear task menu on task instance view

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #18931: URL: https://github.com/apache/airflow/pull/18931#issuecomment-1038487496 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

[GitHub] [airflow] potiuk merged pull request #21520: Add mssql-cli to devel extra in Airflow

2022-02-13 Thread GitBox
potiuk merged pull request #21520: URL: https://github.com/apache/airflow/pull/21520 -- 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:

[GitHub] [airflow] boring-cyborg[bot] commented on issue #21543: 1st Launch issue

2022-02-13 Thread GitBox
boring-cyborg[bot] commented on issue #21543: URL: https://github.com/apache/airflow/issues/21543#issuecomment-1037982086 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] Bowrna commented on pull request #21145: enter the shell breeze2 environment

2022-02-13 Thread GitBox
Bowrna commented on pull request #21145: URL: https://github.com/apache/airflow/pull/21145#issuecomment-1037259920 -- 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

[GitHub] [airflow] ac1997 commented on pull request #21351: Add and use supports_celery attribute for cli celery command validation

2022-02-13 Thread GitBox
ac1997 commented on pull request #21351: URL: https://github.com/apache/airflow/pull/21351#issuecomment-1037536124 Just want to resurface this PR... @potiuk Do you have any thoughts on this? -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] potiuk commented on pull request #20860: Branch python operator decorator

2022-02-13 Thread GitBox
potiuk commented on pull request #20860: URL: https://github.com/apache/airflow/pull/20860#issuecomment-1038394132 I think it is "triggered" by some of the changes you've done in `cncf.kubrmetes`. MyPy is "stable" (i.e. same files generate same errors) but the stability is a bit "fragile"

[GitHub] [airflow] david30907d commented on pull request #19571: :pencil: (providers_google) add a location check

2022-02-13 Thread GitBox
david30907d commented on pull request #19571: URL: https://github.com/apache/airflow/pull/19571#issuecomment-1038158578 -- 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

[GitHub] [airflow] potiuk commented on pull request #19571: :pencil: (providers_google) add a location check

2022-02-13 Thread GitBox
potiuk commented on pull request #19571: URL: https://github.com/apache/airflow/pull/19571#issuecomment-1038351148 > @eladkal sorry for the late reply, I finally figure out how to rebase sweat_smile (i'm git noob) As of last week you can EASILY do rebase with the UI:

[GitHub] [airflow] zbstof commented on pull request #18938: Jinja templates should be rendered in dict keys

2022-02-13 Thread GitBox
zbstof commented on pull request #18938: URL: https://github.com/apache/airflow/pull/18938#issuecomment-1037989301 Ping нд, 13 лют. 2022, 02:08 користувач github-actions[bot] < ***@***.***> пише: > This pull request has been automatically marked as stale because it has >

[GitHub] [airflow] subkanthi commented on pull request #20860: Branch python operator decorator

2022-02-13 Thread GitBox
subkanthi commented on pull request #20860: URL: https://github.com/apache/airflow/pull/20860#issuecomment-1038389898 -- 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

[GitHub] [airflow] chenglongyan closed pull request #21343: Fix test_clear_multiple_external_task_marker timing out

2022-02-13 Thread GitBox
chenglongyan closed pull request #21343: URL: https://github.com/apache/airflow/pull/21343 -- 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:

[GitHub] [airflow] turbaszek merged pull request #21541: Add str to xcom_pull task_ids typing

2022-02-13 Thread GitBox
turbaszek merged pull request #21541: URL: https://github.com/apache/airflow/pull/21541 -- 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:

[GitHub] [airflow] potiuk commented on a change in pull request #21145: enter the shell breeze2 environment

2022-02-13 Thread GitBox
potiuk commented on a change in pull request #21145: URL: https://github.com/apache/airflow/pull/21145#discussion_r805176319 ## File path: dev/breeze/src/airflow_breeze/shell/enter_shell.py ## @@ -0,0 +1,176 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [airflow] potiuk commented on a change in pull request #21549: Adds ADRs resulting from discussions during internship

2022-02-13 Thread GitBox
potiuk commented on a change in pull request #21549: URL: https://github.com/apache/airflow/pull/21549#discussion_r805360094 ## File path: .pre-commit-config.yaml ## @@ -25,6 +25,17 @@ repos: hooks: - id: identity - id: check-hooks-apply + - repo:

[GitHub] [airflow] potiuk closed issue #18925: Add template_ext = ('.json') to databricks operators

2022-02-13 Thread GitBox
potiuk closed issue #18925: URL: https://github.com/apache/airflow/issues/18925 -- 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:

[GitHub] [airflow] rsg17 commented on pull request #21084: [wip] Upload data from GCS to Presto

2022-02-13 Thread GitBox
rsg17 commented on pull request #21084: URL: https://github.com/apache/airflow/pull/21084#issuecomment-1038487841 > @eladkal : yes! Thank you for checking. Will do it this weekend.. @eladkal: I have added tests and docs. Let me know your feedback after you get a chance to review..

[GitHub] [airflow] potiuk edited a comment on issue #21545: Add Go to docker images

2022-02-13 Thread GitBox
potiuk edited a comment on issue #21545: URL: https://github.com/apache/airflow/issues/21545#issuecomment-1038045104 The worst case we can add instructions/examples how to do it in our "extending image" chapter. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] sungpeo edited a comment on pull request #21538: py files doesn't have to be checked is_zipfiles in process_files

2022-02-13 Thread GitBox
sungpeo edited a comment on pull request #21538: URL: https://github.com/apache/airflow/pull/21538#issuecomment-1038161260 -- 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.

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #21538: py files doesn't have to be checked is_zipfiles in process_files

2022-02-13 Thread GitBox
boring-cyborg[bot] commented on pull request #21538: URL: https://github.com/apache/airflow/pull/21538#issuecomment-1037251194 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

[GitHub] [airflow] uranusjr commented on a change in pull request #21538: py files doesn't have to be checked is_zipfiles in process_files

2022-02-13 Thread GitBox
uranusjr commented on a change in pull request #21538: URL: https://github.com/apache/airflow/pull/21538#discussion_r805299701 ## File path: airflow/models/dagbag.py ## @@ -284,7 +284,10 @@ def process_file(self, filepath, only_if_updated=True, safe_mode=True):

[GitHub] [airflow] potiuk commented on pull request #21356: Standardize approach to dependencies

2022-02-13 Thread GitBox
potiuk commented on pull request #21356: URL: https://github.com/apache/airflow/pull/21356#issuecomment-1037338257 I think that one is pretty read if we will be ok with introducing the policies as I proposed . -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] pierrejeambrun commented on issue #21545: Add Go to docker images

2022-02-13 Thread GitBox
pierrejeambrun commented on issue #21545: URL: https://github.com/apache/airflow/issues/21545#issuecomment-1038044332 @potiuk as requested, here is the issue :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] subkanthi commented on pull request #20860: Branch python operator decorator

2022-02-13 Thread GitBox
subkanthi commented on pull request #20860: URL: https://github.com/apache/airflow/pull/20860#issuecomment-1038534686 > I think it is "triggered" by some of the changes you've done in `cncf.kubrmetes`. MyPy is "stable" (i.e. same files generate same errors) but the stability is a bit

[GitHub] [airflow] rsg17 commented on pull request #21084: [wip] Upload data from GCS to Presto

2022-02-13 Thread GitBox
rsg17 commented on pull request #21084: URL: https://github.com/apache/airflow/pull/21084#issuecomment-1038487841 > @eladkal : yes! Thank you for checking. Will do it this weekend.. @eladkal: I have added tests and docs. Let me know your feedback after you get a chance to review..

[GitHub] [airflow] github-actions[bot] commented on issue #10061: Toggle buttons fire wrong URLs when deploying in "subdirectory"

2022-02-13 Thread GitBox
github-actions[bot] commented on issue #10061: URL: https://github.com/apache/airflow/issues/10061#issuecomment-1038487569 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

[GitHub] [airflow] github-actions[bot] commented on issue #12782: Github Oauth issue

2022-02-13 Thread GitBox
github-actions[bot] commented on issue #12782: URL: https://github.com/apache/airflow/issues/12782#issuecomment-1038487546 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

[GitHub] [airflow] github-actions[bot] closed issue #12782: Github Oauth issue

2022-02-13 Thread GitBox
github-actions[bot] closed issue #12782: URL: https://github.com/apache/airflow/issues/12782 -- 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:

[GitHub] [airflow] github-actions[bot] closed issue #10061: Toggle buttons fire wrong URLs when deploying in "subdirectory"

2022-02-13 Thread GitBox
github-actions[bot] closed issue #10061: URL: https://github.com/apache/airflow/issues/10061 -- 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:

[GitHub] [airflow] github-actions[bot] commented on pull request #19683: In DAG dependency detector, use class type instead of class name, related #19582

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #19683: URL: https://github.com/apache/airflow/pull/19683#issuecomment-1038487475 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

[GitHub] [airflow] github-actions[bot] commented on pull request #18931: Add clear task menu on task instance view

2022-02-13 Thread GitBox
github-actions[bot] commented on pull request #18931: URL: https://github.com/apache/airflow/pull/18931#issuecomment-1038487496 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

  1   2   3   >