[GitHub] [airflow] stale[bot] commented on pull request #8884: Use env_vars to for reload config test.

2020-07-03 Thread GitBox
stale[bot] commented on pull request #8884: URL: https://github.com/apache/airflow/pull/8884#issuecomment-653726263 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[GitHub] [airflow] ramdharam commented on issue #9542: on_failure_callback function for external task sensor

2020-07-03 Thread GitBox
ramdharam commented on issue #9542: URL: https://github.com/apache/airflow/issues/9542#issuecomment-653720188 I can pick this up if this seems a valid use case. This is an automated message from the Apache Git Service. To res

[GitHub] [airflow] tirkarthi opened a new pull request #9649: Import ABC from collections.abc

2020-07-03 Thread GitBox
tirkarthi opened a new pull request #9649: URL: https://github.com/apache/airflow/pull/9649 `collections.abc` should be used while using ABC. `collections` is used which is incorrect. One of the other dependencies like logging import `collections.abc` and thus shadows this error. Relevant

[GitHub] [airflow] j-y-matsubara commented on pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on pull request #9531: URL: https://github.com/apache/airflow/pull/9531#issuecomment-653715178 > @j-y-matsubara would you mind a rebase? The k8s tests should be fixed now Sure. All tests were passed!

[airflow] tag nightly-master updated (63a8c79 -> 5e4b801)

2020-07-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to tag nightly-master in repository https://gitbox.apache.org/repos/asf/airflow.git. *** WARNING: tag nightly-master was modified! *** from 63a8c79 (commit) to 5e4b801 (commit) from 63

[GitHub] [airflow] ephraimbuddy opened a new pull request #9648: Update airflow to use the latest Flask App Builder

2020-07-03 Thread GitBox
ephraimbuddy opened a new pull request #9648: URL: https://github.com/apache/airflow/pull/9648 --- Make sure to mark the boxes below before creating PR: [x] - [ ] Description above provides context of the change - [ ] Unit tests coverage for changes (not needed for documenta

[GitHub] [airflow] Acehaidrey commented on pull request #9544: Add metric for scheduling delay between first run task & expected start time

2020-07-03 Thread GitBox
Acehaidrey commented on pull request #9544: URL: https://github.com/apache/airflow/pull/9544#issuecomment-653705002 sorry to keep pinging @ashb , if you get a chance This is an automated message from the Apache Git Service. T

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449726296 ## File path: airflow/configuration.py ## @@ -267,17 +271,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449726296 ## File path: airflow/configuration.py ## @@ -267,17 +271,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] houqp commented on pull request #9502: generate go client from openapi spec

2020-07-03 Thread GitBox
houqp commented on pull request #9502: URL: https://github.com/apache/airflow/pull/9502#issuecomment-653702139 @potiuk moved codegen validation to separate workflow. This is an automated message from the Apache Git Service. T

[GitHub] [airflow] vanka56 commented on a change in pull request #9472: Add drop_partition functionality for HiveMetastoreHook

2020-07-03 Thread GitBox
vanka56 commented on a change in pull request #9472: URL: https://github.com/apache/airflow/pull/9472#discussion_r449715800 ## File path: tests/providers/apache/hive/hooks/test_hive.py ## @@ -383,6 +383,10 @@ def test_table_exists(self): self.hook.table_exists(str(

[airflow] branch master updated: Test are triggered now on more changes. (#9646)

2020-07-03 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 5e4b801 Test are triggered now on more changes

[GitHub] [airflow] potiuk merged pull request #9646: Test are triggered now on more changes.

2020-07-03 Thread GitBox
potiuk merged pull request #9646: URL: https://github.com/apache/airflow/pull/9646 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 t

[GitHub] [airflow] turbaszek commented on pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
turbaszek commented on pull request #9531: URL: https://github.com/apache/airflow/pull/9531#issuecomment-653675958 @j-y-matsubara would you mind a rebase? The k8s tests should be fixed now This is an automated message from t

[GitHub] [airflow] takunnithan commented on a change in pull request #9556: API Endpoint - DagRuns Batch

2020-07-03 Thread GitBox
takunnithan commented on a change in pull request #9556: URL: https://github.com/apache/airflow/pull/9556#discussion_r449701317 ## File path: airflow/api_connexion/schemas/dag_run_schema.py ## @@ -80,15 +80,15 @@ class Meta: datetimeformat = 'iso' strict = Tru

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #9556: API Endpoint - DagRuns Batch

2020-07-03 Thread GitBox
ephraimbuddy commented on a change in pull request #9556: URL: https://github.com/apache/airflow/pull/9556#discussion_r449699768 ## File path: airflow/api_connexion/schemas/dag_run_schema.py ## @@ -80,15 +80,15 @@ class Meta: datetimeformat = 'iso' strict = Tr

[airflow] branch master updated: Fixed failing Kubernetes tests after deny_all for experimental API (#9647)

2020-07-03 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash 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 be6ed86 Fixed failing Kubernetes tests after deny

[GitHub] [airflow] ashb merged pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
ashb merged pull request #9647: URL: https://github.com/apache/airflow/pull/9647 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] ashb commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
ashb commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653669532 Monorepo FTW. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] takunnithan commented on a change in pull request #9556: API Endpoint - DagRuns Batch

2020-07-03 Thread GitBox
takunnithan commented on a change in pull request #9556: URL: https://github.com/apache/airflow/pull/9556#discussion_r449696774 ## File path: tests/api_connexion/endpoints/test_dag_run_endpoint.py ## @@ -346,6 +346,243 @@ def test_end_date_gte_lte(self, url, expected_dag_run_i

[GitHub] [airflow] takunnithan commented on a change in pull request #9556: API Endpoint - DagRuns Batch

2020-07-03 Thread GitBox
takunnithan commented on a change in pull request #9556: URL: https://github.com/apache/airflow/pull/9556#discussion_r449696638 ## File path: tests/api_connexion/endpoints/test_dag_run_endpoint.py ## @@ -346,6 +346,243 @@ def test_end_date_gte_lte(self, url, expected_dag_run_i

[GitHub] [airflow] takunnithan commented on a change in pull request #9556: API Endpoint - DagRuns Batch

2020-07-03 Thread GitBox
takunnithan commented on a change in pull request #9556: URL: https://github.com/apache/airflow/pull/9556#discussion_r449696389 ## File path: airflow/api_connexion/schemas/dag_run_schema.py ## @@ -72,5 +72,25 @@ class DAGRunCollectionSchema(Schema): total_entries = fields.

[jira] [Commented] (AIRFLOW-6786) Adding KafkaConsumerHook, KafkaProducerHook, and KafkaSensor

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

[jira] [Commented] (AIRFLOW-3515) Remove the run_duration option

2020-07-03 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-3515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17150812#comment-17150812 ] ASF GitHub Bot commented on AIRFLOW-3515: - Fokko commented on pull request #432

[GitHub] [airflow] kaxil edited a comment on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil edited a comment on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653649541 > Please do. I think it's the "eat cake and have it too" case. We can fulfill all your expectations (separate releases, issues for users in separate repos) while keeping mu

[GitHub] [airflow] kaxil commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653649541 > Please do. I think it's the "eat cake and have it too" case. We can fulfill all your expectations (separate releases, issues for users in separate repos) while keeping much lowe

[GitHub] [airflow] kaxil commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653648597 > > On the flip side changes in the Helm Chart should not affect Airflow's CI. In this case, the default of disabling API should just be a change in Airflow and should not be a ch

[GitHub] [airflow] potiuk edited a comment on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk edited a comment on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653646797 > On the flip side changes in the Helm Chart should not affect Airflow's CI. In this case, the default of disabling API should just be a change in Airflow and should not b

[GitHub] [airflow] potiuk commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653646797 > On the flip side changes in the Helm Chart should not affect Airflow's CI. In this case, the default of disabling API should just be a change in Airflow and should not be a cha

[GitHub] [airflow-client-go] houqp commented on pull request #1: Add generated go client

2020-07-03 Thread GitBox
houqp commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-653646372 @turbaszek added license header to ci config. This is an automated message from the Apache Git Service. To res

[GitHub] [airflow] kaxil commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653644916 On the flip side changes in the Helm Chart should not affect Airflow's CI. In this case, the default of disabling API should just be a change in Airflow and should not be a change

[GitHub] [airflow] potiuk commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653643945 Having a dedicated config is much nicer solution. And if we were to do this for split repos we would have to implement "workaround" first, "good solution" in a chart repo, releas

[GitHub] [airflow] kaxil commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653643433 But why not use Environment Variables to set configs. That's what we want to allow users to do too with the Chart, they would like to override any config. ---

[GitHub] [airflow] potiuk commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653643245 > Well you could just set up Environment variable, here > > https://github.com/apache/airflow/blob/master/chart/values.yaml#L94-L98 > > So theory and in-practise here

[GitHub] [airflow] kaxil commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653642913 Well you could just set up Environment variable, here https://github.com/apache/airflow/blob/master/chart/values.yaml#L94-L98 So theory and in-practise here are not di

[GitHub] [airflow] potiuk commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653642786 > I think it should still work. In theory, the chart should allow overriding any runtime Airflow configs. In this case we could have used Environment Variable to set config. If w

[GitHub] [airflow] kaxil edited a comment on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil edited a comment on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653642379 > Side note > > @kaxil @dimberman @ashb @turbaszek @mik-laj - this is an example of where splitting to separate repo would have been problematic. > > This chan

[GitHub] [airflow] kaxil edited a comment on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil edited a comment on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653642379 > Side note > > @kaxil @dimberman @ashb @turbaszek @mik-laj - this is an example of where splitting to separate repo would have been problematic. > > This chan

[GitHub] [airflow] kaxil commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
kaxil commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653642379 > Side note > > @kaxil @dimberman @ashb @turbaszek @mik-laj - this is an example of where splitting to separate repo would have been problematic. > > This change shou

[GitHub] [airflow] potiuk edited a comment on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk edited a comment on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653639691 Side note @kaxil @dimberman @ashb @turbaszek @mik-laj - this is an example of where splitting to separate repo would have been problematic. This change should

[GitHub] [airflow] potiuk commented on pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk commented on pull request #9647: URL: https://github.com/apache/airflow/pull/9647#issuecomment-653639691 Side note @kaxil @dimberman @ashb @turbaszek @mik-laj - this is an example of where splitting to separate repo would have been problematic. This change should have b

[GitHub] [airflow] potiuk opened a new pull request #9647: Fixed failing Kubernetes tests after deny_all for experimental API

2020-07-03 Thread GitBox
potiuk opened a new pull request #9647: URL: https://github.com/apache/airflow/pull/9647 --- Make sure to mark the boxes below before creating PR: [x] - [x] Description above provides context of the change - [x] Unit tests coverage for changes (not needed for documentation chang

[GitHub] [airflow] potiuk opened a new pull request #9646: Test are triggered now on more changes.

2020-07-03 Thread GitBox
potiuk opened a new pull request #9646: URL: https://github.com/apache/airflow/pull/9646 --- Make sure to mark the boxes below before creating PR: [x] - [x] Description above provides context of the change - [x] Unit tests coverage for changes (not needed for documentation chang

[GitHub] [airflow] j-y-matsubara commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449667794 ## File path: tests/plugins/test_plugin_ignore.py ## @@ -0,0 +1,96 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more co

[airflow] branch master updated (fddc572 -> a99aaeb)

2020-07-03 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 fddc572 Customizable page size limit in API (#9431) add a99aaeb Allow setting Hashicorp Vault token from F

[airflow] branch master updated (fddc572 -> a99aaeb)

2020-07-03 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 fddc572 Customizable page size limit in API (#9431) add a99aaeb Allow setting Hashicorp Vault token from F

[GitHub] [airflow] kaxil merged pull request #9644: Allow setting Hashicorp Vault token from File

2020-07-03 Thread GitBox
kaxil merged pull request #9644: URL: https://github.com/apache/airflow/pull/9644 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] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449663507 ## File path: airflow/providers/amazon/aws/secrets/secrets_manager.py ## @@ -62,13 +67,15 @@ def __init__( self, connections_prefix: str =

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449663361 ## File path: airflow/configuration.py ## @@ -267,17 +271,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449648643 ## File path: airflow/configuration.py ## @@ -267,17 +271,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449648643 ## File path: airflow/configuration.py ## @@ -267,17 +271,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449648643 ## File path: airflow/configuration.py ## @@ -267,17 +271,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449647991 ## File path: airflow/providers/amazon/aws/secrets/secrets_manager.py ## @@ -62,13 +67,15 @@ def __init__( self, connections_prefix: str =

[GitHub] [airflow] ashb commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
ashb commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449642855 ## File path: airflow/configuration.py ## @@ -115,7 +117,9 @@ class AirflowConfigParser(ConfigParser): # These configuration elements can be fetched as

[GitHub] [airflow] kaxil commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449642561 ## File path: airflow/configuration.py ## @@ -267,17 +272,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] ashb commented on a change in pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
ashb commented on a change in pull request #9645: URL: https://github.com/apache/airflow/pull/9645#discussion_r449639584 ## File path: airflow/configuration.py ## @@ -267,17 +272,49 @@ def _get_env_var_option(self, section, key): env_var_cmd = env_var + '_CMD'

[GitHub] [airflow] kaxil opened a new pull request #9645: Get Airflow configs with sensitive data from Secret Backends

2020-07-03 Thread GitBox
kaxil opened a new pull request #9645: URL: https://github.com/apache/airflow/pull/9645 We should be able to get the followings config containing passwords and other sensitive data from Secret Backends: ``` ('core', 'sql_alchemy_conn'), ('core', 'fernet_key'),

[GitHub] [airflow] potiuk commented on pull request #9637: breeze Dockerfile.ci gcc ver

2020-07-03 Thread GitBox
potiuk commented on pull request #9637: URL: https://github.com/apache/airflow/pull/9637#issuecomment-653598471 @xwydq - Maybe a better solution will be to do what I've done in Production dockerfile - to add build-essential instead of manually adding gcc ? i think we are not concerned abou

[GitHub] [airflow] turbaszek commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
turbaszek commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449633646 ## File path: tests/plugins/test_plugin_ignore.py ## @@ -0,0 +1,96 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contri

[GitHub] [airflow] aneesh-joseph opened a new pull request #9644: add token_path param to vault client

2020-07-03 Thread GitBox
aneesh-joseph opened a new pull request #9644: URL: https://github.com/apache/airflow/pull/9644 --- Make sure to mark the boxes below before creating PR: [x] - [ ] Description above provides context of the change - [ ] Unit tests coverage for changes (not needed for documentatio

[GitHub] [airflow] potiuk closed issue #9017: Error on running this docker on kubernetes

2020-07-03 Thread GitBox
potiuk closed issue #9017: URL: https://github.com/apache/airflow/issues/9017 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 sp

[GitHub] [airflow] potiuk closed issue #7883: KubernetesPodOperator had an event of type Pending

2020-07-03 Thread GitBox
potiuk closed issue #7883: URL: https://github.com/apache/airflow/issues/7883 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 sp

[GitHub] [airflow] potiuk closed issue #9086: Task Instance Slots Available' FAILED

2020-07-03 Thread GitBox
potiuk closed issue #9086: URL: https://github.com/apache/airflow/issues/9086 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 sp

[GitHub] [airflow] potiuk closed issue #8716: Support on packaged dags

2020-07-03 Thread GitBox
potiuk closed issue #8716: URL: https://github.com/apache/airflow/issues/8716 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 sp

[GitHub] [airflow] potiuk closed issue #8416: Clustered Table is Not supported in BQ Operator Airflow 1.10.1

2020-07-03 Thread GitBox
potiuk closed issue #8416: URL: https://github.com/apache/airflow/issues/8416 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 sp

[GitHub] [airflow] potiuk closed issue #9533: Upstream failed whereas Upstream is alright

2020-07-03 Thread GitBox
potiuk closed issue #9533: URL: https://github.com/apache/airflow/issues/9533 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 sp

[GitHub] [airflow] potiuk closed issue #9324: How to create a role that can browse DAGs but not pause/unpause them?

2020-07-03 Thread GitBox
potiuk closed issue #9324: URL: https://github.com/apache/airflow/issues/9324 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 sp

[GitHub] [airflow] potiuk closed issue #8643: Can't setup enviroments for use secrets in docker-swarm

2020-07-03 Thread GitBox
potiuk closed issue #8643: URL: https://github.com/apache/airflow/issues/8643 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 sp

[GitHub] [airflow] potiuk closed issue #8905: Logging with Custom Filenames

2020-07-03 Thread GitBox
potiuk closed issue #8905: URL: https://github.com/apache/airflow/issues/8905 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 sp

[GitHub] [airflow] potiuk closed issue #9642: Wrong default mime_charset in Airflow 1.10.10

2020-07-03 Thread GitBox
potiuk closed issue #9642: URL: https://github.com/apache/airflow/issues/9642 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 sp

[GitHub] [airflow] potiuk closed issue #9643: SSL connection has been closed unexpectedly

2020-07-03 Thread GitBox
potiuk closed issue #9643: URL: https://github.com/apache/airflow/issues/9643 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 sp

[GitHub] [airflow] potiuk closed issue #9036: Task instance log_filepath doesn't include try_number

2020-07-03 Thread GitBox
potiuk closed issue #9036: URL: https://github.com/apache/airflow/issues/9036 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 sp

[GitHub] [airflow] potiuk closed issue #9519: How to use private_key for SSH Operator

2020-07-03 Thread GitBox
potiuk closed issue #9519: URL: https://github.com/apache/airflow/issues/9519 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 sp

[GitHub] [airflow] potiuk closed issue #8906: make success failed

2020-07-03 Thread GitBox
potiuk closed issue #8906: URL: https://github.com/apache/airflow/issues/8906 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 sp

[GitHub] [airflow] potiuk closed issue #7955: Passing network and config/secret to DockerSwarmOperator

2020-07-03 Thread GitBox
potiuk closed issue #7955: URL: https://github.com/apache/airflow/issues/7955 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 sp

[GitHub] [airflow] potiuk closed issue #8656: How to make Airflow to get DAGs from S3 bucket if KubernetesExecutor is used?

2020-07-03 Thread GitBox
potiuk closed issue #8656: URL: https://github.com/apache/airflow/issues/8656 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 sp

[GitHub] [airflow] potiuk commented on issue #9643: SSL connection has been closed unexpectedly

2020-07-03 Thread GitBox
potiuk commented on issue #9643: URL: https://github.com/apache/airflow/issues/9643#issuecomment-653590257 Unfortunately this is not a generic problem - it looks like this is not a generic airlfow problem but rather your configuration problem so it falls under "support request/query" whic

[GitHub] [airflow] bhavaniravi opened a new issue #9643: SSL connection has been closed unexpectedly

2020-07-03 Thread GitBox
bhavaniravi opened a new issue #9643: URL: https://github.com/apache/airflow/issues/9643 **Apache Airflow version**: 1.10.11 **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): 1.17.2 **Environment**: - **Cloud provider or hardware config

[GitHub] [airflow] j-y-matsubara commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449615325 ## File path: tests/plugins/test_ignore/subdir1/test_load_sub1.py ## @@ -0,0 +1,35 @@ +# +# Licensed to the Apache Software Foundation (ASF) under o

[GitHub] [airflow] j-y-matsubara commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449615325 ## File path: tests/plugins/test_ignore/subdir1/test_load_sub1.py ## @@ -0,0 +1,35 @@ +# +# Licensed to the Apache Software Foundation (ASF) under o

[GitHub] [airflow] j-y-matsubara commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449615325 ## File path: tests/plugins/test_ignore/subdir1/test_load_sub1.py ## @@ -0,0 +1,35 @@ +# +# Licensed to the Apache Software Foundation (ASF) under o

[GitHub] [airflow] boring-cyborg[bot] commented on issue #9642: Wrong default mime_charset in Airflow 1.10.10

2020-07-03 Thread GitBox
boring-cyborg[bot] commented on issue #9642: URL: https://github.com/apache/airflow/issues/9642#issuecomment-653578538 Thanks for opening your first issue here! Be sure to follow the issue template! This is an automated

[GitHub] [airflow] j-y-matsubara commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449615325 ## File path: tests/plugins/test_ignore/subdir1/test_load_sub1.py ## @@ -0,0 +1,35 @@ +# +# Licensed to the Apache Software Foundation (ASF) under o

[GitHub] [airflow] Libum opened a new issue #9642: Wrong default mime_charset in Airflow 1.10.10

2020-07-03 Thread GitBox
Libum opened a new issue #9642: URL: https://github.com/apache/airflow/issues/9642 Hello, I've encountered issue with default `mime_charset` in send_email utility in Airflow 1.10.10: https://github.com/apache/airflow/blob/317b0412383ccda571fbef568c9eabd70ab8e666/airflow/utils/ema

[GitHub] [airflow] j-y-matsubara commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449615262 ## File path: tests/plugins/test_plugin_ignore.py ## @@ -0,0 +1,89 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more co

[GitHub] [airflow] j-y-matsubara commented on a change in pull request #9531: Support .airflowignore for plugins

2020-07-03 Thread GitBox
j-y-matsubara commented on a change in pull request #9531: URL: https://github.com/apache/airflow/pull/9531#discussion_r449615325 ## File path: tests/plugins/test_ignore/subdir1/test_load_sub1.py ## @@ -0,0 +1,35 @@ +# +# Licensed to the Apache Software Foundation (ASF) under o

[GitHub] [airflow] elwinarens commented on a change in pull request #9639: Add secrets backend for microsoft azure key vault

2020-07-03 Thread GitBox
elwinarens commented on a change in pull request #9639: URL: https://github.com/apache/airflow/pull/9639#discussion_r449562999 ## File path: airflow/providers/microsoft/azure/secrets/azure_key_vault.py ## @@ -0,0 +1,107 @@ +""" +This module contains a secrets backend for Azure

[GitHub] [airflow] mik-laj commented on issue #9583: Attach roles to AD groups - Azure OAuth

2020-07-03 Thread GitBox
mik-laj commented on issue #9583: URL: https://github.com/apache/airflow/issues/9583#issuecomment-653522828 Is this helpful for you? I don't use Azure OAuth, so I'm not sure this change applies here. https://github.com/dpgaspar/Flask-AppBuilder/pull/1410 --

[GitHub] [airflow] elwinarens commented on a change in pull request #9639: Add secrets backend for microsoft azure key vault

2020-07-03 Thread GitBox
elwinarens commented on a change in pull request #9639: URL: https://github.com/apache/airflow/pull/9639#discussion_r449549782 ## File path: airflow/providers/microsoft/azure/secrets/azure_key_vault.py ## @@ -0,0 +1,107 @@ +""" +This module contains a secrets backend for Azure

[GitHub] [airflow] elwinarens commented on a change in pull request #9639: Add secrets backend for microsoft azure key vault

2020-07-03 Thread GitBox
elwinarens commented on a change in pull request #9639: URL: https://github.com/apache/airflow/pull/9639#discussion_r449549570 ## File path: tests/providers/microsoft/azure/secrets/test_azure_key_vault_backend.py ## @@ -0,0 +1,92 @@ +# Licensed to the Apache Software Foundatio

[GitHub] [airflow] kaxil commented on a change in pull request #9635: fix : _run_task_by_executor pickle_id is None

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9635: URL: https://github.com/apache/airflow/pull/9635#discussion_r449548340 ## File path: airflow/cli/commands/task_command.py ## @@ -75,6 +75,7 @@ def _run_task_by_executor(args, dag, ti): with create_session() as sess

[GitHub] [airflow] haidaraM commented on pull request #7407: [AIRFLOW-6786] Add KafkaConsumerHook, KafkaProduerHook and KafkaSensor

2020-07-03 Thread GitBox
haidaraM commented on pull request #7407: URL: https://github.com/apache/airflow/pull/7407#issuecomment-653510340 Any news on this PR ? This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [airflow] rafaelpierre commented on issue #9583: Attach roles to AD groups - Azure OAuth

2020-07-03 Thread GitBox
rafaelpierre commented on issue #9583: URL: https://github.com/apache/airflow/issues/9583#issuecomment-653499207 @sk2991 +1 > @sk2991 Would you mind sharing your Azure OAuth implementation? This is an automated

[GitHub] [airflow] ashb opened a new issue #9641: Double check UPDATING.md for "2.0" doesn't include things already released

2020-07-03 Thread GitBox
ashb opened a new issue #9641: URL: https://github.com/apache/airflow/issues/9641 We need to double check the UDPDATING.md has been correctly updated on mainline to correctly reflect features that have already been included in 1.10.x releases Specifically that anything included in e

[GitHub] [airflow] ashb opened a new issue #9640: Ensure Airflow 1.10.11 DB can be updated to master

2020-07-03 Thread GitBox
ashb opened a new issue #9640: URL: https://github.com/apache/airflow/issues/9640 To ensure our users can upgrade from Airflow 1.10.11 to 2.0 (when it comes) we need to double check that the Alembic migration "history" is correct. This might involve either creating a merge point, or

[GitHub] [airflow] nathadfield commented on pull request #8745: Install python-ldap library for FAB LDAP authorization with Python3

2020-07-03 Thread GitBox
nathadfield commented on pull request #8745: URL: https://github.com/apache/airflow/pull/8745#issuecomment-653489782 This is definitely important given that the non FAB UI will ultimately be going away. This is an automated

[GitHub] [airflow] kaxil commented on a change in pull request #9639: Add secrets backend for microsoft azure key vault

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9639: URL: https://github.com/apache/airflow/pull/9639#discussion_r449522855 ## File path: tests/providers/microsoft/azure/secrets/test_azure_key_vault_backend.py ## @@ -0,0 +1,92 @@ +# Licensed to the Apache Software Foundation (AS

[GitHub] [airflow] kaxil commented on a change in pull request #9639: Add secrets backend for microsoft azure key vault

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9639: URL: https://github.com/apache/airflow/pull/9639#discussion_r449522757 ## File path: tests/providers/microsoft/azure/secrets/test_azure_key_vault_backend.py ## @@ -0,0 +1,92 @@ +# Licensed to the Apache Software Foundation (AS

[GitHub] [airflow] kaxil commented on a change in pull request #9639: Add secrets backend for microsoft azure key vault

2020-07-03 Thread GitBox
kaxil commented on a change in pull request #9639: URL: https://github.com/apache/airflow/pull/9639#discussion_r449521224 ## File path: airflow/providers/microsoft/azure/secrets/azure_key_vault.py ## @@ -0,0 +1,107 @@ +""" +This module contains a secrets backend for Azure Key V

[GitHub] [airflow] turbaszek commented on a change in pull request #9631: Add function to get current context

2020-07-03 Thread GitBox
turbaszek commented on a change in pull request #9631: URL: https://github.com/apache/airflow/pull/9631#discussion_r449521543 ## File path: airflow/task/context/current.py ## @@ -0,0 +1,69 @@ +# Licensed to the Apache Software Foundation (ASF) under one Review comment:

  1   2   >