[GitHub] [airflow] potiuk merged pull request #11818: Add links to PyPI packages.

2020-10-24 Thread GitBox
potiuk merged pull request #11818: URL: https://github.com/apache/airflow/pull/11818 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] yangrong688 commented on pull request #11757: Add uSmart Securities to INTHEWILD.md

2020-10-24 Thread GitBox
yangrong688 commented on pull request #11757: URL: https://github.com/apache/airflow/pull/11757#issuecomment-715891591 Thanks for your review @kaxil . This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] jbampton commented on pull request #11457: Fix spelling

2020-10-24 Thread GitBox
jbampton commented on pull request #11457: URL: https://github.com/apache/airflow/pull/11457#issuecomment-715890694 Hey @potiuk can you please review  This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] KulykDmytro opened a new issue #11819: LatestOnlyOperator cascades skipped status despite of trigger_rule set

2020-10-24 Thread GitBox
KulykDmytro opened a new issue #11819: URL: https://github.com/apache/airflow/issues/11819 **Apache Airflow version**: 1.10.12 **Kubernetes version**: 1.18.6 **What happened**: Forwarded from https://github.com/apache/airflow/issues/10686 In case when LatestOnlyOperator set as

[GitHub] [airflow] KulykDmytro edited a comment on issue #10686: Common tasks in downstream of multiple branches always set to skipped due to 'Not Previously Skipped' dependency

2020-10-24 Thread GitBox
KulykDmytro edited a comment on issue #10686: URL: https://github.com/apache/airflow/issues/10686#issuecomment-713721131 Initial issue which seems to be fixed with AIRFLOW-4453 has been returned back (at least in 1.10.12) PR: https://github.com/apache/airflow/pull/7464 so

[GitHub] [airflow] KulykDmytro edited a comment on issue #10686: Common tasks in downstream of multiple branches always set to skipped due to 'Not Previously Skipped' dependency

2020-10-24 Thread GitBox
KulykDmytro edited a comment on issue #10686: URL: https://github.com/apache/airflow/issues/10686#issuecomment-713855548 Another one example on 1.10.12 shows same behavior (join task skipping) for downstream tasks despite of any `trigger_rule` set (`all_done`, `none_failed`,

[GitHub] [airflow] KulykDmytro edited a comment on issue #11696: WebUI does not have access to pod logs

2020-10-24 Thread GitBox
KulykDmytro edited a comment on issue #11696: URL: https://github.com/apache/airflow/issues/11696#issuecomment-715886117 WebUI/Scheduler pods should run with serviceAccount which have RBAC permissions on k8s cluster to get logs ```yaml spec.template.spec.serviceAccount: airflow

[GitHub] [airflow] KulykDmytro edited a comment on issue #11696: WebUI does not have access to pod logs

2020-10-24 Thread GitBox
KulykDmytro edited a comment on issue #11696: URL: https://github.com/apache/airflow/issues/11696#issuecomment-715886117 WebUI should run with serviceAccount which have RBAC permissions on k8s cluster to get logs ```yaml spec.template.spec.serviceAccount: airflow ``` ```yaml

[GitHub] [airflow] KulykDmytro edited a comment on issue #11696: WebUI does not have access to pod logs

2020-10-24 Thread GitBox
KulykDmytro edited a comment on issue #11696: URL: https://github.com/apache/airflow/issues/11696#issuecomment-715886117 WebUI should run with serviceAccount which have RBAC permissions on k8s cluster to get logs ``` spec.template.spec.serviceAccount: airflow ``` ``` ---

[GitHub] [airflow] KulykDmytro edited a comment on issue #11696: WebUI does not have access to pod logs

2020-10-24 Thread GitBox
KulykDmytro edited a comment on issue #11696: URL: https://github.com/apache/airflow/issues/11696#issuecomment-715886117 WebUI should run with serviceAccount which have RBAC permissions on k8s cluster to get logs ``` spec.template.spec.serviceAccount: airflow ``` ``` ---

[GitHub] [airflow] KulykDmytro commented on issue #11696: WebUI does not have access to pod logs

2020-10-24 Thread GitBox
KulykDmytro commented on issue #11696: URL: https://github.com/apache/airflow/issues/11696#issuecomment-715886117 WebUI should run with serviceAccount which have RBAC permissions on k8s cluster to get logs ``` ``` ``` --- apiVersion: v1 kind: ServiceAccount metadata:

[GitHub] [airflow] github-actions[bot] commented on pull request #10917: WIP: Fix race conditions in task callback invocations

2020-10-24 Thread GitBox
github-actions[bot] commented on pull request #10917: URL: https://github.com/apache/airflow/pull/10917#issuecomment-715882668 [The Workflow run](https://github.com/apache/airflow/actions/runs/325681417) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] potiuk edited a comment on pull request #11797: Fix locking issue stopping tasks running with SequentialExecutor

2020-10-24 Thread GitBox
potiuk edited a comment on pull request #11797: URL: https://github.com/apache/airflow/pull/11797#issuecomment-715882665 Whoa. One line :) . Thanks @kaxil and @ashb! This is an automated message from the Apache Git Service.

[GitHub] [airflow] potiuk commented on pull request #11797: Fix locking issue stopping tasks running with SequentialExecutor

2020-10-24 Thread GitBox
potiuk commented on pull request #11797: URL: https://github.com/apache/airflow/pull/11797#issuecomment-715882665 Whoa. One line :) . Thanks @kaxil! This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] potiuk opened a new pull request #11818: Add links to PyPI packages.

2020-10-24 Thread GitBox
potiuk opened a new pull request #11818: URL: https://github.com/apache/airflow/pull/11818 Same as in #11801 but in provider's packages. --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] potiuk commented on pull request #11801: Add Project URLs for PyPI page

2020-10-24 Thread GitBox
potiuk commented on pull request #11801: URL: https://github.com/apache/airflow/pull/11801#issuecomment-715881997 Nice! Will add it to providers too! This is an automated message from the Apache Git Service. To respond to

[airflow] branch master updated (4c8e033 -> 3934ef2)

2020-10-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 4c8e033 Fix spelling and grammar (#11814) add 3934ef2 Remove redundant builtins imports (#11809) No new

[GitHub] [airflow] potiuk merged pull request #11809: Remove redundant builtins imports

2020-10-24 Thread GitBox
potiuk merged pull request #11809: URL: https://github.com/apache/airflow/pull/11809 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] potiuk commented on pull request #11809: Remove redundant builtins imports

2020-10-24 Thread GitBox
potiuk commented on pull request #11809: URL: https://github.com/apache/airflow/pull/11809#issuecomment-715881758 Flaky test which I fixed in #11817 merging This is an automated message from the Apache Git Service. To

[GitHub] [airflow] potiuk opened a new pull request #11817: Fixes flaky test test_should_response_200_with_reset_dag_run

2020-10-24 Thread GitBox
potiuk opened a new pull request #11817: URL: https://github.com/apache/airflow/pull/11817 Sometimes the rows were returned in reverse order. Fixes #11816 --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] potiuk opened a new issue #11816: [QUARANTINE] Flaky test TestPostClearTaskInstances.test_should_response_200_with_reset_dag_run

2020-10-24 Thread GitBox
potiuk opened a new issue #11816: URL: https://github.com/apache/airflow/issues/11816 Flaky test here: TestPostClearTaskInstances.test_should_response_200_with_reset_dag_run https://github.com/apache/airflow/pull/11809/checks?check_run_id=1301036003#step:6:1336 Looks

[GitHub] [airflow] potiuk commented on pull request #11785: Switch postgres from 10 to 13

2020-10-24 Thread GitBox
potiuk commented on pull request #11785: URL: https://github.com/apache/airflow/pull/11785#issuecomment-715880093 I modified it slighlty to also allow other versions in Breeze. This way we will see in breeze help that the in-between versions are supported as well. But CI tests wil

[GitHub] [airflow] potiuk edited a comment on pull request #11792: Local Executor is used by default for MySQL/Postgres breeze

2020-10-24 Thread GitBox
potiuk edited a comment on pull request #11792: URL: https://github.com/apache/airflow/pull/11792#issuecomment-715879776 I think it's still worth merging now and we should go ahead and fail sequential for MySQL/Postgres (see devlist) WDYT?

[GitHub] [airflow] potiuk commented on pull request #11792: Local Executor is used by default for MySQL/Postgres breeze

2020-10-24 Thread GitBox
potiuk commented on pull request #11792: URL: https://github.com/apache/airflow/pull/11792#issuecomment-715879776 I think it's still worth merging now and we should go ahead and fal sequential for MySQL/Postgres (see devlist)

[airflow] branch master updated (0218bcf -> 4c8e033)

2020-10-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 0218bcf Use LocalExecutor by default with tmux + Breeze (#11791) add 4c8e033 Fix spelling and grammar

[airflow] branch master updated (0218bcf -> 4c8e033)

2020-10-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 0218bcf Use LocalExecutor by default with tmux + Breeze (#11791) add 4c8e033 Fix spelling and grammar

[GitHub] [airflow] github-actions[bot] commented on pull request #11815: Render k8s yaml for tasks via the Airflow UI

2020-10-24 Thread GitBox
github-actions[bot] commented on pull request #11815: URL: https://github.com/apache/airflow/pull/11815#issuecomment-715878101 [The Workflow run](https://github.com/apache/airflow/actions/runs/325640704) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] potiuk merged pull request #11814: Fix spelling and grammar

2020-10-24 Thread GitBox
potiuk merged pull request #11814: URL: https://github.com/apache/airflow/pull/11814 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] dimberman commented on pull request #11815: Render k8s yaml for tasks via the Airflow UI

2020-10-24 Thread GitBox
dimberman commented on pull request #11815: URL: https://github.com/apache/airflow/pull/11815#issuecomment-715876436 cc: @ryanahamilton I still have a small bug on getting the button on the DAG view to work (also would love to merge this with those assets you built :) )

[GitHub] [airflow] github-actions[bot] commented on pull request #11465: Add Markdown linting to pre-commit

2020-10-24 Thread GitBox
github-actions[bot] commented on pull request #11465: URL: https://github.com/apache/airflow/pull/11465#issuecomment-715876374 [The Workflow run](https://github.com/apache/airflow/actions/runs/325627788) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] github-actions[bot] commented on pull request #11465: Add Markdown linting to pre-commit

2020-10-24 Thread GitBox
github-actions[bot] commented on pull request #11465: URL: https://github.com/apache/airflow/pull/11465#issuecomment-715876333 [The Workflow run](https://github.com/apache/airflow/actions/runs/325627671) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] dimberman opened a new pull request #11815: Render k8s yaml

2020-10-24 Thread GitBox
dimberman opened a new pull request #11815: URL: https://github.com/apache/airflow/pull/11815 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)** for

[GitHub] [airflow] potiuk commented on pull request #11797: Fix locking issue stopping tasks running with SequentialExecutor

2020-10-24 Thread GitBox
potiuk commented on pull request #11797: URL: https://github.com/apache/airflow/pull/11797#issuecomment-715824117 Cool! This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] duyet commented on pull request #11767: Fix: Removed password if HiveServer2Hook in LDAP or CUSTOM mode

2020-10-24 Thread GitBox
duyet commented on pull request #11767: URL: https://github.com/apache/airflow/pull/11767#issuecomment-715822851 @mik-laj @kaxil Could you please help me to review? This is an automated message from the Apache Git Service.

[airflow] branch master updated (7b89a04 -> 0218bcf)

2020-10-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 7b89a04 Replace io.open with builtin open (#11807) add 0218bcf Use LocalExecutor by default with tmux +

[airflow] branch master updated (7b89a04 -> 0218bcf)

2020-10-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 7b89a04 Replace io.open with builtin open (#11807) add 0218bcf Use LocalExecutor by default with tmux +

[GitHub] [airflow] github-actions[bot] commented on pull request #11627: Add D205 pydocstyle check

2020-10-24 Thread GitBox
github-actions[bot] commented on pull request #11627: URL: https://github.com/apache/airflow/pull/11627#issuecomment-715760981 [The Workflow run](https://github.com/apache/airflow/actions/runs/325528369) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] potiuk merged pull request #11791: Use LocalExecutor by default with tmux + Breeze

2020-10-24 Thread GitBox
potiuk merged pull request #11791: URL: https://github.com/apache/airflow/pull/11791 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

<    1   2