[GitHub] [airflow-on-k8s-operator] turbaszek opened a new pull request #19: [Depends on #16] Add build step to CI

2020-03-22 Thread GitBox
turbaszek opened a new pull request #19: [Depends on #16] Add build step to CI URL: https://github.com/apache/airflow-on-k8s-operator/pull/19 Depends on #16 This is an automated message from the Apache Git Service. To respond

[GitHub] [airflow-on-k8s-operator] turbaszek merged pull request #15: Improve clone information in docs

2020-03-22 Thread GitBox
turbaszek merged pull request #15: Improve clone information in docs URL: https://github.com/apache/airflow-on-k8s-operator/pull/15 This is an automated message from the Apache Git Service. To respond to the message, please l

[airflow-on-k8s-operator] branch master updated: Improve clone information in docs (#15)

2020-03-22 Thread turbaszek
This is an automated email from the ASF dual-hosted git repository. turbaszek pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow-on-k8s-operator.git The following commit(s) were added to refs/heads/master by this push: new ec191cb Improve clone infor

[GitHub] [airflow] potiuk commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
potiuk commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#discussion_r395970116 ## File path: airflow/providers/google/cloud/hooks/gcs.py ## @@ -307,6

[GitHub] [airflow-on-k8s-operator] turbaszek opened a new issue #20: Add tests to CI build

2020-03-22 Thread GitBox
turbaszek opened a new issue #20: Add tests to CI build URL: https://github.com/apache/airflow-on-k8s-operator/issues/20 It would be reasonable to run tests (and e2e tests if possible) on CI. This is an automated message from

[GitHub] [airflow] potiuk commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
potiuk commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#discussion_r396065490 ## File path: airflow/providers/google/cloud/hooks/gcs.py ## @@ -307,6

[GitHub] [airflow-on-k8s-operator] turbaszek opened a new issue #21: There's not build rule in Makefile

2020-03-22 Thread GitBox
turbaszek opened a new issue #21: There's not build rule in Makefile URL: https://github.com/apache/airflow-on-k8s-operator/issues/21 According to https://github.com/apache/airflow-on-k8s-operator/blob/master/docs/development.md#building-and-running-locally one should be able to run ```

[GitHub] [airflow-on-k8s-operator] turbaszek opened a new issue #22: Add docker build step to CI

2020-03-22 Thread GitBox
turbaszek opened a new issue #22: Add docker build step to CI URL: https://github.com/apache/airflow-on-k8s-operator/issues/22 Run `docker build` as CI step to assure that the image is building. This is an automated message fr

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396067023 ## File path: airflow/providers/google/cloud/hooks/base.py ## @@ -168,54 +169,20 @@ def _

[GitHub] [airflow] potiuk commented on issue #7786: Restore DaskExecutor tests

2020-03-22 Thread GitBox
potiuk commented on issue #7786: Restore DaskExecutor tests URL: https://github.com/apache/airflow/pull/7786#issuecomment-602164329 Hello @TomAugspurger -> could you please rebase it on top of the latest master? We had some breakages which were unrelated and they should be fixed now. -

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396067167 ## File path: airflow/providers/google/cloud/secrets/secrets_manager.py ## @@ -0,0 +1,132

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396067245 ## File path: airflow/providers/google/cloud/secrets/secrets_manager.py ## @@ -0,0 +1,132

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396067305 ## File path: airflow/providers/google/cloud/utils/credentials_provider.py ## @@ -20,17 +

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396067305 ## File path: airflow/providers/google/cloud/utils/credentials_provider.py ## @@ -20,17 +

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
ephraimbuddy commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#discussion_r396067328 ## File path: airflow/providers/google/cloud/hooks/gcs.py ## @@ -

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396067459 ## File path: airflow/providers/hashicorp/secrets/vault.py ## @@ -31,7 +31,7 @@ clas

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396067868 ## File path: airflow/providers/google/secrets/secrets_manager.py ## @@ -0,0 +1,125 @@ +#

[GitHub] [airflow] turbaszek commented on a change in pull request #7797: Test installation of all provider packages

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7797: Test installation of all provider packages URL: https://github.com/apache/airflow/pull/7797#discussion_r396068387 ## File path: scripts/ci/in_container/run_test_package_installation.sh ## @@ -0,0 +1,64 @@ +#!/usr/bin/e

[GitHub] [airflow] turbaszek commented on issue #7797: Test installation of all provider packages

2020-03-22 Thread GitBox
turbaszek commented on issue #7797: Test installation of all provider packages URL: https://github.com/apache/airflow/pull/7797#issuecomment-602166381 Just out of curiosity, this PR has no JIRA ticket and no Github Issue. Is it ok? How will we include it in the changelog? @potiuk @ashb --

[GitHub] [airflow] potiuk commented on issue #7797: Test installation of all provider packages

2020-03-22 Thread GitBox
potiuk commented on issue #7797: Test installation of all provider packages URL: https://github.com/apache/airflow/pull/7797#issuecomment-602167889 We discussed that not all PRs need to have issues. PR# is more than enough to identify changes and we are going to use PR# in changelog from no

[GitHub] [airflow] potiuk commented on issue #7797: Test installation of all provider packages

2020-03-22 Thread GitBox
potiuk commented on issue #7797: Test installation of all provider packages URL: https://github.com/apache/airflow/pull/7797#issuecomment-602168050 @turbaszek - see https://lists.apache.org/thread.html/r19014fac5aa2aa15a86890aa438940da263e430cb535ca451ef451a4%40%3Cdev.airflow.apache.org

[GitHub] [airflow] codecov-io commented on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service

2020-03-22 Thread GitBox
codecov-io commented on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service URL: https://github.com/apache/airflow/pull/7794#issuecomment-602170571 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7794?src=pr&el=h1) Report > Merging [#7794](https://codecov.i

[GitHub] [airflow] codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking URL: https://github.com/apache/airflow/pull/7245#issuecomment-577833662 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7245?src=pr&el=h1) Report > Merging [#7245](https://c

[GitHub] [airflow] codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service URL: https://github.com/apache/airflow/pull/7794#issuecomment-602170571 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7794?src=pr&el=h1) Report > Merging [#7794](https://co

[GitHub] [airflow] codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service URL: https://github.com/apache/airflow/pull/7794#issuecomment-602170571 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7794?src=pr&el=h1) Report > Merging [#7794](https://co

[GitHub] [airflow] codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking URL: https://github.com/apache/airflow/pull/7245#issuecomment-577833662 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7245?src=pr&el=h1) Report > Merging [#7245](https://c

[GitHub] [airflow] codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking URL: https://github.com/apache/airflow/pull/7245#issuecomment-577833662 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7245?src=pr&el=h1) Report > Merging [#7245](https://c

[GitHub] [airflow] codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service URL: https://github.com/apache/airflow/pull/7794#issuecomment-602170571 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7794?src=pr&el=h1) Report > Merging [#7794](https://co

[GitHub] [airflow] codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking URL: https://github.com/apache/airflow/pull/7245#issuecomment-577833662 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7245?src=pr&el=h1) Report > Merging [#7245](https://c

[GitHub] [airflow] codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking URL: https://github.com/apache/airflow/pull/7245#issuecomment-577833662 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7245?src=pr&el=h1) Report > Merging [#7245](https://c

[GitHub] [airflow] codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking URL: https://github.com/apache/airflow/pull/7245#issuecomment-577833662 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7245?src=pr&el=h1) Report > Merging [#7245](https://c

[GitHub] [airflow] codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service URL: https://github.com/apache/airflow/pull/7794#issuecomment-602170571 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7794?src=pr&el=h1) Report > Merging [#7794](https://co

[GitHub] [airflow] codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service URL: https://github.com/apache/airflow/pull/7794#issuecomment-602170571 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7794?src=pr&el=h1) Report > Merging [#7794](https://co

[GitHub] [airflow] codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7794: [AIRFLOW-7099] Improve system test for Cloud Transfer Service URL: https://github.com/apache/airflow/pull/7794#issuecomment-602170571 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7794?src=pr&el=h1) Report > Merging [#7794](https://co

[GitHub] [airflow] codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7245: [AIRFLOW-6624] Improve webserver command with pidfile checking URL: https://github.com/apache/airflow/pull/7245#issuecomment-577833662 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7245?src=pr&el=h1) Report > Merging [#7245](https://c

[GitHub] [airflow] turbaszek commented on a change in pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730#discussion_r396076732 ## File path: INSTALL ## @@ -1,7 +1,20 @@ # INSTALL / BUILD instructions for Apache Airflow -# [

[GitHub] [airflow] mik-laj commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
mik-laj commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396076835 ## File path: airflow/providers/google/cloud/secrets/secrets_manager.py ## @@ -0,0 +1,132 @

[GitHub] [airflow] mik-laj commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
mik-laj commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396076912 ## File path: airflow/providers/google/cloud/utils/credentials_provider.py ## @@ -158,3 +16

[GitHub] [airflow] turbaszek commented on a change in pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730#discussion_r396076916 ## File path: INSTALL ## @@ -11,12 +24,31 @@ java -jar apache-rat.jar -E ./.rat-excludes -d . # to

[GitHub] [airflow] potiuk commented on a change in pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
potiuk commented on a change in pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730#discussion_r396077467 ## File path: INSTALL ## @@ -11,12 +24,31 @@ java -jar apache-rat.jar -E ./.rat-excludes -d . # to co

[GitHub] [airflow] zhongjiajie opened a new pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
zhongjiajie opened a new pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802 --- Issue link: WILL BE INSERTED BY [boring-cyborg](https://github.com/kaxil/boring-cyborg) Make sure to mark the boxes below before creating PR:

[GitHub] [airflow] codecov-io commented on issue #7801: Remove arg lookup map in CLIFactory

2020-03-22 Thread GitBox
codecov-io commented on issue #7801: Remove arg lookup map in CLIFactory URL: https://github.com/apache/airflow/pull/7801#issuecomment-602179212 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7801?src=pr&el=h1) Report > Merging [#7801](https://codecov.io/gh/apache/airflow/pull/78

[GitHub] [airflow] potiuk commented on issue #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
potiuk commented on issue #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#issuecomment-602181537 Still - there are few static checks failing :) This is an automated message from

[GitHub] [airflow] mik-laj merged pull request #7801: Remove arg lookup map in CLIFactory

2020-03-22 Thread GitBox
mik-laj merged pull request #7801: Remove arg lookup map in CLIFactory URL: https://github.com/apache/airflow/pull/7801 This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [airflow] zhongjiajie edited a comment on issue #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
zhongjiajie edited a comment on issue #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#issuecomment-602182105 I got it, trying to fix it. This is an automated message from the A

[GitHub] [airflow] zhongjiajie commented on issue #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
zhongjiajie commented on issue #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#issuecomment-602182105 I get it, trying to fix it. This is an automated message from the Apache Gi

[GitHub] [airflow] ivorynoise commented on a change in pull request #7724: [AIRFLOW-1536] Inherit umask from parent process in daemon mode

2020-03-22 Thread GitBox
ivorynoise commented on a change in pull request #7724: [AIRFLOW-1536] Inherit umask from parent process in daemon mode URL: https://github.com/apache/airflow/pull/7724#discussion_r396081333 ## File path: airflow/cli/commands/celery_command.py ## @@ -135,6 +135,7 @@ def wo

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396081461 ## File path: airflow/providers/google/cloud/secrets/secrets_manager.py ## @@ -0,0 +1,1

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
ephraimbuddy commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#discussion_r396082445 ## File path: airflow/providers/google/cloud/hooks/gcs.py ## @@ -

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
ephraimbuddy commented on a change in pull request #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#discussion_r396082483 ## File path: airflow/providers/google/cloud/operators/gcs_to_gcs.py #

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082748 ## File path: airflow/providers/opsgenie/operators/opsgenie_alert.py ## @@ -92,7 +93,7 @@ def __init__(sel

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396081799 ## File path: airflow/providers/apache/hive/hooks/hive.py ## @@ -538,7 +542,7 @@ def sasl_factory():

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082221 ## File path: airflow/providers/apache/pinot/hooks/pinot.py ## @@ -66,28 +68,45 @@ def __init__(self,

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082860 ## File path: airflow/providers/qubole/hooks/qubole_check.py ## @@ -39,14 +42,21 @@ def isint(value):

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396081893 ## File path: airflow/providers/apache/hive/hooks/hive.py ## @@ -633,7 +634,7 @@ def get_databases(self, p

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082363 ## File path: airflow/providers/dingding/hooks/dingding.py ## @@ -129,5 +129,5 @@ def send(self):

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396081835 ## File path: airflow/providers/apache/hive/hooks/hive.py ## @@ -575,10 +579,7 @@ def check_for_partition(

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082499 ## File path: airflow/providers/docker/hooks/docker.py ## @@ -84,4 +84,4 @@ def __login(self, client):

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082558 ## File path: airflow/providers/ftp/hooks/ftp.py ## @@ -115,7 +115,7 @@ def describe_directory(self, path)

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082449 ## File path: airflow/providers/docker/hooks/docker.py ## @@ -60,7 +60,7 @@ def __init__(self, se

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396081526 ## File path: airflow/providers/apache/druid/hooks/druid.py ## @@ -121,7 +127,7 @@ def submit_indexing_job

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082817 ## File path: airflow/providers/postgres/hooks/postgres.py ## @@ -95,7 +96,7 @@ def get_conn(self):

[GitHub] [airflow] turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7802: Make airflow/providers pylint compatible URL: https://github.com/apache/airflow/pull/7802#discussion_r396082849 ## File path: airflow/providers/qubole/hooks/qubole_check.py ## @@ -93,17 +109,24 @@ def handle_failure_re

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396084064 ## File path: airflow/providers/google/cloud/secrets/secrets_manager.py ## @@ -0,0 +1,1

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396084072 ## File path: airflow/providers/google/cloud/utils/credentials_provider.py ## @@ -20,17

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396084024 ## File path: airflow/providers/google/cloud/secrets/secrets_manager.py ## @@ -0,0 +1,1

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396084031 ## File path: airflow/providers/google/cloud/hooks/base.py ## @@ -168,54 +169,20 @@ def

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396084081 ## File path: airflow/providers/google/cloud/secrets/secrets_manager.py ## @@ -0,0 +1,1

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396084203 ## File path: airflow/providers/hashicorp/secrets/vault.py ## @@ -31,7 +31,7 @@ cl

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] codecov-io edited a comment on issue #7793: Verify that all providers packages are included

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7793: Verify that all providers packages are included URL: https://github.com/apache/airflow/pull/7793#issuecomment-602187167 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7793?src=pr&el=h1) Report > Merging [#7793](https://codecov.io/gh/ap

[GitHub] [airflow] codecov-io commented on issue #7793: Verify that all providers packages are included

2020-03-22 Thread GitBox
codecov-io commented on issue #7793: Verify that all providers packages are included URL: https://github.com/apache/airflow/pull/7793#issuecomment-602187167 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7793?src=pr&el=h1) Report > Merging [#7793](https://codecov.io/gh/apache/ai

[GitHub] [airflow] codecov-io edited a comment on issue #7793: Verify that all providers packages are included

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7793: Verify that all providers packages are included URL: https://github.com/apache/airflow/pull/7793#issuecomment-602187167 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7793?src=pr&el=h1) Report > Merging [#7793](https://codecov.io/gh/ap

[GitHub] [airflow] codecov-io edited a comment on issue #7793: Verify that all providers packages are included

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7793: Verify that all providers packages are included URL: https://github.com/apache/airflow/pull/7793#issuecomment-602187167 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7793?src=pr&el=h1) Report > Merging [#7793](https://codecov.io/gh/ap

[GitHub] [airflow] xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
xinbinhuang commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396084533 ## File path: airflow/providers/hashicorp/secrets/vault.py ## @@ -40,10 +40,10 @@ class

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7791: Add ability to specify a maximum modified time for objects in GCSToGCSOperator URL: https://github.com/apache/airflow/pull/7791#issuecomment-601966502 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7791?src=pr&el=h1) Report > Merging [

[GitHub] [airflow] codecov-io edited a comment on issue #7793: Verify that all providers packages are included

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7793: Verify that all providers packages are included URL: https://github.com/apache/airflow/pull/7793#issuecomment-602187167 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7793?src=pr&el=h1) Report > Merging [#7793](https://codecov.io/gh/ap

[GitHub] [airflow] codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730#issuecomment-599199404 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7730?src=pr&el=h1) Report > Merging [#7730](https://codecov.io/gh/ap

[GitHub] [airflow] codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730#issuecomment-599199404 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7730?src=pr&el=h1) Report > Merging [#7730](https://codecov.io/gh/ap

[GitHub] [airflow] codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730#issuecomment-599199404 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7730?src=pr&el=h1) Report > Merging [#7730](https://codecov.io/gh/ap

[GitHub] [airflow] codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730#issuecomment-599199404 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7730?src=pr&el=h1) Report > Merging [#7730](https://codecov.io/gh/ap

[GitHub] [airflow] turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
turbaszek commented on a change in pull request #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#discussion_r396086714 ## File path: airflow/providers/hashicorp/secrets/vault.py ## @@ -40,10 +40,10 @@ class V

[GitHub] [airflow] codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#issuecomment-602132463 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7795?src=pr&el=h1) Report > Merging [#7795](https://codeco

[GitHub] [airflow] mik-laj opened a new pull request #7803: Introduce types in airflow.bin.cli

2020-03-22 Thread GitBox
mik-laj opened a new pull request #7803: Introduce types in airflow.bin.cli URL: https://github.com/apache/airflow/pull/7803 --- Issue link: WILL BE INSERTED BY [boring-cyborg](https://github.com/kaxil/boring-cyborg) Make sure to mark the boxes below before creating PR: [x]

[GitHub] [airflow] mik-laj commented on issue #7797: Test installation of all provider packages

2020-03-22 Thread GitBox
mik-laj commented on issue #7797: Test installation of all provider packages URL: https://github.com/apache/airflow/pull/7797#issuecomment-602192290 I started reviewwing. This is an automated message from the Apache Git Servic

[GitHub] [airflow] codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#issuecomment-602132463 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7795?src=pr&el=h1) Report > Merging [#7795](https://codeco

[GitHub] [airflow] codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#issuecomment-602132463 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7795?src=pr&el=h1) Report > Merging [#7795](https://codeco

[GitHub] [airflow] codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#issuecomment-602132463 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7795?src=pr&el=h1) Report > Merging [#7795](https://codeco

[GitHub] [airflow] codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#issuecomment-602132463 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7795?src=pr&el=h1) Report > Merging [#7795](https://codeco

[GitHub] [airflow] mik-laj commented on a change in pull request #7797: Test installation of all provider packages

2020-03-22 Thread GitBox
mik-laj commented on a change in pull request #7797: Test installation of all provider packages URL: https://github.com/apache/airflow/pull/7797#discussion_r396088360 ## File path: scripts/ci/ci_prepare_backport_packages.sh ## @@ -20,14 +20,47 @@ export PYTHON_VERSION=${PY

[GitHub] [airflow] codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager

2020-03-22 Thread GitBox
codecov-io edited a comment on issue #7795: [AIRFLOW-7104] Add Secret backend for GCP Secrets Manager URL: https://github.com/apache/airflow/pull/7795#issuecomment-602132463 # [Codecov](https://codecov.io/gh/apache/airflow/pull/7795?src=pr&el=h1) Report > Merging [#7795](https://codeco

[GitHub] [airflow] mik-laj commented on a change in pull request #7797: Test installation of all provider packages

2020-03-22 Thread GitBox
mik-laj commented on a change in pull request #7797: Test installation of all provider packages URL: https://github.com/apache/airflow/pull/7797#discussion_r396088415 ## File path: scripts/ci/ci_prepare_backport_packages.sh ## @@ -20,14 +20,47 @@ export PYTHON_VERSION=${PY

[jira] [Commented] (AIRFLOW-7067) Add apache-airflow-pinned version

2020-03-22 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-7067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17064256#comment-17064256 ] ASF GitHub Bot commented on AIRFLOW-7067: - potiuk commented on pull request #77

[GitHub] [airflow] potiuk merged pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow

2020-03-22 Thread GitBox
potiuk merged pull request #7730: [AIRFLOW-7067] Pinned version of Apache Airflow URL: https://github.com/apache/airflow/pull/7730 This is an automated message from the Apache Git Service. To respond to the message, please l

  1   2   3   4   >