[GitHub] [airflow] BasPH commented on a change in pull request #6549: [AIRFLOW-5880] Enforce unique task ids

BasPH commented on a change in pull request #6549: [AIRFLOW-5880] Enforce unique task ids URL: https://github.com/apache/airflow/pull/6549#discussion_r345057908 ## File path: airflow/models/dag.py ## @@ -1190,13 +1189,8 @@ def add_task(self, task):

[GitHub] [airflow] potiuk edited a comment on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build

potiuk edited a comment on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build URL: https://github.com/apache/airflow/pull/6536#issuecomment-552773959 Side comment: I think eventually everyone should use pre-commit. It makes it so much easier to keep the

[GitHub] [airflow] potiuk commented on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build

potiuk commented on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build URL: https://github.com/apache/airflow/pull/6536#issuecomment-552773959 Side comment: I think eventually everyone should use pre-commit. It makes it so much easier to keep the source

[GitHub] [airflow] potiuk edited a comment on issue #6547: [AIRFLOW-5898] fix alembic crash due to typing import

potiuk edited a comment on issue #6547: [AIRFLOW-5898] fix alembic crash due to typing import URL: https://github.com/apache/airflow/pull/6547#issuecomment-552771864 I think "custom_typing" is not best name. Maybe "typing_compat" might reflect its purpose better?

[GitHub] [airflow] potiuk commented on issue #6547: [AIRFLOW-5898] fix alembic crash due to typing import

potiuk commented on issue #6547: [AIRFLOW-5898] fix alembic crash due to typing import URL: https://github.com/apache/airflow/pull/6547#issuecomment-552771864 I think "custom_typing" is not best name. Maybe "typing_compat" might reflect its purpose better.

[GitHub] [airflow] codecov-io edited a comment on issue #6512: [AIRFLOW-5824] Added AWS DataSync Hook and Operator

codecov-io edited a comment on issue #6512: [AIRFLOW-5824] Added AWS DataSync Hook and Operator URL: https://github.com/apache/airflow/pull/6512#issuecomment-551648543 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6512?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6512: [AIRFLOW-5824] Added AWS DataSync Hook and Operator

codecov-io edited a comment on issue #6512: [AIRFLOW-5824] Added AWS DataSync Hook and Operator URL: https://github.com/apache/airflow/pull/6512#issuecomment-551648543 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6512?src=pr=h1) Report > Merging

[GitHub] [airflow] potiuk commented on issue #6544: [AIRFLOW-5895] Move HDFS stuff from tests/core.py

potiuk commented on issue #6544: [AIRFLOW-5895] Move HDFS stuff from tests/core.py URL: https://github.com/apache/airflow/pull/6544#issuecomment-552768400 I am sure it's not every time. It never ever happened to me. I am super surprised because exactly the same check is run on Travis as

[jira] [Commented] (AIRFLOW-4593) Memory leak in Airflow scheduler

[ https://issues.apache.org/jira/browse/AIRFLOW-4593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16972109#comment-16972109 ] yuansicheng commented on AIRFLOW-4593: -- I find this problem in version 1.10.6 .  I run the

[GitHub] [airflow] zhongjiajie commented on issue #6487: [AIRFLOW-XXX] Alphabetical table and remove duplicate

zhongjiajie commented on issue #6487: [AIRFLOW-XXX] Alphabetical table and remove duplicate URL: https://github.com/apache/airflow/pull/6487#issuecomment-552748900 @mik-laj I rebase on master, PTAL, if you have time. I think the CI will pass cause this PR only doc change.

[GitHub] [airflow] baolsen commented on issue #6512: [AIRFLOW-5824] Added AWS DataSync Hook and Operator

baolsen commented on issue #6512: [AIRFLOW-5824] Added AWS DataSync Hook and Operator URL: https://github.com/apache/airflow/pull/6512#issuecomment-552745857 Hi @potiuk my changes are done. Ready for review / merge when you are.

[GitHub] [airflow] codecov-io edited a comment on issue #6531: [AIRFLOW-5882] Add ti_dep for not being in the RUNNING state

codecov-io edited a comment on issue #6531: [AIRFLOW-5882] Add ti_dep for not being in the RUNNING state URL: https://github.com/apache/airflow/pull/6531#issuecomment-552724876 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6531?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io commented on issue #6531: [AIRFLOW-5882] Add ti_dep for not being in the RUNNING state

codecov-io commented on issue #6531: [AIRFLOW-5882] Add ti_dep for not being in the RUNNING state URL: https://github.com/apache/airflow/pull/6531#issuecomment-552724876 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6531?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io commented on issue #6545: [AIRFLOW-5896] Move email stuff from tests/core.py

codecov-io commented on issue #6545: [AIRFLOW-5896] Move email stuff from tests/core.py URL: https://github.com/apache/airflow/pull/6545#issuecomment-552723504 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6545?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io commented on issue #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command

codecov-io commented on issue #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command URL: https://github.com/apache/airflow/pull/6543#issuecomment-552719983 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6543?src=pr=h1) Report > Merging

[GitHub] [airflow] mik-laj commented on a change in pull request #6487: [AIRFLOW-XXX] Alphabetical table and remove duplicate

mik-laj commented on a change in pull request #6487: [AIRFLOW-XXX] Alphabetical table and remove duplicate URL: https://github.com/apache/airflow/pull/6487#discussion_r345005099 ## File path: docs/operators-and-hooks-ref.rst ## @@ -671,6 +619,54 @@ These integrations

[GitHub] [airflow] mik-laj commented on issue #6487: [AIRFLOW-XXX] Alphabetical table and remove duplicate

mik-laj commented on issue #6487: [AIRFLOW-XXX] Alphabetical table and remove duplicate URL: https://github.com/apache/airflow/pull/6487#issuecomment-552719533 Sorry. I was busy lately because I was working on a new website. Can you do a rebase?

[GitHub] [airflow] mik-laj commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887]

mik-laj commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887] URL: https://github.com/apache/airflow/pull/6538#discussion_r345003999 ## File path: Dockerfile ## @@ -329,36 +328,45

[GitHub] [airflow] mik-laj commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887]

mik-laj commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887] URL: https://github.com/apache/airflow/pull/6538#discussion_r345003935 ## File path: Dockerfile ## @@ -329,36 +328,45

[jira] [Closed] (AIRFLOW-5903) Remove serve_logs command

[ https://issues.apache.org/jira/browse/AIRFLOW-5903?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula closed AIRFLOW-5903. -- Resolution: Fixed > Remove serve_logs command > - > >

[GitHub] [airflow] codecov-io commented on issue #6551: [AIRFLOW-5900] avoid unnecessary system calls in heartbeat_callback

codecov-io commented on issue #6551: [AIRFLOW-5900] avoid unnecessary system calls in heartbeat_callback URL: https://github.com/apache/airflow/pull/6551#issuecomment-552715725 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6551?src=pr=h1) Report > Merging

[GitHub] [airflow] houqp commented on issue #6342: [AIRFLOW-5662] fix incorrect naming for scheduler used slot metric

houqp commented on issue #6342: [AIRFLOW-5662] fix incorrect naming for scheduler used slot metric URL: https://github.com/apache/airflow/pull/6342#issuecomment-552706447 After thinking more about it, I now think it's better to approach this with a bigger refactoring. Let me know what you

[GitHub] [airflow] dstandish commented on issue #6426: [AIRFLOW-5751] add get_uri method to Connection

dstandish commented on issue #6426: [AIRFLOW-5751] add get_uri method to Connection URL: https://github.com/apache/airflow/pull/6426#issuecomment-552705561 @ashb @mik-laj friendly nag this can help people produce URI reliably

[GitHub] [airflow] codecov-io commented on issue #6550: [AIRFLOW-5897] Allow setting -1 as pool slots value in webserver

codecov-io commented on issue #6550: [AIRFLOW-5897] Allow setting -1 as pool slots value in webserver URL: https://github.com/apache/airflow/pull/6550#issuecomment-552704856 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6550?src=pr=h1) Report > Merging

[GitHub] [airflow] houqp commented on a change in pull request #6342: [AIRFLOW-5662] fix incorrect naming for scheduler used slot metric

houqp commented on a change in pull request #6342: [AIRFLOW-5662] fix incorrect naming for scheduler used slot metric URL: https://github.com/apache/airflow/pull/6342#discussion_r344991021 ## File path: airflow/jobs/scheduler_job.py ## @@ -982,10 +982,13 @@ def

[jira] [Created] (AIRFLOW-5903) Remove serve_logs command

Kamil Bregula created AIRFLOW-5903: -- Summary: Remove serve_logs command Key: AIRFLOW-5903 URL: https://issues.apache.org/jira/browse/AIRFLOW-5903 Project: Apache Airflow Issue Type:

[GitHub] [airflow] akki edited a comment on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator)

akki edited a comment on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator) URL: https://github.com/apache/airflow/pull/5489#issuecomment-552699552 @hredestig You can find the PR here - https://github.com/apache/airflow/pull/6552. Please see if it looks good

[GitHub] [airflow] akki commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator)

akki commented on issue #5489: [AIRFLOW-4843] Allow orchestration via Docker Swarm (SwarmOperator) URL: https://github.com/apache/airflow/pull/5489#issuecomment-552699552 @hredestig You can find the PR here - https://github.com/apache/airflow/pull/6552. Please consider helping with your

[GitHub] [airflow] akki commented on issue #6542: AIRFLOW-5854: Add support for `tty` parameter in Docker related operators

akki commented on issue #6542: AIRFLOW-5854: Add support for `tty` parameter in Docker related operators URL: https://github.com/apache/airflow/pull/6542#issuecomment-552698656 Thank you very much for the quick review! :) Cherry-picked as

[jira] [Commented] (AIRFLOW-5901) Group tests for the tasks and variables command

[ https://issues.apache.org/jira/browse/AIRFLOW-5901?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971977#comment-16971977 ] ASF GitHub Bot commented on AIRFLOW-5901: - mik-laj commented on pull request #6554:

[jira] [Commented] (AIRFLOW-5902) local_task_job is sleeping more often than it needs and spamming task log

[ https://issues.apache.org/jira/browse/AIRFLOW-5902?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971976#comment-16971976 ] ASF GitHub Bot commented on AIRFLOW-5902: - houqp commented on pull request #6553:

[GitHub] [airflow] houqp opened a new pull request #6553: [AIRFLOW-5902] avoid unnecessary sleep to maintain local task job heart rate

houqp opened a new pull request #6553: [AIRFLOW-5902] avoid unnecessary sleep to maintain local task job heart rate URL: https://github.com/apache/airflow/pull/6553 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[GitHub] [airflow] mik-laj opened a new pull request #6554: [AIRFLOW-5901][depends on AIRFLOW-5894] Group tests for the tasks and variables command

mik-laj opened a new pull request #6554: [AIRFLOW-5901][depends on AIRFLOW-5894] Group tests for the tasks and variables command URL: https://github.com/apache/airflow/pull/6554 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following

[jira] [Created] (AIRFLOW-5902) local_task_job is sleeping more often than it needs and spamming task log

QP Hou created AIRFLOW-5902: --- Summary: local_task_job is sleeping more often than it needs and spamming task log Key: AIRFLOW-5902 URL: https://issues.apache.org/jira/browse/AIRFLOW-5902 Project: Apache

[jira] [Commented] (AIRFLOW-5850) Make DockerSwarmOperator capture task logs

[ https://issues.apache.org/jira/browse/AIRFLOW-5850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971973#comment-16971973 ] ASF GitHub Bot commented on AIRFLOW-5850: - akki commented on pull request #6552: AIRFLOW-5850:

[GitHub] [airflow] akki opened a new pull request #6552: AIRFLOW-5850: Capture task logs in DockerSwarmOperator

akki opened a new pull request #6552: AIRFLOW-5850: Capture task logs in DockerSwarmOperator URL: https://github.com/apache/airflow/pull/6552 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Updated] (AIRFLOW-5901) Group tests for the tasks and variables command

[ https://issues.apache.org/jira/browse/AIRFLOW-5901?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula updated AIRFLOW-5901: --- Summary: Group tests for the tasks and variables command (was: Group tests for the tasks

[GitHub] [airflow] codecov-io edited a comment on issue #6527: [AIRFLOW-5876][part of AIRFLOW-5697] Fetch all Dataflow jobs during searching

codecov-io edited a comment on issue #6527: [AIRFLOW-5876][part of AIRFLOW-5697] Fetch all Dataflow jobs during searching URL: https://github.com/apache/airflow/pull/6527#issuecomment-551934161 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6527?src=pr=h1) Report > Merging

[jira] [Created] (AIRFLOW-5901) Group tests for the tasks command

Kamil Bregula created AIRFLOW-5901: -- Summary: Group tests for the tasks command Key: AIRFLOW-5901 URL: https://issues.apache.org/jira/browse/AIRFLOW-5901 Project: Apache Airflow Issue Type:

[GitHub] [airflow] KevinYang21 commented on issue #6550: [AIRFLOW-5897] Allow setting -1 as pool slots value in webserver

KevinYang21 commented on issue #6550: [AIRFLOW-5897] Allow setting -1 as pool slots value in webserver URL: https://github.com/apache/airflow/pull/6550#issuecomment-552692114 @pingzh @saguziel PTAL This is an automated

[jira] [Commented] (AIRFLOW-5900) heartbeat_callback for local_task_job can be more efficient

[ https://issues.apache.org/jira/browse/AIRFLOW-5900?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971947#comment-16971947 ] ASF GitHub Bot commented on AIRFLOW-5900: - houqp commented on pull request #6551:

[GitHub] [airflow] houqp opened a new pull request #6551: [AIRFLOW-5900] avoid unnecessary system calls in heartbeat_callback

houqp opened a new pull request #6551: [AIRFLOW-5900] avoid unnecessary system calls in heartbeat_callback URL: https://github.com/apache/airflow/pull/6551 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Created] (AIRFLOW-5900) heartbeat_callback for local_task_job can be more efficient

QP Hou created AIRFLOW-5900: --- Summary: heartbeat_callback for local_task_job can be more efficient Key: AIRFLOW-5900 URL: https://issues.apache.org/jira/browse/AIRFLOW-5900 Project: Apache Airflow

[GitHub] [airflow] mik-laj edited a comment on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build

mik-laj edited a comment on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build URL: https://github.com/apache/airflow/pull/6536#issuecomment-552681674 Is it not possible to generate this file at another time, e.g. when starting the breeze environment? I

[GitHub] [airflow] mik-laj commented on a change in pull request #6549: [AIRFLOW-5880] Enforce unique task ids

mik-laj commented on a change in pull request #6549: [AIRFLOW-5880] Enforce unique task ids URL: https://github.com/apache/airflow/pull/6549#discussion_r344973262 ## File path: airflow/models/dag.py ## @@ -1190,13 +1189,8 @@ def add_task(self, task):

[GitHub] [airflow] mik-laj commented on a change in pull request #6549: [AIRFLOW-5880] Enforce unique task ids

mik-laj commented on a change in pull request #6549: [AIRFLOW-5880] Enforce unique task ids URL: https://github.com/apache/airflow/pull/6549#discussion_r344973262 ## File path: airflow/models/dag.py ## @@ -1190,13 +1189,8 @@ def add_task(self, task):

[jira] [Commented] (AIRFLOW-5897) Allow setting -1 as pool slots value in webserver

[ https://issues.apache.org/jira/browse/AIRFLOW-5897?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971936#comment-16971936 ] ASF GitHub Bot commented on AIRFLOW-5897: - KevinYang21 commented on pull request #6550:

[GitHub] [airflow] KevinYang21 opened a new pull request #6550: [AIRFLOW-5897] Allow setting -1 as pool slots value in webserver

KevinYang21 opened a new pull request #6550: [AIRFLOW-5897] Allow setting -1 as pool slots value in webserver URL: https://github.com/apache/airflow/pull/6550 ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW-5897) issues and

[GitHub] [airflow] mik-laj commented on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build

mik-laj commented on issue #6536: [AIRFLOW-5885] List of tests is generated by pre-commit not Docker build URL: https://github.com/apache/airflow/pull/6536#issuecomment-552681674 Is it not possible to generate this file at another time, e.g. when starting the breeze environment? I am

[GitHub] [airflow] mik-laj commented on a change in pull request #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command

mik-laj commented on a change in pull request #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command URL: https://github.com/apache/airflow/pull/6543#discussion_r344969914 ## File path: tests/cli/test_cli.py ## @@ -272,13 +279,10 @@ def

[GitHub] [airflow] mik-laj commented on issue #6544: [AIRFLOW-5895] Move HDFS stuff from tests/core.py

mik-laj commented on issue #6544: [AIRFLOW-5895] Move HDFS stuff from tests/core.py URL: https://github.com/apache/airflow/pull/6544#issuecomment-552678321 @potiuk Every time I use pre-commit, it causes wrong file changes. I have to change them to make Travis happy. I do not know the

[jira] [Assigned] (AIRFLOW-5880) Enforce unique task ids

[ https://issues.apache.org/jira/browse/AIRFLOW-5880?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik reassigned AIRFLOW-5880: --- Assignee: Kaxil Naik > Enforce unique task ids > --- > >

[jira] [Commented] (AIRFLOW-5880) Enforce unique task ids

[ https://issues.apache.org/jira/browse/AIRFLOW-5880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971927#comment-16971927 ] ASF GitHub Bot commented on AIRFLOW-5880: - kaxil commented on pull request #6549:

[GitHub] [airflow] kaxil opened a new pull request #6549: [AIRFLOW-5880] Enforce unique task ids

kaxil opened a new pull request #6549: [AIRFLOW-5880] Enforce unique task ids URL: https://github.com/apache/airflow/pull/6549 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-5899) dmypy still reports error

[ https://issues.apache.org/jira/browse/AIRFLOW-5899?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971914#comment-16971914 ] ASF GitHub Bot commented on AIRFLOW-5899: - houqp commented on pull request #6548:

[GitHub] [airflow] houqp opened a new pull request #6548: [AIRFLOW-5899] fix dmypy errors

houqp opened a new pull request #6548: [AIRFLOW-5899] fix dmypy errors URL: https://github.com/apache/airflow/pull/6548 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW/)

[jira] [Created] (AIRFLOW-5899) dmypy still reports error

QP Hou created AIRFLOW-5899: --- Summary: dmypy still reports error Key: AIRFLOW-5899 URL: https://issues.apache.org/jira/browse/AIRFLOW-5899 Project: Apache Airflow Issue Type: Improvement

[jira] [Commented] (AIRFLOW-5898) alembic command crashes due to typing import path collision

[ https://issues.apache.org/jira/browse/AIRFLOW-5898?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971911#comment-16971911 ] ASF GitHub Bot commented on AIRFLOW-5898: - houqp commented on pull request #6547:

[GitHub] [airflow] houqp opened a new pull request #6547: [AIRFLOW-5898] fix alembic crash due to typing import

houqp opened a new pull request #6547: [AIRFLOW-5898] fix alembic crash due to typing import URL: https://github.com/apache/airflow/pull/6547 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Created] (AIRFLOW-5898) alembic command crashes due to typing import path collision

QP Hou created AIRFLOW-5898: --- Summary: alembic command crashes due to typing import path collision Key: AIRFLOW-5898 URL: https://issues.apache.org/jira/browse/AIRFLOW-5898 Project: Apache Airflow

[jira] [Created] (AIRFLOW-5897) Allow setting -1 as pool slots value in webserver

Kevin Yang created AIRFLOW-5897: --- Summary: Allow setting -1 as pool slots value in webserver Key: AIRFLOW-5897 URL: https://issues.apache.org/jira/browse/AIRFLOW-5897 Project: Apache Airflow

[GitHub] [airflow] mattbowden-ookla opened a new pull request #6546: [AIRFLOW-XXX] Add render_template changes to UPDATING.md

mattbowden-ookla opened a new pull request #6546: [AIRFLOW-XXX] Add render_template changes to UPDATING.md URL: https://github.com/apache/airflow/pull/6546 Make sure you have checked _all_ steps below. ### Jira - [X] My PR addresses the following [Airflow

[GitHub] [airflow] potiuk commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887]

potiuk commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887] URL: https://github.com/apache/airflow/pull/6538#discussion_r344932878 ## File path: scripts/ci/local_ci_extract_tests.sh

[GitHub] [airflow] potiuk commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887]

potiuk commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887] URL: https://github.com/apache/airflow/pull/6538#discussion_r344932878 ## File path: scripts/ci/local_ci_extract_tests.sh

[GitHub] [airflow] kaxil commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887]

kaxil commented on a change in pull request #6538: [AIRFLOW-5886] Selective copying of sources in Docker image. Depends on [AIRFLOW-5885] [AIRFLOW-5887] URL: https://github.com/apache/airflow/pull/6538#discussion_r344929538 ## File path: scripts/ci/local_ci_extract_tests.sh

[GitHub] [airflow] kaxil commented on issue #6541: [AIRFLOW-5892] BashOperator does not create temporary shell script

kaxil commented on issue #6541: [AIRFLOW-5892] BashOperator does not create temporary shell script URL: https://github.com/apache/airflow/pull/6541#issuecomment-552634677 It looks like a good improvement. Although, it would be better if we can add the problem that is solved by this PR in

[GitHub] [airflow] houqp commented on issue #6484: [AIRFLOW-5832] add pagerduty hook

houqp commented on issue #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#issuecomment-552633369 Thanks @kaxil and @pgagnon for the reviews! This is an automated message from the

[jira] [Resolved] (AIRFLOW-5832) pagerduty hook

[ https://issues.apache.org/jira/browse/AIRFLOW-5832?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik resolved AIRFLOW-5832. - Fix Version/s: 2.0.0 Resolution: Fixed > pagerduty hook > -- > >

[jira] [Commented] (AIRFLOW-5832) pagerduty hook

[ https://issues.apache.org/jira/browse/AIRFLOW-5832?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971862#comment-16971862 ] ASF subversion and git services commented on AIRFLOW-5832: -- Commit

[GitHub] [airflow] kaxil commented on issue #6484: [AIRFLOW-5832] add pagerduty hook

kaxil commented on issue #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#issuecomment-552631785 Thanks @houqp This is an automated message from the Apache Git Service. To respond to

[jira] [Commented] (AIRFLOW-5832) pagerduty hook

[ https://issues.apache.org/jira/browse/AIRFLOW-5832?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971860#comment-16971860 ] ASF GitHub Bot commented on AIRFLOW-5832: - kaxil commented on pull request #6484:

[GitHub] [airflow] kaxil commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook

kaxil commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#discussion_r344924689 ## File path: airflow/contrib/hooks/pagerduty_hook.py ## @@ -0,0 +1,128 @@ +# -*- coding: utf-8 -*- +# +# Licensed

[GitHub] [airflow] kaxil merged pull request #6484: [AIRFLOW-5832] add pagerduty hook

kaxil merged pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] dstandish commented on a change in pull request #6440: [AIRFLOW-5768] GCP cloud sql don't store ephemeral connection in db

dstandish commented on a change in pull request #6440: [AIRFLOW-5768] GCP cloud sql don't store ephemeral connection in db URL: https://github.com/apache/airflow/pull/6440#discussion_r344923202 ## File path: tests/gcp/hooks/test_cloud_sql.py ## @@ -1213,235 +1173,144 @@

[GitHub] [airflow] dstandish commented on issue #6440: [AIRFLOW-5768] GCP cloud sql don't store ephemeral connection in db

dstandish commented on issue #6440: [AIRFLOW-5768] GCP cloud sql don't store ephemeral connection in db URL: https://github.com/apache/airflow/pull/6440#issuecomment-552629484 thanks @potiuk i have rebased This is an

[GitHub] [airflow] houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook

houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#discussion_r344920107 ## File path: airflow/contrib/hooks/pagerduty_hook.py ## @@ -0,0 +1,128 @@ +# -*- coding: utf-8 -*- +# +# Licensed

[GitHub] [airflow] kaxil commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook

kaxil commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#discussion_r344902795 ## File path: airflow/contrib/hooks/pagerduty_hook.py ## @@ -0,0 +1,128 @@ +# -*- coding: utf-8 -*- +# +# Licensed

[GitHub] [airflow] potiuk commented on issue #6545: [AIRFLOW-5896] Move email stuff from tests/core.py

potiuk commented on issue #6545: [AIRFLOW-5896] Move email stuff from tests/core.py URL: https://github.com/apache/airflow/pull/6545#issuecomment-552597676 Travis is sad here as well :( This is an automated message from the

[GitHub] [airflow] potiuk commented on issue #6544: [AIRFLOW-5895] Move HDFS stuff from tests/core.py

potiuk commented on issue #6544: [AIRFLOW-5895] Move HDFS stuff from tests/core.py URL: https://github.com/apache/airflow/pull/6544#issuecomment-552597512 Travis is sad here as well. I recommend to run pre-commit hooks every time @mik-laj

[GitHub] [airflow] potiuk commented on issue #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command

potiuk commented on issue #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command URL: https://github.com/apache/airflow/pull/6543#issuecomment-552597215 Travis is sad! This is an automated message from

[GitHub] [airflow] houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook

houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#discussion_r344890371 ## File path: airflow/contrib/hooks/pagerduty_hook.py ## @@ -0,0 +1,129 @@ +# -*- coding: utf-8 -*- +# +# Licensed

[GitHub] [airflow] houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook

houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#discussion_r344890371 ## File path: airflow/contrib/hooks/pagerduty_hook.py ## @@ -0,0 +1,129 @@ +# -*- coding: utf-8 -*- +# +# Licensed

[GitHub] [airflow] houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook

houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#discussion_r344889535 ## File path: airflow/contrib/hooks/pagerduty_hook.py ## @@ -0,0 +1,129 @@ +# -*- coding: utf-8 -*- +# +# Licensed

[GitHub] [airflow] houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook

houqp commented on a change in pull request #6484: [AIRFLOW-5832] add pagerduty hook URL: https://github.com/apache/airflow/pull/6484#discussion_r344889060 ## File path: airflow/contrib/hooks/pagerduty_hook.py ## @@ -0,0 +1,129 @@ +# -*- coding: utf-8 -*- +# +# Licensed

[GitHub] [airflow] potiuk commented on a change in pull request #6440: [AIRFLOW-5768] GCP cloud sql don't store ephemeral connection in db

potiuk commented on a change in pull request #6440: [AIRFLOW-5768] GCP cloud sql don't store ephemeral connection in db URL: https://github.com/apache/airflow/pull/6440#discussion_r344885576 ## File path: tests/gcp/hooks/test_cloud_sql.py ## @@ -1213,235 +1173,144 @@ def

[GitHub] [airflow] codecov-io edited a comment on issue #6541: [AIRFLOW-5892] BashOperator does not create temporary shell script

codecov-io edited a comment on issue #6541: [AIRFLOW-5892] BashOperator does not create temporary shell script URL: https://github.com/apache/airflow/pull/6541#issuecomment-552592718 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6541?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io commented on issue #6541: [AIRFLOW-5892] BashOperator does not create temporary shell script

codecov-io commented on issue #6541: [AIRFLOW-5892] BashOperator does not create temporary shell script URL: https://github.com/apache/airflow/pull/6541#issuecomment-552592718 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6541?src=pr=h1) Report > Merging

[GitHub] [airflow] potiuk commented on issue #6447: [AIRFLOW-5792] Straighten out the migrations

potiuk commented on issue #6447: [AIRFLOW-5792] Straighten out the migrations URL: https://github.com/apache/airflow/pull/6447#issuecomment-552590973 Not sure what was the last agreement we had @fokko ? was I supposed to do this or did we agree you do it :) ? Just got some busy week last

[jira] [Commented] (AIRFLOW-5890) Many dag task status is skiped

[ https://issues.apache.org/jira/browse/AIRFLOW-5890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971822#comment-16971822 ] Kaxil Naik commented on AIRFLOW-5890: - Need more information ! Environment details, airflow

[jira] [Closed] (AIRFLOW-5891) Delete pervasively existed DAGs

[ https://issues.apache.org/jira/browse/AIRFLOW-5891?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik closed AIRFLOW-5891. --- Resolution: Implemented Fixed in https://issues.apache.org/jira/browse/AIRFLOW-5481 > Delete

[GitHub] [airflow] potiuk commented on issue #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command

potiuk commented on issue #6543: [AIRFLOW-5894][part of AIRFLOW-5893] Group tests for the Dags command URL: https://github.com/apache/airflow/pull/6543#issuecomment-552568462 Approved. Travis is sad however. This is an

[GitHub] [airflow] potiuk commented on issue #6542: AIRFLOW-5854: Add support for `tty` parameter in Docker related operators

potiuk commented on issue #6542: AIRFLOW-5854: Add support for `tty` parameter in Docker related operators URL: https://github.com/apache/airflow/pull/6542#issuecomment-552566397 Also cherry-picked it to 1.10.7 This is an

[jira] [Commented] (AIRFLOW-5854) Docker logs of Python processes are not visible

[ https://issues.apache.org/jira/browse/AIRFLOW-5854?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971790#comment-16971790 ] ASF subversion and git services commented on AIRFLOW-5854: -- Commit

[jira] [Commented] (AIRFLOW-5870) Optimise pool query and allow infinite pool for tasks

[ https://issues.apache.org/jira/browse/AIRFLOW-5870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971787#comment-16971787 ] ASF subversion and git services commented on AIRFLOW-5870: -- Commit

[jira] [Commented] (AIRFLOW-5888) Use psycopg2-binary package

[ https://issues.apache.org/jira/browse/AIRFLOW-5888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971788#comment-16971788 ] ASF subversion and git services commented on AIRFLOW-5888: -- Commit

[jira] [Commented] (AIRFLOW-5869) Creating DagRuns fails for Deserialized tasks with no start_date

[ https://issues.apache.org/jira/browse/AIRFLOW-5869?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971789#comment-16971789 ] ASF subversion and git services commented on AIRFLOW-5869: -- Commit

[jira] [Updated] (AIRFLOW-5854) Docker logs of Python processes are not visible

[ https://issues.apache.org/jira/browse/AIRFLOW-5854?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk updated AIRFLOW-5854: -- Fix Version/s: (was: 2.0.0) 1.10.7 > Docker logs of Python processes

[jira] [Commented] (AIRFLOW-5854) Docker logs of Python processes are not visible

[ https://issues.apache.org/jira/browse/AIRFLOW-5854?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971775#comment-16971775 ] ASF subversion and git services commented on AIRFLOW-5854: -- Commit

[GitHub] [airflow] potiuk merged pull request #6542: AIRFLOW-5854: Add support for `tty` parameter in Docker related operators

potiuk merged pull request #6542: AIRFLOW-5854: Add support for `tty` parameter in Docker related operators URL: https://github.com/apache/airflow/pull/6542 This is an automated message from the Apache Git Service. To

[jira] [Commented] (AIRFLOW-5854) Docker logs of Python processes are not visible

[ https://issues.apache.org/jira/browse/AIRFLOW-5854?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971774#comment-16971774 ] ASF GitHub Bot commented on AIRFLOW-5854: - potiuk commented on pull request #6542:

  1   2   >