[GitHub] [airflow] potiuk edited a comment on issue #15476: Setting sql_alchemy_conn via secrets backend fails

2021-05-27 Thread GitBox
potiuk edited a comment on issue #15476: URL: https://github.com/apache/airflow/issues/15476#issuecomment-850188239 > The 1.10.x issue is different though and require a different fix. I am afraid it's not critical enough to make into our "critical fixes" policy for 1.10.* . And since

[GitHub] [airflow] potiuk closed issue #15476: Setting sql_alchemy_conn via secrets backend fails

2021-05-27 Thread GitBox
potiuk closed issue #15476: URL: https://github.com/apache/airflow/issues/15476 -- 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. For queries about this service, please con

[GitHub] [airflow] potiuk commented on issue #15476: Setting sql_alchemy_conn via secrets backend fails

2021-05-27 Thread GitBox
potiuk commented on issue #15476: URL: https://github.com/apache/airflow/issues/15476#issuecomment-850188239 > The 1.10.x issue is different though and require a different fix. I am afraid it's not critical enough to make into our "critical fixes" policy for 1.10.* . And since 1.10 E

[GitHub] [airflow] uranusjr commented on a change in pull request #16084: Added new pipeline example for the tutorial docs (Issue #11208)

2021-05-27 Thread GitBox
uranusjr commented on a change in pull request #16084: URL: https://github.com/apache/airflow/pull/16084#discussion_r641309575 ## File path: docs/apache-airflow/tutorial.rst ## @@ -376,3 +376,133 @@ Here's a few things you might want to do next: - Review the :ref:`List o

[GitHub] [airflow] uranusjr commented on issue #15476: Setting sql_alchemy_conn via secrets backend fails

2021-05-27 Thread GitBox
uranusjr commented on issue #15476: URL: https://github.com/apache/airflow/issues/15476#issuecomment-850148545 The 1.10.x issue is different though and require a different fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] uranusjr commented on issue #15476: Setting sql_alchemy_conn via secrets backend fails

2021-05-27 Thread GitBox
uranusjr commented on issue #15476: URL: https://github.com/apache/airflow/issues/15476#issuecomment-850148224 On 2.x this has been fixed in #16088. -- 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

[GitHub] [airflow] ryanahamilton commented on a change in pull request #16129: Format more dates with timezone

2021-05-27 Thread GitBox
ryanahamilton commented on a change in pull request #16129: URL: https://github.com/apache/airflow/pull/16129#discussion_r641083047 ## File path: airflow/www/templates/airflow/dag_details.html ## @@ -50,11 +50,11 @@ {{ title }} Start Date - {{ dag.start_

[GitHub] [airflow] github-actions[bot] commented on pull request #15270: The KubernetesPodOperator Comprehensive Guide

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #15270: URL: https://github.com/apache/airflow/pull/15270#issuecomment-850022638 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 y

[GitHub] [airflow] github-actions[bot] commented on pull request #15277: Remove support for FAB `APP_THEME`

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #15277: URL: https://github.com/apache/airflow/pull/15277#issuecomment-850022630 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 y

[GitHub] [airflow] ephraimbuddy commented on issue #15488: Dynamic DAGs that disappear end up stuck in queued state.

2021-05-27 Thread GitBox
ephraimbuddy commented on issue #15488: URL: https://github.com/apache/airflow/issues/15488#issuecomment-850001116 I reproduced this in 2.1.0 by removing a dag while it's queued. Set default pool to 9. Below is the dag I used and to reproduce it, you need to trigger the dag multiple ti

[airflow] branch master updated (4ea78f8 -> 9c80bfb)

2021-05-27 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 4ea78f8 Chart: Add extra ini config to ``pgbouncer`` (#16120) add 9c80bfb Add Helm Chart Release Guide (#1

[GitHub] [airflow] kaxil merged pull request #16121: Add Helm Chart Release Guide

2021-05-27 Thread GitBox
kaxil merged pull request #16121: URL: https://github.com/apache/airflow/pull/16121 -- 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. For queries about this service, please

[GitHub] [airflow] jedcunningham closed pull request #16126: Allows webserver.base_url to be templated

2021-05-27 Thread GitBox
jedcunningham closed pull request #16126: URL: https://github.com/apache/airflow/pull/16126 -- 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. For queries about this service

[airflow] branch master updated: Chart: Add extra ini config to ``pgbouncer`` (#16120)

2021-05-27 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 4ea78f8 Chart: Add extra ini config to ``pg

[GitHub] [airflow] kaxil merged pull request #16120: Chart: Add extra ini config to pgbouncer

2021-05-27 Thread GitBox
kaxil merged pull request #16120: URL: https://github.com/apache/airflow/pull/16120 -- 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. For queries about this service, please

[airflow] branch master updated (23e4391 -> 4c50110)

2021-05-27 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 23e4391 Update Chart version to ``1.1.0-rc1`` (#16124) add 4c50110 Fix Typo: ``RedctableItem`` -> ``Redact

[GitHub] [airflow] kaxil merged pull request #16119: Typo: RedctableItem -> RedactableItem

2021-05-27 Thread GitBox
kaxil merged pull request #16119: URL: https://github.com/apache/airflow/pull/16119 -- 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. For queries about this service, please

[airflow] branch master updated (3beae71 -> 23e4391)

2021-05-27 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 3beae71 Add ``.gitattributes`` for ignoring tests files in ``git archive`` (#16122) add 23e4391 Update Ch

[GitHub] [airflow] kaxil merged pull request #16124: Update Chart version to ``1.1.0-rc1``

2021-05-27 Thread GitBox
kaxil merged pull request #16124: URL: https://github.com/apache/airflow/pull/16124 -- 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. For queries about this service, please

[airflow] branch master updated (10ed42a -> 3beae71)

2021-05-27 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 10ed42a Fix hooks extended from http hook (#16109) add 3beae71 Add ``.gitattributes`` for ignoring tests f

[GitHub] [airflow] kaxil merged pull request #16122: Add ``.gitattributes`` for ignoring tests files in ``git archive``

2021-05-27 Thread GitBox
kaxil merged pull request #16122: URL: https://github.com/apache/airflow/pull/16122 -- 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. For queries about this service, please

[GitHub] [airflow] ephraimbuddy closed pull request #15336: Fail task when containers inside a pod fails

2021-05-27 Thread GitBox
ephraimbuddy closed pull request #15336: URL: https://github.com/apache/airflow/pull/15336 -- 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. For queries about this service,

[GitHub] [airflow] jpyen commented on a change in pull request #16095: Bug Pod Template File Values Ignored

2021-05-27 Thread GitBox
jpyen commented on a change in pull request #16095: URL: https://github.com/apache/airflow/pull/16095#discussion_r641007018 ## File path: airflow/providers/cncf/kubernetes/CHANGELOG.rst ## @@ -19,6 +19,15 @@ Changelog - + +1.2.1 +. + +Bug Fixes +~ + +* `

[GitHub] [airflow] aliavni commented on issue #13322: Backfill crashes with "KeyError: TaskInstanceKey" when task has retries

2021-05-27 Thread GitBox
aliavni commented on issue #13322: URL: https://github.com/apache/airflow/issues/13322#issuecomment-849939475 I am having the same issue here on `airflow 2.0.2`. In my case, the dag works fine when deployed to the airflow server, but fails when I run or debug it in my IDE The excep

[GitHub] [airflow] danielcrisap closed issue #16123: Missing dags mount in WebServer

2021-05-27 Thread GitBox
danielcrisap closed issue #16123: URL: https://github.com/apache/airflow/issues/16123 -- 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. For queries about this service, plea

[GitHub] [airflow] danielcrisap opened a new issue #16131: Missing dags mount in WebServer

2021-05-27 Thread GitBox
danielcrisap opened a new issue #16131: URL: https://github.com/apache/airflow/issues/16131 **Apache Airflow version**: 2.1.0 **Kubernetes version**: `Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.8-eks-96780e", GitCommit:"96780e1b30acbf0a52c38b6030d7853e575bcdf

[GitHub] [airflow] zachliu opened a new issue #16130: Tree View is blocked by DagRuns status when showing all 356 DagRuns

2021-05-27 Thread GitBox
zachliu opened a new issue #16130: URL: https://github.com/apache/airflow/issues/16130 **Apache Airflow version**: 2.1.0 **Browsers**: Chrome and Firefox **What happened**: When showing all 365 DagRuns, they overlap with the "tree" on the left and we can't see the tree

[GitHub] [airflow] bbovenzi opened a new pull request #16129: Format more dates with timezone

2021-05-27 Thread GitBox
bbovenzi opened a new pull request #16129: URL: https://github.com/apache/airflow/pull/16129 A bunch of dates in the UI were not being formatted with the user's selected timezone from the dropdown in top navigation. I went through and updated as many as I could find. One exception,

[GitHub] [airflow] jedcunningham opened a new pull request #16126: Allows webserver.base_url to be templated

2021-05-27 Thread GitBox
jedcunningham opened a new pull request #16126: URL: https://github.com/apache/airflow/pull/16126 As `config`'s documentation states values are passed through `tpl`, one would expect `config.webserver.base_url` to also support templating. -- This is an automated message from the Apache

[GitHub] [airflow] potiuk commented on issue #16113: Not able to access airflow-variables at worker

2021-05-27 Thread GitBox
potiuk commented on issue #16113: URL: https://github.com/apache/airflow/issues/16113#issuecomment-849926688 you need to put a real variable name in there and not "Variable_Name" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [airflow] potiuk closed issue #16113: Not able to access airflow-variables at worker

2021-05-27 Thread GitBox
potiuk closed issue #16113: URL: https://github.com/apache/airflow/issues/16113 -- 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. For queries about this service, please con

[GitHub] [airflow] bbovenzi commented on issue #16112: List only a single level of TaskGroup in tree view

2021-05-27 Thread GitBox
bbovenzi commented on issue #16112: URL: https://github.com/apache/airflow/issues/16112#issuecomment-849920192 Tell me if I got this right. The other issue mentioned is to show only the TaskGroup and orphan tasks in the Tree View. Then this issue focuses on what happens when you clic

[airflow] branch master updated (2c6b003 -> 10ed42a)

2021-05-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 2c6b003 Updated table component (#15805) add 10ed42a Fix hooks extended from http hook (#16109) No new revis

[GitHub] [airflow] potiuk merged pull request #16109: Fix hooks extended from http hook

2021-05-27 Thread GitBox
potiuk merged pull request #16109: URL: https://github.com/apache/airflow/pull/16109 -- 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. For queries about this service, pleas

[airflow] branch master updated (65519ab -> 2c6b003)

2021-05-27 Thread ryanahamilton
This is an automated email from the ASF dual-hosted git repository. ryanahamilton pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 65519ab Fixes problem where conf variable was used before initialization (#16088) add 2c6b003 Updated

[GitHub] [airflow] ryanahamilton closed issue #15018: Build out custom Table components

2021-05-27 Thread GitBox
ryanahamilton closed issue #15018: URL: https://github.com/apache/airflow/issues/15018 -- 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. For queries about this service, ple

[GitHub] [airflow] ryanahamilton merged pull request #15805: Updated table component

2021-05-27 Thread GitBox
ryanahamilton merged pull request #15805: URL: https://github.com/apache/airflow/pull/15805 -- 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. For queries about this service

[GitHub] [airflow] github-actions[bot] commented on pull request #16121: Add Helm Chart Release Guide

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16121: URL: https://github.com/apache/airflow/pull/16121#issuecomment-849915379 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 decide

[GitHub] [airflow] github-actions[bot] commented on pull request #16077: Add updated-name wrappers for built-in FAB methods.

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16077: URL: https://github.com/apache/airflow/pull/16077#issuecomment-849914724 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 m

[GitHub] [airflow] jedcunningham commented on a change in pull request #16095: Bug Pod Template File Values Ignored

2021-05-27 Thread GitBox
jedcunningham commented on a change in pull request #16095: URL: https://github.com/apache/airflow/pull/16095#discussion_r640935707 ## File path: airflow/providers/cncf/kubernetes/CHANGELOG.rst ## @@ -19,6 +19,15 @@ Changelog - + +1.2.1 +. + +Bug Fixes +

[GitHub] [airflow] github-actions[bot] commented on pull request #16122: Add ``.gitattributes`` for ignoring tests files in ``git archive``

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16122: URL: https://github.com/apache/airflow/pull/16122#issuecomment-849908621 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 decide

[GitHub] [airflow] github-actions[bot] commented on pull request #16084: Added new pipeline example for the tutorial docs (Issue #11208)

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16084: URL: https://github.com/apache/airflow/pull/16084#issuecomment-849907998 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 decide

[GitHub] [airflow] potiuk commented on pull request #16095: Bug Pod Template File Values Ignored

2021-05-27 Thread GitBox
potiuk commented on pull request #16095: URL: https://github.com/apache/airflow/pull/16095#issuecomment-849906802 > All checks passed. By the way, why are there two seperate k8s test suites one in `/tests/proivders/cncf/kubernetes` the other in `/kubernetes_tests/` ? cncf.kubernetes

[GitHub] [airflow] Alxander64 commented on issue #16125: KubernetesPodOperator env_vars Jinja Templates Not Used

2021-05-27 Thread GitBox
Alxander64 commented on issue #16125: URL: https://github.com/apache/airflow/issues/16125#issuecomment-849905482 Just wanted to add that I've also tried this example blow: ``` from airflow.contrib.operators.kubernetes_pod_operator import KubernetesPodOperator from kubernetes.cl

[GitHub] [airflow] jpyen commented on pull request #16095: Bug Pod Template File Values Ignored

2021-05-27 Thread GitBox
jpyen commented on pull request #16095: URL: https://github.com/apache/airflow/pull/16095#issuecomment-849896936 All checks passed. By the way, why are there two seperate k8s test suites one in `/tests/proivders/cncf/kubernetes` the other in `/kubernetes_tests/` ? -- This is an automated

[GitHub] [airflow] jhtimmins commented on issue #15476: Setting sql_alchemy_conn via secrets backend fails

2021-05-27 Thread GitBox
jhtimmins commented on issue #15476: URL: https://github.com/apache/airflow/issues/15476#issuecomment-849895268 @metropolis-ameer thanks, that's actually super helpful. We'll see if someone else can take this one -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] Alxander64 opened a new issue #16125: KubernetesPodOperator env_vars Jinja Templates Not Used

2021-05-27 Thread GitBox
Alxander64 opened a new issue #16125: URL: https://github.com/apache/airflow/issues/16125 **Apache Airflow version**: v2.0.1 **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): 1.16 **Environment**: Kubernetes (EKS) - **Cloud provider or hardwa

[GitHub] [airflow] jhtimmins commented on pull request #16077: Add updated-name wrappers for built-in FAB methods.

2021-05-27 Thread GitBox
jhtimmins commented on pull request #16077: URL: https://github.com/apache/airflow/pull/16077#issuecomment-849870496 @potiuk can you take another look? I've added typing and either updated or explained the naming choices. -- This is an automated message from the Apache Git Service. To re

[airflow] branch constraints-master updated: Fix manually upgraded 0.5.1 of sphinx-rtd-theme

2021-05-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch constraints-master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-master by this push: new 1d3f65b Fix manually

[GitHub] [airflow] github-actions[bot] commented on pull request #16124: Update Chart version to ``1.1.0-rc1``

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16124: URL: https://github.com/apache/airflow/pull/16124#issuecomment-849861785 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 m

[GitHub] [airflow] kaxil opened a new pull request #16124: Update Chart version to ``1.1.0-rc1``

2021-05-27 Thread GitBox
kaxil opened a new pull request #16124: URL: https://github.com/apache/airflow/pull/16124 Let's change the Chart version to differentiate from currently released version. --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://gith

[GitHub] [airflow] metropolis-ameer commented on issue #15476: Setting sql_alchemy_conn via secrets backend fails

2021-05-27 Thread GitBox
metropolis-ameer commented on issue #15476: URL: https://github.com/apache/airflow/issues/15476#issuecomment-849853179 It may be a while (multiple months) before I could attempt it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [airflow] boring-cyborg[bot] commented on issue #16123: Missing dags mount in WebServer

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

[GitHub] [airflow] danielcrisap opened a new issue #16123: Missing dags mount in WebServer

2021-05-27 Thread GitBox
danielcrisap opened a new issue #16123: URL: https://github.com/apache/airflow/issues/16123 Hello 👋 Everyone, I'm new using Airflow, but I notice that my webservers a not mounting my dags with gitSync. I think that is because a wrong condition on webserver deployment. FROM

[GitHub] [airflow] kaxil opened a new pull request #16122: Add ``.gitattributes`` for ignoring tests files in ``git archive``

2021-05-27 Thread GitBox
kaxil opened a new pull request #16122: URL: https://github.com/apache/airflow/pull/16122 I did this manually while releasing the Helm Chart. This is useful to ignore directories when releasing "source code" via "git archive" command --- **^ Add meaningful description abov

[GitHub] [airflow] github-actions[bot] commented on pull request #16120: Chart: Add extra ini config to pgbouncer

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16120: URL: https://github.com/apache/airflow/pull/16120#issuecomment-849829687 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 m

[GitHub] [airflow] ashb commented on issue #16071: Secret masking fails on io objects

2021-05-27 Thread GitBox
ashb commented on issue #16071: URL: https://github.com/apache/airflow/issues/16071#issuecomment-849819172 Hmm curious, be setting that of _should_ have disabled the redacting. I'll look at what is going on there too -- This is an automated message from the Apache Git Service. To r

[GitHub] [airflow] kaxil opened a new pull request #16121: Add Helm Chart Release Guide

2021-05-27 Thread GitBox
kaxil opened a new pull request #16121: URL: https://github.com/apache/airflow/pull/16121 This commits adds "manual" instructions to release Helm Chart. This will be automated by a script over coming days. --- **^ Add meaningful description above** Read the **

[GitHub] [airflow] github-actions[bot] commented on pull request #16119: Typo: RedctableItem -> RedactableItem

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16119: URL: https://github.com/apache/airflow/pull/16119#issuecomment-849808983 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 tak

[GitHub] [airflow] madison-ookla commented on pull request #16118: Don't fail to log if we can't redact something

2021-05-27 Thread GitBox
madison-ookla commented on pull request #16118: URL: https://github.com/apache/airflow/pull/16118#issuecomment-849802293 Awesome, thanks @ashb! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [airflow] madison-ookla commented on issue #16071: Secret masking fails on io objects

2021-05-27 Thread GitBox
madison-ookla commented on issue #16071: URL: https://github.com/apache/airflow/issues/16071#issuecomment-849800668 @ashb unfortunately that setting seems to have no effect. If I set that to `False`, the logging still fails. I believe it's due to this line: https://github.com/apache/airflo

[GitHub] [airflow] jedcunningham opened a new pull request #16120: Add extra ini config to pgbouncer

2021-05-27 Thread GitBox
jedcunningham opened a new pull request #16120: URL: https://github.com/apache/airflow/pull/16120 This allows users to add extra ini config to pgbouncer, for example to adjust how frequently pgbouncer stats are output or reserve_pool size. -- This is an automated message from the Apach

[GitHub] [airflow] madison-ookla removed a comment on issue #16071: Secret masking fails on io objects

2021-05-27 Thread GitBox
madison-ookla removed a comment on issue #16071: URL: https://github.com/apache/airflow/issues/16071#issuecomment-849798386 Ah, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the speci

[GitHub] [airflow] madison-ookla commented on issue #16071: Secret masking fails on io objects

2021-05-27 Thread GitBox
madison-ookla commented on issue #16071: URL: https://github.com/apache/airflow/issues/16071#issuecomment-849798386 Ah, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comm

[GitHub] [airflow] BenoitHanotte edited a comment on issue #10026: Duplicate tasks invoked for a single task_id when manually invoked task details modal.

2021-05-27 Thread GitBox
BenoitHanotte edited a comment on issue #10026: URL: https://github.com/apache/airflow/issues/10026#issuecomment-849741546 I have started seeing the same issue since migrating to Airflow 2.1 when clearing task states, never had it with Airflow 2.0.1 :thinking: ``` INFO 2021-05-27

[GitHub] [airflow] ashb closed issue #16062: Unable to call logging.exception() in 2.1.0

2021-05-27 Thread GitBox
ashb closed issue #16062: URL: https://github.com/apache/airflow/issues/16062 -- 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. For queries about this service, please conta

[GitHub] [airflow] uranusjr opened a new pull request #16119: Typo: RedctableItem -> RedactableItem

2021-05-27 Thread GitBox
uranusjr opened a new pull request #16119: URL: https://github.com/apache/airflow/pull/16119 Fixing trivial stuff when they appear... --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.r

[GitHub] [airflow] uranusjr commented on issue #16062: Unable to call logging.exception() in 2.1.0

2021-05-27 Thread GitBox
uranusjr commented on issue #16062: URL: https://github.com/apache/airflow/issues/16062#issuecomment-849788656 Oh, Ash already fixed this in #16047. This is a duplicate then. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [airflow] Sanchit112 commented on pull request #16084: Added new pipeline example for the tutorial docs (Issue #11208)

2021-05-27 Thread GitBox
Sanchit112 commented on pull request #16084: URL: https://github.com/apache/airflow/pull/16084#issuecomment-849784685 Hey @uranusjr @kaxil, I have made changes, and the pre-commit test is built successfully. Can you merge this now? -- This is an automated message from the Apache Git Serv

[GitHub] [airflow] uranusjr commented on a change in pull request #15397: AIP-39: Use TimeTable interface to implement scheduling inside the DAG class

2021-05-27 Thread GitBox
uranusjr commented on a change in pull request #15397: URL: https://github.com/apache/airflow/pull/15397#discussion_r640792786 ## File path: airflow/models/dag.py ## @@ -533,94 +537,54 @@ def next_dagrun_info( "automated" DagRuns for this dag (scheduled or backfill

[airflow] branch master updated (9d06ee8 -> 65519ab)

2021-05-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9d06ee8 Fix apply defaults for task decorator (#16085) add 65519ab Fixes problem where conf variable was used

[GitHub] [airflow] potiuk closed issue #16079: NameError: name `conf` is not defined in configuration.py after upgraded to 2.1.0

2021-05-27 Thread GitBox
potiuk closed issue #16079: URL: https://github.com/apache/airflow/issues/16079 -- 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. For queries about this service, please con

[GitHub] [airflow] potiuk closed issue #15685: Undefined `conf` when using AWS secrets manager backend and `sql_alchemy_conn_secret`

2021-05-27 Thread GitBox
potiuk closed issue #15685: URL: https://github.com/apache/airflow/issues/15685 -- 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. For queries about this service, please con

[GitHub] [airflow] potiuk merged pull request #16088: Fixes problem where conf variable was used before initialization

2021-05-27 Thread GitBox
potiuk merged pull request #16088: URL: https://github.com/apache/airflow/pull/16088 -- 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. For queries about this service, pleas

[GitHub] [airflow] uranusjr commented on a change in pull request #15397: AIP-39: Use TimeTable interface to implement scheduling inside the DAG class

2021-05-27 Thread GitBox
uranusjr commented on a change in pull request #15397: URL: https://github.com/apache/airflow/pull/15397#discussion_r640789403 ## File path: airflow/models/dag.py ## @@ -533,94 +537,54 @@ def next_dagrun_info( "automated" DagRuns for this dag (scheduled or backfill

[GitHub] [airflow] ashb commented on a change in pull request #15397: AIP-39: Use TimeTable interface to implement scheduling inside the DAG class

2021-05-27 Thread GitBox
ashb commented on a change in pull request #15397: URL: https://github.com/apache/airflow/pull/15397#discussion_r640784403 ## File path: airflow/models/dag.py ## @@ -533,94 +537,54 @@ def next_dagrun_info( "automated" DagRuns for this dag (scheduled or backfill, bu

[GitHub] [airflow] lfreina commented on issue #13542: Task stuck in "scheduled" or "queued" state, pool has all slots queued, nothing is executing

2021-05-27 Thread GitBox
lfreina commented on issue #13542: URL: https://github.com/apache/airflow/issues/13542#issuecomment-849769325 I am having more or less the same issue with Kubernetes Executor. I sent the SIGUSR2 to the scheduler and got this: ``` [2021-05-27 16:15:55,628] {kubernetes_executor.py:2

[GitHub] [airflow] Junnplus commented on issue #16081: Add dashboard to see DAGs (system wide status) overview

2021-05-27 Thread GitBox
Junnplus commented on issue #16081: URL: https://github.com/apache/airflow/issues/16081#issuecomment-849763959 Sound awesome. I'm showing it on grafana by reading statsd metrics, is better that if airflow can provide dashboard. -- This is an automated message from the Apache Git Service.

[GitHub] [airflow] github-actions[bot] commented on pull request #16118: Don't fail to log if we can't redact something

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #16118: URL: https://github.com/apache/airflow/pull/16118#issuecomment-849746726 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 tak

[GitHub] [airflow] potiuk commented on pull request #16109: Fix hooks extended from http hook

2021-05-27 Thread GitBox
potiuk commented on pull request #16109: URL: https://github.com/apache/airflow/pull/16109#issuecomment-849745206 > Thanks a lot @potiuk. I tried to use breeze but could not figure out the correct way to run it. About getting the sorting order wrong I pasted the values after running the te

[GitHub] [airflow] eladkal commented on a change in pull request #16115: Add support for extra parameters to samba client

2021-05-27 Thread GitBox
eladkal commented on a change in pull request #16115: URL: https://github.com/apache/airflow/pull/16115#discussion_r640751662 ## File path: airflow/providers/samba/hooks/samba.py ## @@ -36,12 +36,26 @@ def __init__(self, samba_conn_id: str = default_conn_name) -> None:

[GitHub] [airflow] BenoitHanotte edited a comment on issue #10026: Duplicate tasks invoked for a single task_id when manually invoked task details modal.

2021-05-27 Thread GitBox
BenoitHanotte edited a comment on issue #10026: URL: https://github.com/apache/airflow/issues/10026#issuecomment-849741546 I have started seeing the same issue since migrating to Airflow 2.1 when clearing task states, never had it with Airflow 2.0.1 :thinking: ``` INFO 2021-05-27

[GitHub] [airflow] BenoitHanotte commented on issue #10026: Duplicate tasks invoked for a single task_id when manually invoked task details modal.

2021-05-27 Thread GitBox
BenoitHanotte commented on issue #10026: URL: https://github.com/apache/airflow/issues/10026#issuecomment-849741546 I have started seeing the same issue since migrating to Airflow 2.1, never had it with Airflow 2.0.1 :thinking: ``` INFO 2021-05-27 15:40:20,582 airflow.task.task_

[GitHub] [airflow] msumit commented on pull request #16109: Fix hooks extended from http hook

2021-05-27 Thread GitBox
msumit commented on pull request #16109: URL: https://github.com/apache/airflow/pull/16109#issuecomment-849741283 Thanks a lot @potiuk. I tried to use breeze but could not figure out the correct way to run it. About getting the sorting order wrong I pasted the values after running the test

[GitHub] [airflow] tothandor edited a comment on issue #16112: List only a single level of TaskGroup in tree view

2021-05-27 Thread GitBox
tothandor edited a comment on issue #16112: URL: https://github.com/apache/airflow/issues/16112#issuecomment-849720314 Sure. Here's a graph view collapsed and grouped with TaskGroup: ![image](https://user-images.githubusercontent.com/4449214/119849789-36cfcc00-bf0d-11eb-976b-dbcb

[GitHub] [airflow] github-actions[bot] commented on pull request #15989: Fix Celery executor getting stuck randomly because of reset_signals in multiprocessing

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #15989: URL: https://github.com/apache/airflow/pull/15989#issuecomment-849736601 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 tak

[airflow] branch master updated (cdc9f1a -> 9d06ee8)

2021-05-27 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from cdc9f1a Fill the "job_id" field for `airflow task run` without `--local`/`--raw` for KubeExecutor (#16108) ad

[GitHub] [airflow] potiuk merged pull request #16085: Fix default_args not applying to TaskFlow `@task` operators

2021-05-27 Thread GitBox
potiuk merged pull request #16085: URL: https://github.com/apache/airflow/pull/16085 -- 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. For queries about this service, pleas

[GitHub] [airflow] ZackBarry commented on issue #16101: BigQueryHook `create_empty_dataset` missing `datasetReference`

2021-05-27 Thread GitBox
ZackBarry commented on issue #16101: URL: https://github.com/apache/airflow/issues/16101#issuecomment-849726818 Sure thing @potiuk -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific c

[GitHub] [airflow] potiuk edited a comment on pull request #16109: Fix hooks extended from http hook

2021-05-27 Thread GitBox
potiuk edited a comment on pull request #16109: URL: https://github.com/apache/airflow/pull/16109#issuecomment-849724039 After entering the image with above `shell` command: ``` pytest tests/core/test_providers_manager.py::TestProviderManager::test_hooks ``` ``` _

[GitHub] [airflow] potiuk commented on pull request #16109: Fix hooks extended from http hook

2021-05-27 Thread GitBox
potiuk commented on pull request #16109: URL: https://github.com/apache/airflow/pull/16109#issuecomment-849724200 Wrong sorting order :) -- 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 speci

[GitHub] [airflow] potiuk commented on pull request #16109: Fix hooks extended from http hook

2021-05-27 Thread GitBox
potiuk commented on pull request #16109: URL: https://github.com/apache/airflow/pull/16109#issuecomment-849724039 ``` TestProviderManager.test_hooks

[GitHub] [airflow] ashb commented on pull request #15989: Fix Celery executor getting stuck randomly because of reset_signals in multiprocessing

2021-05-27 Thread GitBox
ashb commented on pull request #15989: URL: https://github.com/apache/airflow/pull/15989#issuecomment-849723516 Could you just address the few small niggles (mostly to minimize the change so it is easier to backport, and easier to read.) -- This is an automated message from the Apache Gi

[GitHub] [airflow] ashb commented on a change in pull request #15989: Fix Celery executor getting stuck randomly because of reset_signals in multiprocessing

2021-05-27 Thread GitBox
ashb commented on a change in pull request #15989: URL: https://github.com/apache/airflow/pull/15989#discussion_r640727665 ## File path: tests/executors/test_celery_executor.py ## @@ -484,3 +485,57 @@ def test_should_support_base_backend(self): assert [ 'D

[GitHub] [airflow] potiuk commented on pull request #16109: Fix hooks extended from http hook

2021-05-27 Thread GitBox
potiuk commented on pull request #16109: URL: https://github.com/apache/airflow/pull/16109#issuecomment-849720797 looking at it. BTW. When you look at the output it's possible to run breeze in the very CI image that was used during the tests locally: This way you can inspect

[GitHub] [airflow] tothandor commented on issue #16112: List only a single level of TaskGroup in tree view

2021-05-27 Thread GitBox
tothandor commented on issue #16112: URL: https://github.com/apache/airflow/issues/16112#issuecomment-849720314 Sure. Here's a graph view collapsed and grouped with TaskGroup: ![image](https://user-images.githubusercontent.com/4449214/119849789-36cfcc00-bf0d-11eb-976b-dbcb22dcb49

[GitHub] [airflow] ashb commented on a change in pull request #15382: Fix dag.clear() to set multiple dags to running when necessary

2021-05-27 Thread GitBox
ashb commented on a change in pull request #15382: URL: https://github.com/apache/airflow/pull/15382#discussion_r640719602 ## File path: airflow/models/dag.py ## @@ -1126,6 +1126,11 @@ def set_dag_runs_state( end_date: Optional[datetime] = None, dag_ids: List[

[GitHub] [airflow] ashb commented on issue #16071: Secret masking fails on io objects

2021-05-27 Thread GitBox
ashb commented on issue #16071: URL: https://github.com/apache/airflow/issues/16071#issuecomment-849712040 @madison-ookla A temporary work around if you need it set https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html#hide-sensitive-var-conn-fields to false -- T

[GitHub] [airflow] ashb commented on pull request #15989: Fix Celery executor getting stuck randomly because of reset_signals in multiprocessing

2021-05-27 Thread GitBox
ashb commented on pull request #15989: URL: https://github.com/apache/airflow/pull/15989#issuecomment-849712560 Oh yes, will take a look. -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #15382: Fix dag.clear() to set multiple dags to running when necessary

2021-05-27 Thread GitBox
github-actions[bot] commented on pull request #15382: URL: https://github.com/apache/airflow/pull/15382#issuecomment-849711728 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 tak

  1   2   3   >