[jira] [Closed] (AIRFLOW-6067) [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class

2019-11-25 Thread JungraeKim (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6067?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] JungraeKim closed AIRFLOW-6067. --- Resolution: Incomplete > [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class >

[jira] [Commented] (AIRFLOW-6067) [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class

2019-11-25 Thread JungraeKim (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16982223#comment-16982223 ] JungraeKim commented on AIRFLOW-6067: - Closed by https://issues.apache.org/jira/browse/AIRFLOW-6068

[GitHub] [airflow] tyg03485 commented on issue #6664: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class

2019-11-25 Thread GitBox
tyg03485 commented on issue #6664: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class URL: https://github.com/apache/airflow/pull/6664#issuecomment-558498096 I'm not sure why there are so many commit in PR

[GitHub] [airflow] tyg03485 closed pull request #6664: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class

2019-11-25 Thread GitBox
tyg03485 closed pull request #6664: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class URL: https://github.com/apache/airflow/pull/6664 This is an automated message from the Apache

[jira] [Commented] (AIRFLOW-6067) [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16982209#comment-16982209 ] ASF GitHub Bot commented on AIRFLOW-6067: - tyg03485 commented on pull request #6664:

[jira] [Commented] (AIRFLOW-6067) [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16982208#comment-16982208 ] ASF GitHub Bot commented on AIRFLOW-6067: - tyg03485 commented on pull request #6664:

[GitHub] [airflow] tyg03485 opened a new pull request #6664: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class

2019-11-25 Thread GitBox
tyg03485 opened a new pull request #6664: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class URL: https://github.com/apache/airflow/pull/6664 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following

[jira] [Commented] (AIRFLOW-6067) [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16982203#comment-16982203 ] ASF GitHub Bot commented on AIRFLOW-6067: - tyg03485 commented on pull request #6663:

[jira] [Commented] (AIRFLOW-6067) [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16982201#comment-16982201 ] ASF GitHub Bot commented on AIRFLOW-6067: - tyg03485 commented on pull request #6663:

[GitHub] [airflow] tyg03485 closed pull request #6663: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class

2019-11-25 Thread GitBox
tyg03485 closed pull request #6663: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class URL: https://github.com/apache/airflow/pull/6663 This is an automated message from the Apache

[GitHub] [airflow] tyg03485 opened a new pull request #6663: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class

2019-11-25 Thread GitBox
tyg03485 opened a new pull request #6663: [AIRFLOW-6067] Add _set_pod_runtime_info_envs function for applying PodRuntimeInfoEnv class URL: https://github.com/apache/airflow/pull/6663 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following

[GitHub] [airflow] efedotova commented on a change in pull request #6585: More GSOD improvements

2019-11-25 Thread GitBox
efedotova commented on a change in pull request #6585: More GSOD improvements URL: https://github.com/apache/airflow/pull/6585#discussion_r350569553 ## File path: TESTING.rst ## @@ -0,0 +1,712 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more

[jira] [Created] (AIRFLOW-6067) [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class

2019-11-25 Thread JungraeKim (Jira)
JungraeKim created AIRFLOW-6067: --- Summary: [Airflow] pod_runtime_info_envs doesn't apply PodRuntimeInfoEnv class Key: AIRFLOW-6067 URL: https://issues.apache.org/jira/browse/AIRFLOW-6067 Project:

[GitHub] [airflow] msumit commented on a change in pull request #6654: [AIRFLOW-6055] Option for exponential backoff in Sensors

2019-11-25 Thread GitBox
msumit commented on a change in pull request #6654: [AIRFLOW-6055] Option for exponential backoff in Sensors URL: https://github.com/apache/airflow/pull/6654#discussion_r350562979 ## File path: tests/sensors/test_base_sensor.py ## @@ -502,3 +502,28 @@ def

[GitHub] [airflow] feng-tao commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue

2019-11-25 Thread GitBox
feng-tao commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue URL: https://github.com/apache/airflow/pull/6607#issuecomment-558481597 cc @mistercrunch Max, do you know if FAB changes any API/interface that Airflow is using?

[GitHub] [airflow] feng-tao commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue

2019-11-25 Thread GitBox
feng-tao commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue URL: https://github.com/apache/airflow/pull/6607#issuecomment-558481412 how about a custom DAG user role? This is an automated

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] *DRAFT* Neo4j operator and hook

2019-11-25 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] *DRAFT* Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r350558450 ## File path: airflow/contrib/operators/neo4j_operator.py ## @@ -0,0 +1,92 @@ +# -*- coding:

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350483599 ## File path: airflow/plugins_manager.py ## @@ -142,41

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350483186 ## File path: airflow/executors/executor_type_aliases.py ##

[GitHub] [airflow] codecov-io edited a comment on issue #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
codecov-io edited a comment on issue #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#issuecomment-557836097 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6601?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
codecov-io edited a comment on issue #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#issuecomment-557836097 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6601?src=pr=h1) Report > Merging

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350482741 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350483061 ## File path: airflow/executors/celery_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350482741 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350482286 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] mik-laj commented on issue #6563: [AIRFLOW-5915] Add support for the new documentation theme

2019-11-25 Thread GitBox
mik-laj commented on issue #6563: [AIRFLOW-5915] Add support for the new documentation theme URL: https://github.com/apache/airflow/pull/6563#issuecomment-558393019 Bootstrap is under MIT license. We are modifying its source code, so we need to have a header. If we imported correctly,

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350480464 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350479888 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350479680 ## File path: airflow/jobs/scheduler_job.py ## @@ -66,7

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350477875 ## File path: airflow/executors/local_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350477808 ## File path: airflow/executors/local_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350477769 ## File path: airflow/executors/local_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350477715 ## File path: airflow/executors/local_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350474598 ## File path: airflow/executors/local_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350473869 ## File path: airflow/executors/executor_types.py ## @@

[GitHub] [airflow] stale[bot] commented on issue #6288: AIRFLOW-5540 adding rise exception when the _resolve_connection method fails

2019-11-25 Thread GitBox
stale[bot] commented on issue #6288: AIRFLOW-5540 adding rise exception when the _resolve_connection method fails URL: https://github.com/apache/airflow/pull/6288#issuecomment-558384822 This issue has been automatically marked as stale because it has not had recent activity. It will be

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350472349 ## File path: airflow/executors/executor_types.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350472226 ## File path: airflow/executors/celery_executor.py ## @@

[GitHub] [airflow] potiuk opened a new pull request #6662: [AIRFLOW-6066] Added pre-commit checks for accidental debug stmts

2019-11-25 Thread GitBox
potiuk opened a new pull request #6662: [AIRFLOW-6066] Added pre-commit checks for accidental debug stmts URL: https://github.com/apache/airflow/pull/6662 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-6066) We need pre-commit check to check for pydevd

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6066?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16981973#comment-16981973 ] ASF GitHub Bot commented on AIRFLOW-6066: - potiuk commented on pull request #6662:

[jira] [Updated] (AIRFLOW-6066) We need pre-commit check to check for pydevd

2019-11-25 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6066?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk updated AIRFLOW-6066: -- Issue Type: Improvement (was: Bug) > We need pre-commit check to check for pydevd >

[jira] [Created] (AIRFLOW-6066) We need pre-commit check to check for pydevd

2019-11-25 Thread Jarek Potiuk (Jira)
Jarek Potiuk created AIRFLOW-6066: - Summary: We need pre-commit check to check for pydevd Key: AIRFLOW-6066 URL: https://issues.apache.org/jira/browse/AIRFLOW-6066 Project: Apache Airflow

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350467066 ## File path: airflow/executors/celery_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350466698 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350466451 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350466077 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350465745 ## File path: airflow/executors/base_executor.py ## @@

[GitHub] [airflow] potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r350465460 ## File path: airflow/executors/all_executors.py ## @@

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350464059 ## File path: tests/pytest.ini ## @@ -0,0 +1,37 @@ +# Licensed to the Apache Software Foundation

[GitHub] [airflow] ashb commented on issue #6563: [AIRFLOW-5915] Add support for the new documentation theme

2019-11-25 Thread GitBox
ashb commented on issue #6563: [AIRFLOW-5915] Add support for the new documentation theme URL: https://github.com/apache/airflow/pull/6563#issuecomment-558375029 What license is bootstrap released under? Is it right to put an ASF header on it?

[GitHub] [airflow] ashb commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
ashb commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350460727 ## File path: tests/__init__.py ## @@ -20,4 +20,4 @@ # flake8: noqa from .api import * # type: ignore

[GitHub] [airflow] ashb commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
ashb commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350460405 ## File path: setup.py ## @@ -285,20 +285,20 @@ def write_version(filename: str = os.path.join(*["airflow",

[GitHub] [airflow] ashb commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
ashb commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350461364 ## File path: tests/pytest.ini ## @@ -0,0 +1,37 @@ +# Licensed to the Apache Software Foundation (ASF) under

[jira] [Commented] (AIRFLOW-5959) Change import paths for "jira" modules

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5959?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16981955#comment-16981955 ] ASF GitHub Bot commented on AIRFLOW-5959: - ratb3rt commented on pull request #6661:

[GitHub] [airflow] ratb3rt opened a new pull request #6661: [AIRFLOW-5959] AIP-21 Change import paths for "jira" modules

2019-11-25 Thread GitBox
ratb3rt opened a new pull request #6661: [AIRFLOW-5959] AIP-21 Change import paths for "jira" modules URL: https://github.com/apache/airflow/pull/6661 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350453123 ## File path: tests/conftest.py ## @@ -0,0 +1,47 @@ +# Licensed to the Apache Software Foundation

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350450470 ## File path: airflow/www/views.py ## @@ -1029,7 +1029,6 @@ def blocked(self, session=None):

[jira] [Commented] (AIRFLOW-6065) Task Handler allowing us to save logs to Stackdriver Logging

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16981939#comment-16981939 ] ASF GitHub Bot commented on AIRFLOW-6065: - mik-laj commented on pull request #6660:

[GitHub] [airflow] mik-laj opened a new pull request #6660: [AIRFLOW-6065][depends on AIRFLOW-6047] Add StackdriverTaskHandler

2019-11-25 Thread GitBox
mik-laj opened a new pull request #6660: [AIRFLOW-6065][depends on AIRFLOW-6047] Add StackdriverTaskHandler URL: https://github.com/apache/airflow/pull/6660 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350446945 ## File path: tests/task/task_runner/test_standard_task_runner.py ## @@ -121,11 +121,12 @@ def

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350446945 ## File path: tests/task/task_runner/test_standard_task_runner.py ## @@ -121,11 +121,12 @@ def

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
nuclearpinguin commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350446526 ## File path: tests/pytest.ini ## @@ -0,0 +1,37 @@ +# Licensed to the Apache Software Foundation

[jira] [Assigned] (AIRFLOW-6065) Task Handler allowing us to save logs to Stackdriver Logging

2019-11-25 Thread Kamil Bregula (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6065?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula reassigned AIRFLOW-6065: -- Assignee: Kamil Bregula Issue Type: New Feature (was: Bug) Summary:

[jira] [Created] (AIRFLOW-6065) Add StackdriverLoggingHandler

2019-11-25 Thread Kamil Bregula (Jira)
Kamil Bregula created AIRFLOW-6065: -- Summary: Add StackdriverLoggingHandler Key: AIRFLOW-6065 URL: https://issues.apache.org/jira/browse/AIRFLOW-6065 Project: Apache Airflow Issue Type: Bug

[GitHub] [airflow] mik-laj commented on a change in pull request #6650: [AIRFLOW-6051] Make DAG optional during displaying the log

2019-11-25 Thread GitBox
mik-laj commented on a change in pull request #6650: [AIRFLOW-6051] Make DAG optional during displaying the log URL: https://github.com/apache/airflow/pull/6650#discussion_r350442484 ## File path: airflow/www/views.py ## @@ -578,7 +578,8 @@ def

[GitHub] [airflow] mik-laj commented on a change in pull request #6650: [AIRFLOW-6051] Make DAG optional during displaying the log

2019-11-25 Thread GitBox
mik-laj commented on a change in pull request #6650: [AIRFLOW-6051] Make DAG optional during displaying the log URL: https://github.com/apache/airflow/pull/6650#discussion_r350442484 ## File path: airflow/www/views.py ## @@ -578,7 +578,8 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350429351 ## File path: airflow/www/views.py ## @@ -1029,7 +1029,6 @@ def blocked(self, session=None):

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350441235 ## File path: tests/test_logging_config.py ## @@ -157,8 +157,13 @@ def setUp(self): def tearDown(self):

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350431945 ## File path: setup.py ## @@ -299,6 +300,9 @@ def write_version(filename: str = os.path.join(*["airflow",

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350437457 ## File path: tests/conftest.py ## @@ -0,0 +1,47 @@ +# Licensed to the Apache Software Foundation (ASF)

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350439617 ## File path: tests/pytest.ini ## @@ -0,0 +1,37 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350440504 ## File path: tests/task/task_runner/test_standard_task_runner.py ## @@ -121,11 +121,12 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350441421 ## File path: tests/www/test_views.py ## @@ -1129,7 +1129,8 @@ class TestDagACLView(TestBase): """

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350430355 ## File path: setup.py ## @@ -285,20 +285,20 @@ def write_version(filename: str = os.path.join(*["airflow",

[GitHub] [airflow] potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350429351 ## File path: airflow/www/views.py ## @@ -1029,7 +1029,6 @@ def blocked(self, session=None):

[GitHub] [airflow] mik-laj commented on issue #6644: [AIRFLOW-6047] Simplify the logging configuration template

2019-11-25 Thread GitBox
mik-laj commented on issue #6644: [AIRFLOW-6047] Simplify the logging configuration template URL: https://github.com/apache/airflow/pull/6644#issuecomment-558350745 @serkef Airflow runs a lot of processes. Each process initiates the logger from the beginning, so displaying the full

[GitHub] [airflow] potiuk commented on a change in pull request #6659: [AIRFLOW-6063] Remove astroid dependency

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6659: [AIRFLOW-6063] Remove astroid dependency URL: https://github.com/apache/airflow/pull/6659#discussion_r350431657 ## File path: setup.py ## @@ -277,7 +277,6 @@ def write_version(filename: str = os.path.join(*["airflow",

[GitHub] [airflow] mik-laj commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest

2019-11-25 Thread GitBox
mik-laj commented on a change in pull request #6472: [AIRFLOW-6058] Running tests with pytest URL: https://github.com/apache/airflow/pull/6472#discussion_r350427665 ## File path: tests/conftest.py ## @@ -0,0 +1,47 @@ +# Licensed to the Apache Software Foundation (ASF)

[jira] [Resolved] (AIRFLOW-5915) Add support for the new documentation theme

2019-11-25 Thread Kamil Bregula (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5915?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula resolved AIRFLOW-5915. Fix Version/s: 1.10.7 Resolution: Fixed > Add support for the new documentation

[jira] [Reopened] (AIRFLOW-5915) Add support for the new documentation theme

2019-11-25 Thread Kamil Bregula (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5915?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula reopened AIRFLOW-5915: > Add support for the new documentation theme > ---

[jira] [Closed] (AIRFLOW-5915) Add support for the new documentation theme

2019-11-25 Thread Kamil Bregula (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5915?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula closed AIRFLOW-5915. -- Assignee: Kamil Bregula Resolution: Fixed > Add support for the new documentation

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#discussion_r350425642 ## File path: airflow/models/baseoperator.py ## @@ -233,31 +232,28 @@ class derived from

[GitHub] [airflow] potiuk commented on issue #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
potiuk commented on issue #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#issuecomment-558342681 @ashb @kaxil -> all comments should be addressed. This is an

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#discussion_r350424929 ## File path: airflow/serialization/base_serialization.py ## @@ -22,17 +22,18 @@ import

[jira] [Commented] (AIRFLOW-5915) Add support for the new documentation theme

2019-11-25 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5915?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16981893#comment-16981893 ] ASF GitHub Bot commented on AIRFLOW-5915: - mik-laj commented on pull request #6563:

[GitHub] [airflow] mik-laj merged pull request #6563: [AIRFLOW-5915] Add support for the new documentation theme

2019-11-25 Thread GitBox
mik-laj merged pull request #6563: [AIRFLOW-5915] Add support for the new documentation theme URL: https://github.com/apache/airflow/pull/6563 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer

2019-11-25 Thread GitBox
kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer URL: https://github.com/apache/airflow/pull/6396#discussion_r350423129 ## File path: tests/operators/test_redshift_to_s3_operator.py ## @@ -31,7 +32,11 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer

2019-11-25 Thread GitBox
kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer URL: https://github.com/apache/airflow/pull/6396#discussion_r350422465 ## File path: tests/operators/test_redshift_to_s3_operator.py ## @@ -31,7 +32,11 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer

2019-11-25 Thread GitBox
kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer URL: https://github.com/apache/airflow/pull/6396#discussion_r350422465 ## File path: tests/operators/test_redshift_to_s3_operator.py ## @@ -31,7 +32,11 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer

2019-11-25 Thread GitBox
kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer URL: https://github.com/apache/airflow/pull/6396#discussion_r350422465 ## File path: tests/operators/test_redshift_to_s3_operator.py ## @@ -31,7 +32,11 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer

2019-11-25 Thread GitBox
kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer URL: https://github.com/apache/airflow/pull/6396#discussion_r350421888 ## File path: tests/operators/test_redshift_to_s3_operator.py ## @@ -31,7 +32,11 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer

2019-11-25 Thread GitBox
kaxil commented on a change in pull request #6396: [AIRFLOW-5726] Delete table as file name in RedshiftToS3Transfer URL: https://github.com/apache/airflow/pull/6396#discussion_r350421888 ## File path: tests/operators/test_redshift_to_s3_operator.py ## @@ -31,7 +32,11 @@

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#discussion_r350421126 ## File path: airflow/serialization/serialized_dag.py ## @@ -111,26 +95,26 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#discussion_r350421055 ## File path: airflow/serialization/serialized_dag.py ## @@ -111,26 +95,26 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6563: [AIRFLOW-5915] Add support for the new documentation theme

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6563: [AIRFLOW-5915] Add support for the new documentation theme URL: https://github.com/apache/airflow/pull/6563#discussion_r350416680 ## File path: airflow/www/static/css/flash.css ## @@ -1,33 +1,33 @@ /** -* Licensed to

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#discussion_r350415226 ## File path: airflow/models/baseoperator.py ## @@ -233,31 +232,28 @@ class derived from

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables

2019-11-25 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Remove cyclic imports and pylint disables URL: https://github.com/apache/airflow/pull/6601#discussion_r350414046 ## File path: docs/conf.py ## @@ -36,7 +36,7 @@ import sys from typing import Dict

[GitHub] [airflow] codecov-io edited a comment on issue #6659: [AIRFLOW-6063] Remove astroid dependency

2019-11-25 Thread GitBox
codecov-io edited a comment on issue #6659: [AIRFLOW-6063] Remove astroid dependency URL: https://github.com/apache/airflow/pull/6659#issuecomment-558332095 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6659?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io commented on issue #6659: [AIRFLOW-6063] Remove astroid dependency

2019-11-25 Thread GitBox
codecov-io commented on issue #6659: [AIRFLOW-6063] Remove astroid dependency URL: https://github.com/apache/airflow/pull/6659#issuecomment-558332095 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6659?src=pr=h1) Report > Merging

[airflow-site] branch aip-11 updated (b085bc8 -> 943783f)

2019-11-25 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula pushed a change to branch aip-11 in repository https://gitbox.apache.org/repos/asf/airflow-site.git. from b085bc8 Set overflows to auto and applied webkit styles to all scrollbars (#187) add 943783f Set

  1   2   3   4   >