[jira] [Commented] (AIRFLOW-13) Migrate Travis CI to Apache repo

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-13?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16591222#comment-16591222 ] Gerardo Curiel commented on AIRFLOW-13: --- [~kaxilnaik] it seems to me this one can b

[GitHub] xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212533643 ## File path: airflow/contrib/hooks/bigquery_hook.py

[GitHub] xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212533643 ## File path: airflow/contrib/hooks/bigquery_hook.py

[jira] [Assigned] (AIRFLOW-507) Use Travis' trusty environment for CI

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-507?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gerardo Curiel reassigned AIRFLOW-507: -- Assignee: Gerardo Curiel > Use Travis' trusty environment for CI > --

[jira] [Commented] (AIRFLOW-507) Use Travis' trusty environment for CI

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-507?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16591205#comment-16591205 ] Gerardo Curiel commented on AIRFLOW-507: The TravisCI build uses trusty now. Thi

[jira] [Commented] (AIRFLOW-2806) test_mark_success_no_kill test breaks intermittently on CI

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2806?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16591204#comment-16591204 ] Gerardo Curiel commented on AIRFLOW-2806: - Hi [~tedmiston] , can we close this?

[jira] [Commented] (AIRFLOW-1042) Easy Unit Testing with Docker

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-1042?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16591197#comment-16591197 ] Gerardo Curiel commented on AIRFLOW-1042: - [~kaxilnaik] solved in https://issue

[jira] [Commented] (AIRFLOW-602) Unit Test Cases Doesn't run in Master Branch

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16591193#comment-16591193 ] Gerardo Curiel commented on AIRFLOW-602: [~kaxilnaik] can we close this one in f

[jira] [Commented] (AIRFLOW-404) Travis cache can get borked make sure to retry downloading if unpacking fails

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16591192#comment-16591192 ] Gerardo Curiel commented on AIRFLOW-404: This one can be closed [~kaxilnaik] >

[jira] [Commented] (AIRFLOW-2157) Builds in TravisCI are so unstable now

2018-08-23 Thread Gerardo Curiel (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16591190#comment-16591190 ] Gerardo Curiel commented on AIRFLOW-2157: - This one can be closed [~kaxilnaik]

[GitHub] tedmiston commented on issue #3656: [AIRFLOW-2803] Fix all ESLint issues

2018-08-23 Thread GitBox
tedmiston commented on issue #3656: [AIRFLOW-2803] Fix all ESLint issues URL: https://github.com/apache/incubator-airflow/pull/3656#issuecomment-415647080 @verdan Please see the revised diff / new squashed commit, and review when you have a chance. There were some opinionated decisi

[GitHub] gerardo commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose

2018-08-23 Thread GitBox
gerardo commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose URL: https://github.com/apache/incubator-airflow/pull/3797#issuecomment-415643427 @dimberman given the Kubernetes ci scripts runs outside docker, [this line](https://github.com/apache/incubator-airflow/pu

[GitHub] gerardo commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose

2018-08-23 Thread GitBox
gerardo commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose URL: https://github.com/apache/incubator-airflow/pull/3797#issuecomment-415640654 @dimberman I'll have a look now This is an automated mess

[jira] [Created] (AIRFLOW-2953) Installing postgres and gcp_api doesn't install PostgresToGoogleCloudStorageOperator

2018-08-23 Thread robert kelly (JIRA)
robert kelly created AIRFLOW-2953: - Summary: Installing postgres and gcp_api doesn't install PostgresToGoogleCloudStorageOperator Key: AIRFLOW-2953 URL: https://issues.apache.org/jira/browse/AIRFLOW-2953

[GitHub] XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212488052 ## File path: airflow/contrib/operators/sftp_operator.py ## @@ -77,13 +8

[GitHub] XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212487903 ## File path: airflow/contrib/operators/ssh_operator.py ## @@ -68,14 +72

[GitHub] XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212488849 ## File path: airflow/contrib/operators/sftp_operator.py ## @@ -77,13 +8

[GitHub] XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212488052 ## File path: airflow/contrib/operators/sftp_operator.py ## @@ -77,13 +8

[GitHub] XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212487903 ## File path: airflow/contrib/operators/ssh_operator.py ## @@ -68,14 +72

[GitHub] XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
XD-DENG commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212487852 ## File path: airflow/contrib/operators/sftp_operator.py ## @@ -77,13 +8

[jira] [Commented] (AIRFLOW-2951) dag_run end_date Null after a dag is finished

2018-08-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590912#comment-16590912 ] ASF GitHub Bot commented on AIRFLOW-2951: - YingboWang opened a new pull request

[GitHub] YingboWang opened a new pull request #3798: [AIRFLOW-2951] Update dag_run table end_date when state change

2018-08-23 Thread GitBox
YingboWang opened a new pull request #3798: [AIRFLOW-2951] Update dag_run table end_date when state change URL: https://github.com/apache/incubator-airflow/pull/3798 The existing airflow only change dag_run table end_date value when a user teminate a dag in web UI. The end_date will not

[GitHub] dimberman commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose

2018-08-23 Thread GitBox
dimberman commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose URL: https://github.com/apache/incubator-airflow/pull/3797#issuecomment-415601935 https://github.com/apache/incubator-airflow/commit/ede67299c4b2b64fc0a72a25838ba29682463d81#diff-354f30a63fb0907d4ad5726

[GitHub] dimberman commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose

2018-08-23 Thread GitBox
dimberman commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose URL: https://github.com/apache/incubator-airflow/pull/3797#issuecomment-415597921 @bolkedebruin @Fokko @kaxil PTAL. I'm going to try to get this all working tonight (though I assume the tests should pa

[GitHub] dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415596492 Ok so that was caused by the recent dockerized ci pipeline. I'm addressing the issue here https://github.com/apache

[jira] [Commented] (AIRFLOW-2938) Invalid 'extra' field in connection can raise an AttributeError when attempting to edit

2018-08-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590896#comment-16590896 ] ASF GitHub Bot commented on AIRFLOW-2938: - feng-tao closed pull request #3785:

[GitHub] feng-tao closed pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection …

2018-08-23 Thread GitBox
feng-tao closed pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection … URL: https://github.com/apache/incubator-airflow/pull/3785 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sak

[GitHub] dimberman opened a new pull request #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose

2018-08-23 Thread GitBox
dimberman opened a new pull request #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose URL: https://github.com/apache/incubator-airflow/pull/3797 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow Jira](https://issues.ap

[jira] [Commented] (AIRFLOW-2952) Dockerized CI pipeline has silently broken integration testing for KubernetesExecutor

2018-08-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2952?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590891#comment-16590891 ] ASF GitHub Bot commented on AIRFLOW-2952: - dimberman opened a new pull request

[jira] [Created] (AIRFLOW-2952) Dockerized CI pipeline has silently broken integration testing for KubernetesExecutor

2018-08-23 Thread Daniel Imberman (JIRA)
Daniel Imberman created AIRFLOW-2952: Summary: Dockerized CI pipeline has silently broken integration testing for KubernetesExecutor Key: AIRFLOW-2952 URL: https://issues.apache.org/jira/browse/AIRFLOW-2952

[GitHub] dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415588605 Wait. No. This is on the master branch. Thats... not good. Going to investigate further. -

[GitHub] dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415587425 @jpds It looks like you removed the kube/deploy folder but it's not removed from the setup_kubernetes script https

[GitHub] tswast commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
tswast commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212471376 ## File path: airflow/contrib/hooks/bigquery_hook.py ###

[GitHub] dimberman commented on a change in pull request #3782: [AIRFLOW-2936] Use official Python images as base image for Docker

2018-08-23 Thread GitBox
dimberman commented on a change in pull request #3782: [AIRFLOW-2936] Use official Python images as base image for Docker URL: https://github.com/apache/incubator-airflow/pull/3782#discussion_r212471258 ## File path: scripts/ci/kubernetes/docker/airflow-init.sh ## @@ -17,9

[GitHub] tswast commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
tswast commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212469303 ## File path: airflow/contrib/hooks/bigquery_hook.py ###

[GitHub] xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212469186 ## File path: airflow/contrib/hooks/bigquery_hook.py

[GitHub] xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212466137 ## File path: airflow/contrib/hooks/bigquery_hook.py

[GitHub] xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212466137 ## File path: airflow/contrib/hooks/bigquery_hook.py

[GitHub] Fokko commented on issue #3780: [AIRFLOW-2933] Enable Codecov on Docker-CI Build

2018-08-23 Thread GitBox
Fokko commented on issue #3780: [AIRFLOW-2933] Enable Codecov on Docker-CI Build URL: https://github.com/apache/incubator-airflow/pull/3780#issuecomment-415578973 So the coverage report gets uploaded: https://codecov.io/gh/apache/incubator-airflow/commit/c1566951dcecb91a2a739700e2b84f3dcee

[GitHub] tswast commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
tswast commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212463435 ## File path: airflow/contrib/hooks/bigquery_hook.py ###

[jira] [Assigned] (AIRFLOW-2237) Airflow SLA is not checked for subdag tasks

2018-08-23 Thread Tomas Zubiri (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2237?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tomas Zubiri reassigned AIRFLOW-2237: - Assignee: Tomas Zubiri > Airflow SLA is not checked for subdag tasks > ---

[GitHub] feng-tao commented on a change in pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection …

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection … URL: https://github.com/apache/incubator-airflow/pull/3785#discussion_r212455107 ## File path: airflow/www_rbac/views.py ## @@ -1923,7 +1923,10 @@ d

[GitHub] feng-tao commented on issue #3790: [AIRFLOW-2994] Fix command status check in Qubole Check operator

2018-08-23 Thread GitBox
feng-tao commented on issue #3790: [AIRFLOW-2994] Fix command status check in Qubole Check operator URL: https://github.com/apache/incubator-airflow/pull/3790#issuecomment-415564978 It seems python3 fails. Do you know if it is related? -

[GitHub] feng-tao commented on a change in pull request #3796: [AIRFLOW-2824] - Add config to disable default conn creation

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3796: [AIRFLOW-2824] - Add config to disable default conn creation URL: https://github.com/apache/incubator-airflow/pull/3796#discussion_r212450745 ## File path: airflow/utils/db.py ## @@ -286,6 +284,16 @@ def initdb(rbac=Fal

[jira] [Commented] (AIRFLOW-2824) Disable loading of default connections via airflow config

2018-08-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2824?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590785#comment-16590785 ] ASF GitHub Bot commented on AIRFLOW-2824: - andscoop opened a new pull request #

[jira] [Work started] (AIRFLOW-2951) dag_run end_date Null after a dag is finished

2018-08-23 Thread Yingbo Wang (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2951?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Work on AIRFLOW-2951 started by Yingbo Wang. > dag_run end_date Null after a dag is finished >

[jira] [Created] (AIRFLOW-2951) dag_run end_date Null after a dag is finished

2018-08-23 Thread Yingbo Wang (JIRA)
Yingbo Wang created AIRFLOW-2951: Summary: dag_run end_date Null after a dag is finished Key: AIRFLOW-2951 URL: https://issues.apache.org/jira/browse/AIRFLOW-2951 Project: Apache Airflow Issu

[GitHub] andscoop opened a new pull request #3796: [AIRFLOW-2824] - Add config to disable default conn creation

2018-08-23 Thread GitBox
andscoop opened a new pull request #3796: [AIRFLOW-2824] - Add config to disable default conn creation URL: https://github.com/apache/incubator-airflow/pull/3796 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://

[jira] [Commented] (AIRFLOW-2932) GoogleCloudStorageHook - allow compression of file

2018-08-23 Thread Iuliia Volkova (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2932?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590781#comment-16590781 ] Iuliia Volkova commented on AIRFLOW-2932: - [~kaxilnaik] and [~tswast] what do y

[GitHub] xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212444862 ## File path: tests/contrib/hooks/test_bigquery_hook.py #

[GitHub] feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212443735 ## File path: airflow/contrib/operators/sftp_operator.py ## @@ -77,13 +

[GitHub] xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs'

2018-08-23 Thread GitBox
xnuinside commented on a change in pull request #3733: [AIRFLOW-491] Add cache parameter in BigQuery query method - with 'api_resource_configs' URL: https://github.com/apache/incubator-airflow/pull/3733#discussion_r212443568 ## File path: tests/contrib/hooks/test_bigquery_hook.py #

[GitHub] feng-tao commented on a change in pull request #3795: [AIRFLOW-2949] Add syntax highlight for single quote strings

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3795: [AIRFLOW-2949] Add syntax highlight for single quote strings URL: https://github.com/apache/incubator-airflow/pull/3795#discussion_r212443447 ## File path: airflow/www/static/main.css ## @@ -262,3 +262,4 @@ div.square {

[GitHub] feng-tao commented on a change in pull request #3791: Adding King.com to the list of companies.

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3791: Adding King.com to the list of companies. URL: https://github.com/apache/incubator-airflow/pull/3791#discussion_r212442372 ## File path: README.md ## @@ -184,6 +184,7 @@ Currently **officially** using Airflow: 1. [JobT

[GitHub] feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212441692 ## File path: airflow/contrib/operators/ssh_operator.py ## @@ -68,14 +7

[GitHub] feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212442049 ## File path: airflow/contrib/operators/sftp_operator.py ## @@ -77,13 +

[GitHub] feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793#discussion_r212441652 ## File path: airflow/contrib/operators/sftp_operator.py ## @@ -77,13 +

[jira] [Commented] (AIRFLOW-2949) Syntax Highlight for Single Quote

2018-08-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590723#comment-16590723 ] ASF GitHub Bot commented on AIRFLOW-2949: - tzulberti-jampp opened a new pull re

[GitHub] tzulberti-jampp opened a new pull request #3795: [AIRFLOW-2949] Add syntax highlight for single quote strings

2018-08-23 Thread GitBox
tzulberti-jampp opened a new pull request #3795: [AIRFLOW-2949] Add syntax highlight for single quote strings URL: https://github.com/apache/incubator-airflow/pull/3795 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](h

[jira] [Created] (AIRFLOW-2950) Running Airflow behind a proxy

2018-08-23 Thread Shivakumar Gopalakrishnan (JIRA)
Shivakumar Gopalakrishnan created AIRFLOW-2950: -- Summary: Running Airflow behind a proxy Key: AIRFLOW-2950 URL: https://issues.apache.org/jira/browse/AIRFLOW-2950 Project: Apache Airflow

[jira] [Updated] (AIRFLOW-2949) Syntax Highlight for Single Quote

2018-08-23 Thread Tomas Zulberti (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2949?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tomas Zulberti updated AIRFLOW-2949: Summary: Syntax Highlight for Single Quote (was: Syntax highlight for single quote) > S

[jira] [Updated] (AIRFLOW-2949) Syntax highlight for single quote

2018-08-23 Thread Tomas Zulberti (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2949?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tomas Zulberti updated AIRFLOW-2949: Summary: Syntax highlight for single quote (was: Syntax Highligt for single quote) > Sy

[jira] [Created] (AIRFLOW-2949) Syntax Highligt for single quote

2018-08-23 Thread Tomas Zulberti (JIRA)
Tomas Zulberti created AIRFLOW-2949: --- Summary: Syntax Highligt for single quote Key: AIRFLOW-2949 URL: https://issues.apache.org/jira/browse/AIRFLOW-2949 Project: Apache Airflow Issue Type:

[GitHub] feng-tao closed pull request #3794: [AIRFLOW-XXX] Added G Adventures to Users

2018-08-23 Thread GitBox
feng-tao closed pull request #3794: [AIRFLOW-XXX] Added G Adventures to Users URL: https://github.com/apache/incubator-airflow/pull/3794 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] feng-tao commented on a change in pull request #3786: [AIRFLOW-XXX] Note on min_file_process_interval

2018-08-23 Thread GitBox
feng-tao commented on a change in pull request #3786: [AIRFLOW-XXX] Note on min_file_process_interval URL: https://github.com/apache/incubator-airflow/pull/3786#discussion_r212398092 ## File path: airflow/config_templates/default_airflow.cfg ## @@ -432,7 +432,8 @@ schedule

[GitHub] samuelmullin opened a new pull request #3794: [AIRFLOW-XXX] Added G Adventures to Users

2018-08-23 Thread GitBox
samuelmullin opened a new pull request #3794: [AIRFLOW-XXX] Added G Adventures to Users URL: https://github.com/apache/incubator-airflow/pull/3794 Make sure you have checked _all_ steps below. ### Jira - [X] My PR addresses the following [Airflow Jira](https://issues.apache.o

[GitHub] wrp commented on a change in pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection …

2018-08-23 Thread GitBox
wrp commented on a change in pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection … URL: https://github.com/apache/incubator-airflow/pull/3785#discussion_r212363527 ## File path: airflow/www_rbac/views.py ## @@ -1923,7 +1923,10 @@ def pr

[GitHub] gsemet commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
gsemet commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415460295 @ashb I do not see what your remark has to do with Airflow, to what I see and use in other charts in this repository,

[GitHub] dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415454497 I think we're in a similar mind about the current state of helm. I think this'll be useful for people wanting to

[GitHub] ashb commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
ashb commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415451730 Yeah, I'm waiting quite eagerly for Helm3. I'm not a fan of this right now, but I also don't feel that strongly about it

[GitHub] tedmiston commented on issue #3656: [WIP][AIRFLOW-2803] Fix all ESLint issues

2018-08-23 Thread GitBox
tedmiston commented on issue #3656: [WIP][AIRFLOW-2803] Fix all ESLint issues URL: https://github.com/apache/incubator-airflow/pull/3656#issuecomment-415451613 @verdan Yes! I'll push up my new work on this today. This is an a

[jira] [Commented] (AIRFLOW-2948) Arg checking & better doc for SSHOperator and SFTPOperator

2018-08-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2948?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590346#comment-16590346 ] ASF GitHub Bot commented on AIRFLOW-2948: - XD-DENG opened a new pull request #3

[GitHub] XD-DENG opened a new pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator

2018-08-23 Thread GitBox
XD-DENG opened a new pull request #3793: [AIRFLOW-2948] Arg check & better doc - SSHOperator & SFTPOperator URL: https://github.com/apache/incubator-airflow/pull/3793 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](htt

[jira] [Created] (AIRFLOW-2948) Arg checking & better doc for SSHOperator and SFTPOperator

2018-08-23 Thread Xiaodong DENG (JIRA)
Xiaodong DENG created AIRFLOW-2948: -- Summary: Arg checking & better doc for SSHOperator and SFTPOperator Key: AIRFLOW-2948 URL: https://issues.apache.org/jira/browse/AIRFLOW-2948 Project: Apache Airfl

[GitHub] dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
dimberman commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415448465 @ashb I agree that helms super-user tiller is a really bad security model but there are best practices users can

[GitHub] ashb commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
ashb commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415446816 I'm also not a fan of that model as it's a resource hog ;) -

[GitHub] jpds edited a comment on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
jpds edited a comment on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415445252 @ashb That's not a hard-requirement from Helm, it's completely possible to deploy Tiller in a namespace with only

[GitHub] jpds commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
jpds commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415445252 @ashb That's not a hard-requirement from Helm, it's completely possible to deploy Tiller in a namespace. --

[GitHub] ashb commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
ashb commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415444761 This is a -1 from me - I don't like Helm's requirement to be run as an all-powerful user in the cluster. --

[GitHub] jpds commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
jpds commented on issue #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792#issuecomment-415443337 @dimberman @gsemet This is an automated message from th

[jira] [Commented] (AIRFLOW-2947) Example Kubernetes manifests should be a Helm chart

2018-08-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2947?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590329#comment-16590329 ] ASF GitHub Bot commented on AIRFLOW-2947: - jpds opened a new pull request #3792

[GitHub] jpds opened a new pull request #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor

2018-08-23 Thread GitBox
jpds opened a new pull request #3792: [AIRFLOW-2947] Added Helm chart for Kubernetes executor URL: https://github.com/apache/incubator-airflow/pull/3792 ### Jira - [X] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW/) issues and references

[jira] [Created] (AIRFLOW-2947) Example Kubernetes manifests should be a Helm chart

2018-08-23 Thread Jon Davies (JIRA)
Jon Davies created AIRFLOW-2947: --- Summary: Example Kubernetes manifests should be a Helm chart Key: AIRFLOW-2947 URL: https://issues.apache.org/jira/browse/AIRFLOW-2947 Project: Apache Airflow

[GitHub] nathadfield opened a new pull request #3791: Adding King.com to the list of companies.

2018-08-23 Thread GitBox
nathadfield opened a new pull request #3791: Adding King.com to the list of companies. URL: https://github.com/apache/incubator-airflow/pull/3791 Adding King.com to the list of Airflow users. Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses t

[GitHub] msumit closed pull request #3789: Add Arquivei to companies list

2018-08-23 Thread GitBox
msumit closed pull request #3789: Add Arquivei to companies list URL: https://github.com/apache/incubator-airflow/pull/3789 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[jira] [Updated] (AIRFLOW-2946) Connection times out on airflow worker

2018-08-23 Thread Avik Aggarwal (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2946?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Avik Aggarwal updated AIRFLOW-2946: --- Description: Hi  I have Airflow cluster setup running Celery executors with Postgresql ins

[jira] [Created] (AIRFLOW-2946) Connection times out on airflow worker

2018-08-23 Thread Avik Aggarwal (JIRA)
Avik Aggarwal created AIRFLOW-2946: -- Summary: Connection times out on airflow worker Key: AIRFLOW-2946 URL: https://issues.apache.org/jira/browse/AIRFLOW-2946 Project: Apache Airflow Issue T

[GitHub] sakshi2894 opened a new pull request #3790: [AIRFLOW-2994] Fix command status check in Qubole Check operator

2018-08-23 Thread GitBox
sakshi2894 opened a new pull request #3790: [AIRFLOW-2994] Fix command status check in Qubole Check operator URL: https://github.com/apache/incubator-airflow/pull/3790 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow Jira](ht

[GitHub] leonardoam opened a new pull request #3789: Add Arquivei to companies list

2018-08-23 Thread GitBox
leonardoam opened a new pull request #3789: Add Arquivei to companies list URL: https://github.com/apache/incubator-airflow/pull/3789 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse

[jira] [Resolved] (AIRFLOW-2859) DateTimes returned from the database are not converted to UTC

2018-08-23 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor resolved AIRFLOW-2859. Resolution: Fixed > DateTimes returned from the database are not converted to UTC

[jira] [Closed] (AIRFLOW-1824) Set Multi dag dependency

2018-08-23 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-1824?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor closed AIRFLOW-1824. -- Resolution: Duplicate > Set Multi dag dependency > - > >

[jira] [Closed] (AIRFLOW-1748) Task is not being scheduled daily as expected in Airflow

2018-08-23 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor closed AIRFLOW-1748. -- Resolution: Cannot Reproduce Can't reproduce this - if you are still having issues a

[jira] [Closed] (AIRFLOW-1926) Mouseover on icons

2018-08-23 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-1926?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor closed AIRFLOW-1926. -- Resolution: Duplicate > Mouseover on icons > -- > > K

[jira] [Resolved] (AIRFLOW-2867) Airflow Python Code not compatible to coding guidelines and standards

2018-08-23 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2867?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor resolved AIRFLOW-2867. Resolution: Fixed > Airflow Python Code not compatible to coding guidelines and s

[jira] [Closed] (AIRFLOW-526) Pin all dependencies versions in setup.py

2018-08-23 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-526?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor closed AIRFLOW-526. - Resolution: Won't Fix Closing as WontFix based on discussion on github > Pin all depend

[jira] [Resolved] (AIRFLOW-1565) Incompatibility with Flask-Login-0.4.0

2018-08-23 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-1565?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor resolved AIRFLOW-1565. Resolution: Fixed Fix Version/s: 1.9.0 > Incompatibility with Flask-Login-

[GitHub] wrp commented on a change in pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection …

2018-08-23 Thread GitBox
wrp commented on a change in pull request #3785: [AIRFLOW-2938] Handle improperly formatted extra field in connection … URL: https://github.com/apache/incubator-airflow/pull/3785#discussion_r212284424 ## File path: airflow/www_rbac/views.py ## @@ -1923,7 +1923,10 @@ def pr

[GitHub] ashb closed pull request #3788: [AIRFLOW-XXX] Replaces incorrect env var name in INSTALL

2018-08-23 Thread GitBox
ashb closed pull request #3788: [AIRFLOW-XXX] Replaces incorrect env var name in INSTALL URL: https://github.com/apache/incubator-airflow/pull/3788 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] PointyShinyBurning opened a new pull request #3788: [AIRFLOW-XXX] Replaces incorrect env var name in INSTALL

2018-08-23 Thread GitBox
PointyShinyBurning opened a new pull request #3788: [AIRFLOW-XXX] Replaces incorrect env var name in INSTALL URL: https://github.com/apache/incubator-airflow/pull/3788 Make sure you have checked _all_ steps below. ### Jira - [x ] My PR addresses the following [Airflow Jira](h

[jira] [Commented] (AIRFLOW-987) `airflow kerberos` ignores --keytab and --principal arguments

2018-08-23 Thread jack (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590091#comment-16590091 ] jack commented on AIRFLOW-987: -- [~bolke] is it still an issue? > `airflow kerberos` ignore

  1   2   >