[GitHub] [airflow] vumdao opened a new issue #18183: TypeError: cannot pickle '_thread.lock' object

2021-09-11 Thread GitBox
vumdao opened a new issue #18183: URL: https://github.com/apache/airflow/issues/18183 ### Apache Airflow version 2.1.3 (latest released) ### Operating System Kubernetes version: eks 1.18 - using Celery executor ### Versions of Apache Airflow Providers

[GitHub] [airflow] boring-cyborg[bot] commented on issue #18183: TypeError: cannot pickle '_thread.lock' object

2021-09-11 Thread GitBox
boring-cyborg[bot] commented on issue #18183: URL: https://github.com/apache/airflow/issues/18183#issuecomment-917559552 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] david30907d commented on issue #17558: New job fail immediately after start, no logs

2021-09-11 Thread GitBox
david30907d commented on issue #17558: URL: https://github.com/apache/airflow/issues/17558#issuecomment-917540756 bumped into the same issue, `DummyOperator` failed immediately with any log -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] github-actions[bot] commented on pull request #18182: Remove Brent from Collaborators

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18182: URL: https://github.com/apache/airflow/pull/18182#issuecomment-917538798 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] kaxil opened a new pull request #18182: Remove Brent from Collaborators

2021-09-11 Thread GitBox
kaxil opened a new pull request #18182: URL: https://github.com/apache/airflow/pull/18182 Brent is already a committer so we don't this entry here. It was needed only when he was not. --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil opened a new pull request #18181: Fix broken link in ``dev/REFRESHING_CI_CACHE.md``

2021-09-11 Thread GitBox
kaxil opened a new pull request #18181: URL: https://github.com/apache/airflow/pull/18181 The link was broken --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] branch fix-nroken-link created (now 3eb9155)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch fix-nroken-link in repository https://gitbox.apache.org/repos/asf/airflow.git. at 3eb9155 Fix broken link in ``dev/REFRESHING_CI_CACHE.md`` This branch includes the following new

[airflow] 01/01: Fix broken link in ``dev/REFRESHING_CI_CACHE.md``

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch fix-nroken-link in repository https://gitbox.apache.org/repos/asf/airflow.git commit 3eb9155327161b38c6ce20e8311b5b92af0438d6 Author: Kaxil Naik AuthorDate: Sun Sep 12 03:12:40 2021 +0100

[GitHub] [airflow] kaxil opened a new pull request #18180: Fix typo in task fail migration

2021-09-11 Thread GitBox
kaxil opened a new pull request #18180: URL: https://github.com/apache/airflow/pull/18180 `faskfail` -> `task fail` --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil opened a new pull request #18179: Doc: Use ``closer.lua`` script for downloading sources

2021-09-11 Thread GitBox
kaxil opened a new pull request #18179: URL: https://github.com/apache/airflow/pull/18179 - Follows first point of https://infra.apache.org/release-download-pages.html#download-page to use `https://www.apache.org/dyn/closer.lua/PROJECT/VERSION/SOURCE-RELEASE` for mirrors - Fixes bug

[GitHub] [airflow] dsubhasis commented on issue #18178: Neo4j can't connect

2021-09-11 Thread GitBox
dsubhasis commented on issue #18178: URL: https://github.com/apache/airflow/issues/18178#issuecomment-917515663 My installation is as follows: CONSTRAINT_URL=" https://raw.githubusercontent.com/apache/airflow/constraints-2.1.3/constraints-3.6.txt " AIRFLOW_VERSION=2.1.3

[GitHub] [airflow] github-actions[bot] commented on issue #8731: Logging module (custom) no longer working

2021-09-11 Thread GitBox
github-actions[bot] commented on issue #8731: URL: https://github.com/apache/airflow/issues/8731#issuecomment-917507284 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] github-actions[bot] commented on pull request #17253: add encoding config in FtpHook

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #17253: URL: https://github.com/apache/airflow/pull/17253#issuecomment-917507218 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] subkanthi commented on issue #18050: Duplicating the same connection twice gives "Integrity error, probably unique constraint"

2021-09-11 Thread GitBox
subkanthi commented on issue #18050: URL: https://github.com/apache/airflow/issues/18050#issuecomment-917505967 My opinion also sides with @BasPH , to query with a regex pattern to find all the connections which has _copy[0-9], but should we add a column to indicate that it was a

[airflow] branch main updated: Update various pre-commits (#18176)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik 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 15a9945 Update various pre-commits (#18176)

[GitHub] [airflow] kaxil merged pull request #18176: Update various pre-commits

2021-09-11 Thread GitBox
kaxil merged pull request #18176: URL: https://github.com/apache/airflow/pull/18176 -- 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] mik-laj commented on issue #18178: Neo4j can't connect

2021-09-11 Thread GitBox
mik-laj commented on issue #18178: URL: https://github.com/apache/airflow/issues/18178#issuecomment-917496714 Do you use constraints files to install Airflow? http://airflow.apache.org/docs/apache-airflow/stable/installation.html#constraints-files -- This is an automated message from

[GitHub] [airflow] boring-cyborg[bot] commented on issue #18178: Neo4j can't connect

2021-09-11 Thread GitBox
boring-cyborg[bot] commented on issue #18178: URL: https://github.com/apache/airflow/issues/18178#issuecomment-917496097 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] dsubhasis opened a new issue #18178: Neo4j can't connect

2021-09-11 Thread GitBox
dsubhasis opened a new issue #18178: URL: https://github.com/apache/airflow/issues/18178 ### Apache Airflow Provider(s) neo4j ### Versions of Apache Airflow Providers apache-airflow-providers-celery==1.0.1 apache-airflow-providers-cncf-kubernetes==1.0.2

[GitHub] [airflow] github-actions[bot] commented on pull request #18176: Update various pre-commits

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18176: URL: https://github.com/apache/airflow/pull/18176#issuecomment-917495481 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

[airflow] branch v2-1-stable updated (9168a0b -> 4377be3)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch v2-1-stable in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9168a0b Clearly document no breaking change for ``>=2.1.2, <=2.1.4`` add 0d04672 Set task state to

[GitHub] [airflow] kaxil merged pull request #18175: Sync `v2-1-stable` and `v2-1-test` to release `2.1.4`

2021-09-11 Thread GitBox
kaxil merged pull request #18175: URL: https://github.com/apache/airflow/pull/18175 -- 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] kaxil opened a new pull request #18177: Fix minor issues in Airflow release guide

2021-09-11 Thread GitBox
kaxil opened a new pull request #18177: URL: https://github.com/apache/airflow/pull/18177 Fixed wrong links and wrong artefact name --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] annotated tag 2.1.4rc1 updated (4377be3 -> 68e9c36)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to annotated tag 2.1.4rc1 in repository https://gitbox.apache.org/repos/asf/airflow.git. *** WARNING: tag 2.1.4rc1 was modified! *** from 4377be3 (commit) to 68e9c36 (tag) tagging

[GitHub] [airflow] github-actions[bot] commented on pull request #18175: Sync `v2-1-stable` and `v2-1-test` to release `2.1.4`

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18175: URL: https://github.com/apache/airflow/pull/18175#issuecomment-917485366 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

svn commit: r49883 - /dev/airflow/2.1.4rc1/

2021-09-11 Thread kaxilnaik
Author: kaxilnaik Date: Sat Sep 11 21:45:15 2021 New Revision: 49883 Log: Add artifacts for Airflow 2.1.4rc1 Added: dev/airflow/2.1.4rc1/ dev/airflow/2.1.4rc1/apache-airflow-2.1.4-source.tar.gz (with props) dev/airflow/2.1.4rc1/apache-airflow-2.1.4-source.tar.gz.asc

[airflow] annotated tag constraints-2.1.4rc1 updated (41f56db -> 7a2e1ab)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to annotated tag constraints-2.1.4rc1 in repository https://gitbox.apache.org/repos/asf/airflow.git. *** WARNING: tag constraints-2.1.4rc1 was modified! *** from 41f56db (tag) to 7a2e1ab

[airflow] annotated tag constraints-2.1.4rc1 updated (706ef46 -> 41f56db)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to annotated tag constraints-2.1.4rc1 in repository https://gitbox.apache.org/repos/asf/airflow.git. *** WARNING: tag constraints-2.1.4rc1 was modified! *** from 706ef46 (commit) to 41f56db

[GitHub] [airflow] kaxil opened a new pull request #18176: Update various pre-commits

2021-09-11 Thread GitBox
kaxil opened a new pull request #18176: URL: https://github.com/apache/airflow/pull/18176 ``` black: 21.7b0 -> 21.8b0 blacken-docs: v1.10.0 -> v1.11.0 pyupgrade: v2.23.3 -> v2.25.1 yamllint: v1.26.2 -> v1.26.3 flynt: 0.66 -> 0.69 ``` --- **^ Add

[airflow] branch main updated: Fix usage of ``range(len())`` to ``enumerate`` (#18174)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik 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 29af57b Fix usage of ``range(len())`` to

[GitHub] [airflow] kaxil merged pull request #18174: Fix usage of ``range(len())`` to ``enumerate``

2021-09-11 Thread GitBox
kaxil merged pull request #18174: URL: https://github.com/apache/airflow/pull/18174 -- 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] kaxil opened a new pull request #18175: V2 1 test

2021-09-11 Thread GitBox
kaxil opened a new pull request #18175: URL: https://github.com/apache/airflow/pull/18175 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more

[airflow] 02/02: Update CHANGELOG.txt

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 4377be3b38fbdfde876e727f0c73d8f978a3b1c7 Author: Kaxil Naik AuthorDate: Sat Sep 11 22:01:53 2021 +0100

[airflow] 01/02: Set task state to failed when pod is DELETED while running (#18095)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 0d046727201bbc08c5579c72c6f453b93af7741e Author: lindsable <47788186+lindsa...@users.noreply.github.com>

[airflow] branch v2-1-test updated (9168a0b -> 4377be3)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9168a0b Clearly document no breaking change for ``>=2.1.2, <=2.1.4`` new 0d04672 Set task state to

[GitHub] [airflow] github-actions[bot] commented on pull request #18174: Fix usage of ``range(len())`` to ``enumerate``

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18174: URL: https://github.com/apache/airflow/pull/18174#issuecomment-917475468 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] potiuk commented on pull request #18163: Sync `v2-1-stable` and `v2-1-test` to release `2.1.4`

2021-09-11 Thread GitBox
potiuk commented on pull request #18163: URL: https://github.com/apache/airflow/pull/18163#issuecomment-917475226 Woohoo! -- 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.

[airflow] branch v2-1-stable updated (2b80c1e -> 9168a0b)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch v2-1-stable in repository https://gitbox.apache.org/repos/asf/airflow.git. from 2b80c1e Bump ``sphinxcontrib-spelling`` and minor improvements (#16675) add 4d63c0f Improve

[GitHub] [airflow] kaxil merged pull request #18163: Sync `v2-1-stable` and `v2-1-test` to release `2.1.4`

2021-09-11 Thread GitBox
kaxil merged pull request #18163: URL: https://github.com/apache/airflow/pull/18163 -- 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] kaxil opened a new pull request #18174: Fix usage of ``range(len())`` to ``enumerate``

2021-09-11 Thread GitBox
kaxil opened a new pull request #18174: URL: https://github.com/apache/airflow/pull/18174 https://www.reddit.com/r/learnpython/comments/4af432/who_is_teaching_beginners_to_use_rangeleniterable/ has good reasons :) Also

[airflow] branch main updated: Set task state to failed when pod is DELETED while running (#18095)

2021-09-11 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi 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 e2d069f Set task state to failed when

[GitHub] [airflow] ephraimbuddy closed issue #17693: Kubernetes Executor: Tasks Stuck in Queued State indefinitely (or until scheduler restart).

2021-09-11 Thread GitBox
ephraimbuddy closed issue #17693: URL: https://github.com/apache/airflow/issues/17693 -- 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] ephraimbuddy merged pull request #18095: Kubernetes Executor Bug Fix: Set task state to failed when pod is DELETED while running

2021-09-11 Thread GitBox
ephraimbuddy merged pull request #18095: URL: https://github.com/apache/airflow/pull/18095 -- 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] mik-laj commented on a change in pull request #18168: Rename LocalToAzureDataLakeStorageOperator to LocalFilesystemToADLSOperator

2021-09-11 Thread GitBox
mik-laj commented on a change in pull request #18168: URL: https://github.com/apache/airflow/pull/18168#discussion_r706663960 ## File path: airflow/providers/microsoft/azure/transfers/local_to_adls.py ## @@ -100,3 +100,20 @@ def execute(self, context: dict) -> None:

[GitHub] [airflow] mik-laj closed issue #18172: install of airflow fails in argoCD because post-install hook in migrate-database-job.yaml

2021-09-11 Thread GitBox
mik-laj closed issue #18172: URL: https://github.com/apache/airflow/issues/18172 -- 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] mik-laj commented on issue #18172: install of airflow fails in argoCD because post-install hook in migrate-database-job.yaml

2021-09-11 Thread GitBox
mik-laj commented on issue #18172: URL: https://github.com/apache/airflow/issues/18172#issuecomment-917470180 Duplicate: https://github.com/apache/airflow/issues/17447 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[airflow] branch main updated (f9969c1 -> db72f40)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from f9969c1 Separate Installing from sources section and add more details (#18171) add db72f40 Remove

[GitHub] [airflow] kaxil merged pull request #18173: Remove redundant single quote from Breeze build image script

2021-09-11 Thread GitBox
kaxil merged pull request #18173: URL: https://github.com/apache/airflow/pull/18173 -- 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 #18173: Remove redundant single quote from Breeze build image script

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18173: URL: https://github.com/apache/airflow/pull/18173#issuecomment-917466354 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] kaxil opened a new pull request #18173: Remove redundant single quote from Breeze build image script

2021-09-11 Thread GitBox
kaxil opened a new pull request #18173: URL: https://github.com/apache/airflow/pull/18173 Before: ``` ERROR: Bad value for install-airflow-version: '2.2.0b1'. Only numerical versions allowed for PROD image here'! ``` After: ``` ERROR: Bad value for

[airflow] branch v2-1-test updated (bae5be0 -> 9168a0b)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git. from bae5be0 Separate Installing from sources section and add more details (#18171) add 9168a0b Clearly

[GitHub] [airflow] boring-cyborg[bot] commented on issue #18172: install of airflow fails in argoCD because post-install hook in migrate-database-job.yaml

2021-09-11 Thread GitBox
boring-cyborg[bot] commented on issue #18172: URL: https://github.com/apache/airflow/issues/18172#issuecomment-917463604 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] angel9484 opened a new issue #18172: install of airflow fails in argoCD because post-install hook in migrate-database-job.yaml

2021-09-11 Thread GitBox
angel9484 opened a new issue #18172: URL: https://github.com/apache/airflow/issues/18172 ### Official Helm Chart version 1.1.0 (latest released) ### Apache Airflow version 2.1.2 ### Kubernetes Version 1.18 ### Helm Chart configuration no

[airflow] branch v2-1-test updated (a525a4e -> bae5be0)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git. discard a525a4e Fix `TestSecurity.test_current_user_has_permissions` (#17916) discard e54e70c Fix missing

[airflow] branch main updated: Separate Installing from sources section and add more details (#18171)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik 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 f9969c1 Separate Installing from sources

[GitHub] [airflow] kaxil merged pull request #18171: Separate Installing from sources section and add more details

2021-09-11 Thread GitBox
kaxil merged pull request #18171: URL: https://github.com/apache/airflow/pull/18171 -- 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] kaxil commented on pull request #18171: Separate Installing from sources section and add more details

2021-09-11 Thread GitBox
kaxil commented on pull request #18171: URL: https://github.com/apache/airflow/pull/18171#issuecomment-917459127 Test it locally, passed -- 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

[airflow] branch separate-source-installation created (now 529376c)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch separate-source-installation in repository https://gitbox.apache.org/repos/asf/airflow.git. at 529376c Update installing-from-sources.rst This branch includes the following new

[airflow] 01/01: Update installing-from-sources.rst

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch separate-source-installation in repository https://gitbox.apache.org/repos/asf/airflow.git commit 529376ca4483f67586c8ad0b0de97f494a2754f4 Author: Kaxil Naik AuthorDate: Sat Sep 11 20:09:44

[GitHub] [airflow] github-actions[bot] commented on pull request #18171: Separate Installing from sources section and add more details

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18171: URL: https://github.com/apache/airflow/pull/18171#issuecomment-917455124 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] kaxil opened a new pull request #18171: Separate Installing from sources section and add more details

2021-09-11 Thread GitBox
kaxil opened a new pull request #18171: URL: https://github.com/apache/airflow/pull/18171 This PR separate installing Airflow from sources section and also fixes links for binary source, it had `-bin` suffix which we don't use anymore. And I have added section on verifying integrity. And

[GitHub] [airflow] jhtimmins commented on a change in pull request #18160: Apply parent dag permissions to subdags.

2021-09-11 Thread GitBox
jhtimmins commented on a change in pull request #18160: URL: https://github.com/apache/airflow/pull/18160#discussion_r706646302 ## File path: airflow/www/security.py ## @@ -485,12 +485,14 @@ def has_access(self, action_name, resource_name, user=None) -> bool:

[GitHub] [airflow] github-actions[bot] commented on pull request #18166: Add note about params on trigger DAG page

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18166: URL: https://github.com/apache/airflow/pull/18166#issuecomment-917448761 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

[airflow] branch main updated: Allow publishing Docker images with more pre-release versions (#18170)

2021-09-11 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 0b2b711 Allow publishing Docker images with more

[GitHub] [airflow] potiuk merged pull request #18170: Allow publishing Docker images with more pre-release versions

2021-09-11 Thread GitBox
potiuk merged pull request #18170: URL: https://github.com/apache/airflow/pull/18170 -- 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] kaxil commented on a change in pull request #18166: Add note about params on trigger DAG page

2021-09-11 Thread GitBox
kaxil commented on a change in pull request #18166: URL: https://github.com/apache/airflow/pull/18166#discussion_r706642661 ## File path: airflow/www/templates/airflow/trigger.html ## @@ -47,6 +47,9 @@ Trigger DAG: {{ dag_id }} To access configuration in

[GitHub] [airflow] kaxil opened a new pull request #18170: Allow publishing Docker images with more pre-release versions

2021-09-11 Thread GitBox
kaxil opened a new pull request #18170: URL: https://github.com/apache/airflow/pull/18170 Without it, it currently fails with the following: ``` Building and pushing 2.2.0b1 Airflow PROD image for 3.9 ERROR: Bad value for install-airflow-version: '2.2.0b1'. Only numerical

[airflow] branch v2-1-test updated (e54e70c -> a525a4e)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git. from e54e70c Fix missing `create_dummy_dag` fixture add a525a4e Fix

[airflow] branch main updated (3d6c86c -> 3e3c48a)

2021-09-11 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 3d6c86c Fix spelling mistake in documentation (#18167) add 3e3c48a Apply parent dag permissions to subdags.

[GitHub] [airflow] kaxil closed issue #17652: Subdags permissions not getting listed on Airflow UI

2021-09-11 Thread GitBox
kaxil closed issue #17652: URL: https://github.com/apache/airflow/issues/17652 -- 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] kaxil merged pull request #18160: Apply parent dag permissions to subdags.

2021-09-11 Thread GitBox
kaxil merged pull request #18160: URL: https://github.com/apache/airflow/pull/18160 -- 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] tnyz commented on a change in pull request #17998: Fix error when create external table using table resource

2021-09-11 Thread GitBox
tnyz commented on a change in pull request #17998: URL: https://github.com/apache/airflow/pull/17998#discussion_r706640208 ## File path: airflow/providers/google/cloud/operators/bigquery.py ## @@ -1195,36 +1200,24 @@ def execute(self, context) -> None: else:

[GitHub] [airflow] turbaszek commented on a change in pull request #17100: Advanced Params using json-schema

2021-09-11 Thread GitBox
turbaszek commented on a change in pull request #17100: URL: https://github.com/apache/airflow/pull/17100#discussion_r706638994 ## File path: airflow/models/param.py ## @@ -0,0 +1,208 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] turbaszek commented on a change in pull request #17100: Advanced Params using json-schema

2021-09-11 Thread GitBox
turbaszek commented on a change in pull request #17100: URL: https://github.com/apache/airflow/pull/17100#discussion_r706638528 ## File path: airflow/example_dags/example_complex_params.py ## @@ -0,0 +1,46 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [airflow] turbaszek commented on a change in pull request #17998: Fix error when create external table using table resource

2021-09-11 Thread GitBox
turbaszek commented on a change in pull request #17998: URL: https://github.com/apache/airflow/pull/17998#discussion_r706638081 ## File path: airflow/providers/google/cloud/operators/bigquery.py ## @@ -1195,36 +1200,24 @@ def execute(self, context) -> None: else:

[GitHub] [airflow] turbaszek commented on pull request #18150: Remove check for at least one schema

2021-09-11 Thread GitBox
turbaszek commented on pull request #18150: URL: https://github.com/apache/airflow/pull/18150#issuecomment-917441607 @c-nuro can you please fix the static checks? ``` airflow/providers/google/cloud/transfers/gcs_to_bigquery.py:23:1: F401 'airflow.exceptions.AirflowException' imported

[GitHub] [airflow] github-actions[bot] commented on pull request #18150: Remove check for at least one schema

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18150: URL: https://github.com/apache/airflow/pull/18150#issuecomment-917441536 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] eladkal commented on a change in pull request #17068: Influxdb Hook

2021-09-11 Thread GitBox
eladkal commented on a change in pull request #17068: URL: https://github.com/apache/airflow/pull/17068#discussion_r706635048 ## File path: tests/providers/influxdb/hooks/test_influxDbHook.py ## @@ -0,0 +1,56 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [airflow] eladkal opened a new pull request #18168: Rename LocalToAzureDataLakeStorageOperator to LocalFilesystemToADLSOperator

2021-09-11 Thread GitBox
eladkal opened a new pull request #18168: URL: https://github.com/apache/airflow/pull/18168 Renaming for consistency:. Local -> LocalFilesystem based on: `LocalFilesystemToS3Operator`, `LocalFilesystemToGCSOperator` shorting AzureDataLakeStorage to ADLS based on:

[airflow] branch main updated: Fix spelling mistake in documentation (#18167)

2021-09-11 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 3d6c86c Fix spelling mistake in documentation

[GitHub] [airflow] potiuk merged pull request #18167: Fix spelling mistake in documentation

2021-09-11 Thread GitBox
potiuk merged pull request #18167: URL: https://github.com/apache/airflow/pull/18167 -- 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 #18167: Fix spelling mistake in documentation

2021-09-11 Thread GitBox
github-actions[bot] commented on pull request #18167: URL: https://github.com/apache/airflow/pull/18167#issuecomment-917436999 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] ephraimbuddy commented on a change in pull request #17207: Fix external_executor_id not being set for manually run jobs.

2021-09-11 Thread GitBox
ephraimbuddy commented on a change in pull request #17207: URL: https://github.com/apache/airflow/pull/17207#discussion_r706633517 ## File path: tests/executors/test_celery_executor.py ## @@ -288,6 +291,9 @@ def test_command_validation(self, command, expected_exception):

[GitHub] [airflow] potiuk commented on issue #18050: Duplicating the same connection twice gives "Integrity error, probably unique constraint"

2021-09-11 Thread GitBox
potiuk commented on issue #18050: URL: https://github.com/apache/airflow/issues/18050#issuecomment-917433291 Pragmatic proposal (which I mentioned in the PR) is to have for loop with 1-10 and try them in sequence (and fail if we already have 10 copies). I can imagine someone making 2-3

[GitHub] [airflow] BasPH commented on issue #18050: Duplicating the same connection twice gives "Integrity error, probably unique constraint"

2021-09-11 Thread GitBox
BasPH commented on issue #18050: URL: https://github.com/apache/airflow/issues/18050#issuecomment-917432275 Copies will be created after a user action so I can't imagine ever having to query for e.g. 1 connections to find N. In my opinion it's not very user-friendly to refuse `copy2`.

[GitHub] [airflow] potiuk opened a new pull request #18167: Fix spelling mistake in documentation

2021-09-11 Thread GitBox
potiuk opened a new pull request #18167: URL: https://github.com/apache/airflow/pull/18167 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more

[airflow] branch v2-1-test updated (e4df035 -> e54e70c)

2021-09-11 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git. from e4df035 Fix spelling mistake add e54e70c Fix missing `create_dummy_dag` fixture No new revisions were

[airflow] branch v2-1-test updated (1f938d6 -> e4df035)

2021-09-11 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch v2-1-test in repository https://gitbox.apache.org/repos/asf/airflow.git. from 1f938d6 Improves quick-start docker-compose warnings and documentation add e4df035 Fix spelling mistake

[GitHub] [airflow] eladkal commented on pull request #18163: Sync `v2-1-stable` and `v2-1-test` to release `2.1.4`

2021-09-11 Thread GitBox
eladkal commented on pull request #18163: URL: https://github.com/apache/airflow/pull/18163#issuecomment-917430530 > Why do you think it should be there @eladkal :)? We do not cherry-pick all bugfixes to "patchlevel" releases, it's always risky - I think 2.1.4 is mostly about stabilising

[GitHub] [airflow] subkanthi commented on a change in pull request #17068: Influxdb Hook

2021-09-11 Thread GitBox
subkanthi commented on a change in pull request #17068: URL: https://github.com/apache/airflow/pull/17068#discussion_r706629474 ## File path: tests/providers/influxdb/operators/test_influxDbHook.py ## @@ -0,0 +1,48 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [airflow] subkanthi commented on a change in pull request #17068: Influxdb Hook

2021-09-11 Thread GitBox
subkanthi commented on a change in pull request #17068: URL: https://github.com/apache/airflow/pull/17068#discussion_r706629474 ## File path: tests/providers/influxdb/operators/test_influxDbHook.py ## @@ -0,0 +1,48 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [airflow] Jorricks commented on pull request #17207: Fix external_executor_id not being set for manually run jobs.

2021-09-11 Thread GitBox
Jorricks commented on pull request #17207: URL: https://github.com/apache/airflow/pull/17207#issuecomment-917430008 > You have some conflicts :) Fixed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] potiuk commented on pull request #18163: Sync `v2-1-stable` and `v2-1-test` to release `2.1.4`

2021-09-11 Thread GitBox
potiuk commented on pull request #18163: URL: https://github.com/apache/airflow/pull/18163#issuecomment-917429607 > I think #17269 was missed from releases Why do you think it should be there @eladkal :)? We do not cherry-pick all bugfixes to "patchlevel" releases, it's always

[airflow] branch main updated: Improves quick-start docker-compose warnings and documentation (#18164)

2021-09-11 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 8593d7f Improves quick-start docker-compose

[GitHub] [airflow] potiuk merged pull request #18164: Improves quick-start docker-compose warnings and documentation

2021-09-11 Thread GitBox
potiuk merged pull request #18164: URL: https://github.com/apache/airflow/pull/18164 -- 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] lindsable commented on pull request #18095: Kubernetes Executor Bug Fix: Set task state to failed when pod is DELETED while running

2021-09-11 Thread GitBox
lindsable commented on pull request #18095: URL: https://github.com/apache/airflow/pull/18095#issuecomment-917426093 @ephraimbuddy done! -- 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] subkanthi commented on a change in pull request #17068: Influxdb Hook

2021-09-11 Thread GitBox
subkanthi commented on a change in pull request #17068: URL: https://github.com/apache/airflow/pull/17068#discussion_r706621763 ## File path: docs/apache-airflow-providers-influxdb/operators/influxdb.rst ## @@ -0,0 +1,23 @@ + .. Licensed to the Apache Software Foundation (ASF)

[GitHub] [airflow] subkanthi commented on a change in pull request #17068: Influxdb Hook

2021-09-11 Thread GitBox
subkanthi commented on a change in pull request #17068: URL: https://github.com/apache/airflow/pull/17068#discussion_r706621763 ## File path: docs/apache-airflow-providers-influxdb/operators/influxdb.rst ## @@ -0,0 +1,23 @@ + .. Licensed to the Apache Software Foundation (ASF)

[GitHub] [airflow] eladkal commented on pull request #18163: Sync `v2-1-stable` and `v2-1-test` to release `2.1.4`

2021-09-11 Thread GitBox
eladkal commented on pull request #18163: URL: https://github.com/apache/airflow/pull/18163#issuecomment-917414840 I think https://github.com/apache/airflow/pull/17269 was missed from releases -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] blcksrx commented on pull request #17974: WIP: Implement before stm and after stm in DBApiHooks

2021-09-11 Thread GitBox
blcksrx commented on pull request #17974: URL: https://github.com/apache/airflow/pull/17974#issuecomment-917411521 I believe after acceptance of this proposal on `DBApiHook`, We should work on Hooks that is child of this Hook and provide functionality like @uranusjr suggested. We can

  1   2   >