[GitHub] [airflow] potix2 commented on pull request #11531: Add type hints to aws provider

2020-10-16 Thread GitBox
potix2 commented on pull request #11531: URL: https://github.com/apache/airflow/pull/11531#issuecomment-710753161 I checked failed tests in local but it's successful. I try to trigger actions in CI. This is an automated

[GitHub] [airflow] mlgruby opened a new pull request #11609: strict type checking for provider Google

2020-10-16 Thread GitBox
mlgruby opened a new pull request #11609: URL: https://github.com/apache/airflow/pull/11609 Strict type checks for provider google cloud. Part of: #9708 @kaxil a quick one to review - This should complete strict type checking for provider google! --- **^ Add meaningful

[airflow] tag nightly-master updated (d305876 -> ee9bdff)

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

[airflow] branch master updated: Remove redundant code to serialized k8s.V1Pod (#11602)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 16ed974 Remove redundant code to

[airflow] branch master updated: Remove redundant code to serialized k8s.V1Pod (#11602)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 16ed974 Remove redundant code to

[GitHub] [airflow] kaxil merged pull request #11602: Remove redundant code to serialized k8s.V1Pod

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

[GitHub] [airflow] github-actions[bot] commented on pull request #10587: Add @dag decorator

2020-10-16 Thread GitBox
github-actions[bot] commented on pull request #10587: URL: https://github.com/apache/airflow/pull/10587#issuecomment-710716958 [The Workflow run](https://github.com/apache/airflow/actions/runs/311720324) is cancelling this PR. Building images for the PR has failed. Follow the the workflow

[GitHub] [airflow] ad-m commented on pull request #8550: Add DataflowStartFlexTemplateOperator

2020-10-16 Thread GitBox
ad-m commented on pull request #8550: URL: https://github.com/apache/airflow/pull/8550#issuecomment-710715682 Ja także dziękuje @TobKed za rozwój projektu Airflow! This is an automated message from the Apache Git Service. To

[GitHub] [airflow] kaxil opened a new pull request #11608: Replace old screenshots for managing Connections with new

2020-10-16 Thread GitBox
kaxil opened a new pull request #11608: URL: https://github.com/apache/airflow/pull/11608 Replace old screenshots for managing Connections with new screenshots --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] branch master updated: Minor doc improvements in blob_storage_to_gcs.rst (#11607)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new ee9bdff Minor doc improvements in

[GitHub] [airflow] kaxil merged pull request #11607: Minor doc improvements in blob_storage_to_gcs.rst

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

[GitHub] [airflow] kaxil merged pull request #11601: Fix Task definition in docs

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

[GitHub] [airflow] dimberman commented on pull request #11592: Fix #11591 lost redirection

2020-10-16 Thread GitBox
dimberman commented on pull request #11592: URL: https://github.com/apache/airflow/pull/11592#issuecomment-710711023 @mpermana I know that we know eachother but that doesn't mean I'm going to merge untested code :). Especially since this fixes a bug, can you please write a unit test

[airflow] branch master updated (b6ab0b5 -> 673386d)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from b6ab0b5 Fix rendering of code-block in operator/kubernetes.rst (#11606) add 673386d Fix Task definition

[airflow] branch master updated (210a948 -> b6ab0b5)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 210a948 Fix tcp keepalive parameters parsing (#11594) add b6ab0b5 Fix rendering of code-block in

[airflow] branch master updated: Fix tcp keepalive parameters parsing (#11594)

2020-10-16 Thread dimberman
This is an automated email from the ASF dual-hosted git repository. dimberman pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 210a948 Fix tcp keepalive parameters

[GitHub] [airflow] dimberman merged pull request #11594: Fix tcp keepalive parameters parsing

2020-10-16 Thread GitBox
dimberman merged pull request #11594: URL: https://github.com/apache/airflow/pull/11594 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

[GitHub] [airflow] kaxil merged pull request #11606: Fix rendering of code-block in operator/kubernetes.rst

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

[GitHub] [airflow] mik-laj commented on a change in pull request #11589: WIP: Perform "mini scheduling run" after task has finished

2020-10-16 Thread GitBox
mik-laj commented on a change in pull request #11589: URL: https://github.com/apache/airflow/pull/11589#discussion_r506771453 ## File path: airflow/models/dagrun.py ## @@ -43,6 +44,20 @@ from airflow.utils.types import DagRunType +class _TISchedulingDecision(TypedDict):

[GitHub] [airflow] mik-laj commented on a change in pull request #11589: WIP: Perform "mini scheduling run" after task has finished

2020-10-16 Thread GitBox
mik-laj commented on a change in pull request #11589: URL: https://github.com/apache/airflow/pull/11589#discussion_r506771250 ## File path: airflow/jobs/scheduler_job.py ## @@ -1664,42 +1664,11 @@ def _schedule_dag_run(self, dag_run: DagRun, currently_active_runs: int,

[airflow] branch master updated (b718b51 -> 55a4d8c)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from b718b51 Minor improvements to docs/tutorial.rst (#11604) add 55a4d8c Remove unnecessary use of

[GitHub] [airflow] kaxil merged pull request #11605: Remove unnecessary use of comprehension in setup_provider_packages.py

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

[airflow] branch master updated (bb615f2 -> b718b51)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from bb615f2 Cross Reference XCom in tutorial (#11600) add b718b51 Minor improvements to docs/tutorial.rst

[GitHub] [airflow] kaxil merged pull request #11604: Minor improvements to docs/tutorial.rst

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

[airflow] branch master updated (84dc2fb -> bb615f2)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 84dc2fb Set doc_md when using task decorator and function has __doc__ (#11598) add bb615f2 Cross

[GitHub] [airflow] kaxil merged pull request #11600: Cross Reference XCom in tutorial

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

[GitHub] [airflow] kaxil opened a new pull request #11607: Minor doc improvements in blob_storage_to_gcs.rst

2020-10-16 Thread GitBox
kaxil opened a new pull request #11607: URL: https://github.com/apache/airflow/pull/11607 - Sentence completion - code-block rendering ![image](https://user-images.githubusercontent.com/8811558/96323709-4487af80-1016-11eb-8ec3-477df39b75d9.png) - Link to English docs for Azure

[airflow] branch master updated (00dd758 -> 84dc2fb)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 00dd758 Raises a warning for provide_context instead of killing the task (#11597) add 84dc2fb Set doc_md

[airflow] branch master updated (00dd758 -> 84dc2fb)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 00dd758 Raises a warning for provide_context instead of killing the task (#11597) add 84dc2fb Set doc_md

[GitHub] [airflow] kaxil merged pull request #11598: Set doc_md when using task decorator and function has __doc__

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

[GitHub] [airflow] kaxil opened a new pull request #11606: Fix rendering of code-block in operator/kubernetes.rst

2020-10-16 Thread GitBox
kaxil opened a new pull request #11606: URL: https://github.com/apache/airflow/pull/11606 Docs were rendered incorrectly because of a missing blank line ![image](https://user-images.githubusercontent.com/8811558/96323468-25d4e900-1015-11eb-9e79-1a277672569d.png)

[GitHub] [airflow] kaxil opened a new pull request #11605: Remove unnecessary use of comprehension in setup_provider_packages.py

2020-10-16 Thread GitBox
kaxil opened a new pull request #11605: URL: https://github.com/apache/airflow/pull/11605 Since we are just iterating without modification, it is simpler to just use `list(PROVIDERS_REQUIREMENTS.keys())` --- **^ Add meaningful description above** Read the **[Pull

[GitHub] [airflow] kaxil opened a new pull request #11604: Minor improvements to docs/tutorial.rst

2020-10-16 Thread GitBox
kaxil opened a new pull request #11604: URL: https://github.com/apache/airflow/pull/11604 Sentence completions by adding periods --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] boring-cyborg[bot] commented on issue #11603: KubernetesExecutor with volumes and volume_mounts specified does not launch tasks

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

[GitHub] [airflow] yunduz opened a new issue #11603: KubernetesExecutor with volumes and volume_mounts specified does not launch tasks

2020-10-16 Thread GitBox
yunduz opened a new issue #11603: URL: https://github.com/apache/airflow/issues/11603 **Apache Airflow version**: 1.10.12 **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): Openshift 3.11 (Kubernetes 1.11) **Environment**: - **Cloud

[GitHub] [airflow] kaxil opened a new pull request #11602: Remove redundant code to serialized k8s.V1Pod

2020-10-16 Thread GitBox
kaxil opened a new pull request #11602: URL: https://github.com/apache/airflow/pull/11602 https://github.com/apache/airflow/pull/11558/files#r506763359 --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil commented on a change in pull request #11558: Guard against kubernetes not being installed

2020-10-16 Thread GitBox
kaxil commented on a change in pull request #11558: URL: https://github.com/apache/airflow/pull/11558#discussion_r506764891 ## File path: airflow/serialization/serialized_objects.py ## @@ -187,12 +190,12 @@ def _serialize(cls, var: Any) -> Any: # Unfortunately there is no

[GitHub] [airflow] kaxil opened a new pull request #11601: Fix Task definition in docs

2020-10-16 Thread GitBox
kaxil opened a new pull request #11601: URL: https://github.com/apache/airflow/pull/11601 I don't know why this was called constructor, am I missing something here?? --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] mjpieters commented on a change in pull request #11558: Guard against kubernetes not being installed

2020-10-16 Thread GitBox
mjpieters commented on a change in pull request #11558: URL: https://github.com/apache/airflow/pull/11558#discussion_r506763359 ## File path: airflow/serialization/serialized_objects.py ## @@ -187,12 +190,12 @@ def _serialize(cls, var: Any) -> Any: # Unfortunately there is

[GitHub] [airflow] github-actions[bot] commented on pull request #10587: Add @dag decorator

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

[GitHub] [airflow] kaxil opened a new pull request #11600: Cross Reference XCom in tutorial

2020-10-16 Thread GitBox
kaxil opened a new pull request #11600: URL: https://github.com/apache/airflow/pull/11600 Cross Reference XCom in tutorial --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] casassg commented on pull request #10587: Add @dag decorator

2020-10-16 Thread GitBox
casassg commented on pull request #10587: URL: https://github.com/apache/airflow/pull/10587#issuecomment-710688343 Rolledback a change as I didn't understand well how DagRun could be changed it's configuration from trigger interfaces. I proposed #11599 to fill the UI gap I see here.

[GitHub] [airflow] casassg opened a new issue #11599: Default DagRun configuration

2020-10-16 Thread GitBox
casassg opened a new issue #11599: URL: https://github.com/apache/airflow/issues/11599 **Description** It would nice to have the trigger UI pre-populated with default values for your dagrun configuration:

[GitHub] [airflow] github-actions[bot] commented on pull request #11598: Set doc_md when using task decorator and function has __doc__

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

[GitHub] [airflow] dimberman merged pull request #11597: Raises a warning for provide_context instead of killing the task

2020-10-16 Thread GitBox
dimberman merged pull request #11597: URL: https://github.com/apache/airflow/pull/11597 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

[airflow] branch master updated: Raises a warning for provide_context instead of killing the task (#11597)

2020-10-16 Thread dimberman
This is an automated email from the ASF dual-hosted git repository. dimberman pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 00dd758 Raises a warning for

[airflow] branch master updated (be72817 -> 4582c80)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from be72817 Allow loading plugins on Airflow start-up (#11596) add 4582c80 UX Enhancement: Add button to

[airflow] branch master updated (be72817 -> 4582c80)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from be72817 Allow loading plugins on Airflow start-up (#11596) add 4582c80 UX Enhancement: Add button to

[airflow] branch master updated (0c5bbe8 -> be72817)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 0c5bbe8 Replace methods on state with frozenset properties (#11576) add be72817 Allow loading plugins on

[airflow] branch master updated (be72817 -> 4582c80)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from be72817 Allow loading plugins on Airflow start-up (#11596) add 4582c80 UX Enhancement: Add button to

[airflow] branch master updated (0c5bbe8 -> be72817)

2020-10-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 0c5bbe8 Replace methods on state with frozenset properties (#11576) add be72817 Allow loading plugins on

[GitHub] [airflow] kaxil commented on a change in pull request #11396: Add creating_job_id to DagRun table

2020-10-16 Thread GitBox
kaxil commented on a change in pull request #11396: URL: https://github.com/apache/airflow/pull/11396#discussion_r506739203 ## File path: airflow/migrations/versions/364159666cbd_add_job_id_to_dagrun_table.py ## @@ -0,0 +1,44 @@ +# +# Licensed to the Apache Software

[GitHub] [airflow] kaxil merged pull request #11583: UX Enhancement: Add button to clear search query from DAG search

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

[GitHub] [airflow] kaxil merged pull request #11596: Allow loading plugins on Airflow start-up

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

[GitHub] [airflow] github-actions[bot] commented on pull request #10587: Add @dag decorator

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

[GitHub] [airflow] casassg opened a new pull request #11598: Set doc_md when using task decorator and function has __doc__

2020-10-16 Thread GitBox
casassg opened a new pull request #11598: URL: https://github.com/apache/airflow/pull/11598 This is a nice to have addition to `@task` decorator. The idea is that we can use function documentation to populate documentation of operator. This simplifies development and encourages users to

[GitHub] [airflow] github-actions[bot] commented on pull request #11526: Adds automated installation of dependent packages

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

[GitHub] [airflow] kaxil commented on pull request #11597: Raises a warning for provide_context instead of killing the task

2020-10-16 Thread GitBox
kaxil commented on pull request #11597: URL: https://github.com/apache/airflow/pull/11597#issuecomment-710645488 Just an FYI for future reviewers, we already have the following line in `Updating.md`: > The change is backwards compatible, setting provide_context will add the

[GitHub] [airflow] mik-laj commented on pull request #11597: Raises a warning for provide_context instead of killing the task

2020-10-16 Thread GitBox
mik-laj commented on pull request #11597: URL: https://github.com/apache/airflow/pull/11597#issuecomment-710644860 I don't know if using kwargs is the best idea in this case. I have to check if the IDE supports it. This is

[GitHub] [airflow] github-actions[bot] commented on pull request #11597: Raises a warning for provide_context instead of killing the task

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

[GitHub] [airflow] ryanahamilton commented on a change in pull request #11583: UX Enhancement: Add button to clear search query from DAG search

2020-10-16 Thread GitBox
ryanahamilton commented on a change in pull request #11583: URL: https://github.com/apache/airflow/pull/11583#discussion_r506714600 ## File path: airflow/www/templates/airflow/dags.html ## @@ -313,6 +316,12 @@ DAGs } }); +$('#search_form').on('reset',

[GitHub] [airflow] ashb commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
ashb commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710607306 > > allow creating more tables when running airflow db upgrade via plugins for which it is needed that the plugin is loaded when Scheduler starts. > > I still don't

[GitHub] [airflow] potiuk commented on pull request #8707: FIX Entrypoint and _CMD config variables (#8705)

2020-10-16 Thread GitBox
potiuk commented on pull request #8707: URL: https://github.com/apache/airflow/pull/8707#issuecomment-710598955 @NBardelot - are you going to complete that one? This is an automated message from the Apache Git Service. To

[GitHub] [airflow] nullhack commented on pull request #9464: Fix DockerOperator xcom

2020-10-16 Thread GitBox
nullhack commented on pull request #9464: URL: https://github.com/apache/airflow/pull/9464#issuecomment-710595586 Ok, I'll change the code then to include deprecation warning and the new flag with default value as binary

[GitHub] [airflow] ryanahamilton commented on pull request #11583: UX Enhancement: Add button to clear search query from DAG search

2020-10-16 Thread GitBox
ryanahamilton commented on pull request #11583: URL: https://github.com/apache/airflow/pull/11583#issuecomment-710591573 I've got a follow-up to this PR in the works that will apply this same pattern to the tag filter as well:

[GitHub] [airflow] potiuk commented on issue #11543: Scheduler Deadlock in tests for MySQL 5.7

2020-10-16 Thread GitBox
potiuk commented on issue #11543: URL: https://github.com/apache/airflow/issues/11543#issuecomment-710587828 I'd love to get to the bottom of the problem. Retrying is really a workaround for the real problem in those cases, I believe.

[GitHub] [airflow] mpermana commented on pull request #11592: Fix #11591 lost redirection

2020-10-16 Thread GitBox
mpermana commented on pull request #11592: URL: https://github.com/apache/airflow/pull/11592#issuecomment-710583352 @dimberman Added elaborate description , no we don't need test , you know me, thank you This is an

[GitHub] [airflow] potiuk commented on pull request #11587: #10472: Add D200 pydocstyle check

2020-10-16 Thread GitBox
potiuk commented on pull request #11587: URL: https://github.com/apache/airflow/pull/11587#issuecomment-710567148 Can you please rebase @Samira-g-js ? there are some conflicts :( This is an automated message from the Apache

[jira] [Commented] (AIRFLOW-6829) Auto apply apply_default decorator to all sub-classess

2020-10-16 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6829?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17215639#comment-17215639 ] ASF GitHub Bot commented on AIRFLOW-6829: - github-actions[bot] commented on pull request #7450:

[GitHub] [airflow] github-actions[bot] commented on pull request #7450: [AIRFLOW-6829][WIP] Auto-apply apply_default

2020-10-16 Thread GitBox
github-actions[bot] commented on pull request #7450: URL: https://github.com/apache/airflow/pull/7450#issuecomment-710559724 [The Workflow run](https://github.com/apache/airflow/actions/runs/311248383) is cancelling this PR. Building images for the PR has failed. Follow the the workflow

[airflow] branch master updated (44031bf -> 0c5bbe8)

2020-10-16 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 44031bf Fix broken migration for Sqlite3 (#11573) add 0c5bbe8 Replace methods on state with frozenset

[airflow] branch master updated (44031bf -> 0c5bbe8)

2020-10-16 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 44031bf Fix broken migration for Sqlite3 (#11573) add 0c5bbe8 Replace methods on state with frozenset

[GitHub] [airflow] mik-laj commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
mik-laj commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710545254 > allow creating more tables when running airflow db upgrade via plugins for which it is needed that the plugin is loaded when Scheduler starts. I still don't

[GitHub] [airflow] ashb merged pull request #11576: Replace collection methods on State with frozenset properties

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

[GitHub] [airflow] kaxil commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710538257 > > to any DAG that needs to have access to plugin-provided operators, sensors or hooks, I suppose. > > We plan to end support for operators and Hooks plugins in the near

[GitHub] [airflow] mik-laj commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
mik-laj commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710535998 > to any DAG that needs to have access to plugin-provided operators, sensors or hooks, I suppose. We plan to end support for operators and Hooks plugins in the near

[GitHub] [airflow] kaxil edited a comment on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil edited a comment on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710532498 > > allow creating more tables when running airflow db upgrade via plugins for which it is needed that the plugin is loaded when Scheduler starts. > > Can we not

[GitHub] [airflow] kaxil edited a comment on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil edited a comment on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710532498 > > allow creating more tables when running airflow db upgrade via plugins for which it is needed that the plugin is loaded when Scheduler starts. > > Can we not

[GitHub] [airflow] potiuk closed pull request #11420: Using new version of setup-kind using new add-path mechanism

2020-10-16 Thread GitBox
potiuk closed pull request #11420: URL: https://github.com/apache/airflow/pull/11420 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] github-actions[bot] commented on pull request #10587: Add @dag decorator

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

[GitHub] [airflow] kaxil commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710532498 > > allow creating more tables when running airflow db upgrade via plugins for which it is needed that the plugin is loaded when Scheduler starts. > > Can we not load

[GitHub] [airflow] mik-laj commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
mik-laj commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710530941 > allow creating more tables when running airflow db upgrade via plugins for which it is needed that the plugin is loaded when Scheduler starts. Can we not load plugins

[GitHub] [airflow] kaxil edited a comment on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil edited a comment on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710489207 > What are the use cases for this option? Why would the user want to set these options to true? For a good number of examples listed here

[GitHub] [airflow] github-actions[bot] commented on pull request #10587: Add @dag decorator

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

[GitHub] [airflow] kaxil commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710489207 > What are the use cases for this option? Why would the user want to set these options to true? For a good number of examples listed here, it is ideal to load plugins at

[airflow] branch master updated (4d611f2 -> 44031bf)

2020-10-16 Thread msumit
This is an automated email from the ASF dual-hosted git repository. msumit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 4d611f2 Clean up _trigger_dag function (#11584) add 44031bf Fix broken migration for Sqlite3 (#11573) No

[GitHub] [airflow] msumit merged pull request #11573: Fix migration for Sqlite3

2020-10-16 Thread GitBox
msumit merged pull request #11573: URL: https://github.com/apache/airflow/pull/11573 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] kaxil commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710481212 > Another workaround is to just import these items directly, not from the `airflow.(hooks|operators|sensors).` namespace. For hooks, operators and sensors I would

[GitHub] [airflow] mjpieters commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
mjpieters commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710478877 Another workaround is to just import these items directly, not from the `airflow.(hooks|operators|sensors).` namespace.

[GitHub] [airflow] kaxil commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710476213 > TBH, the lack of plugin loading in the scheduler _annoying_. I can also add > > ```python > from airflow import plugins_manager >

[GitHub] [airflow] kaxil commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
kaxil commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710476478 > Can you add some docs to `docs/plugins.rst`? https://airflow.readthedocs.io/en/latest/plugins.html Added docs

[GitHub] [airflow] mjpieters commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
mjpieters commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710474026 TBH, the lack of plugin loading in the scheduler _annoying_. I can also add ```python from airflow import plugins_manager plugins_manager.ensure_plugins_loaded()

[airflow] branch master updated (84c7028 -> 4d611f2)

2020-10-16 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 84c7028 Fix tooltip typo (#11593) add 4d611f2 Clean up _trigger_dag function (#11584) No new revisions were

[GitHub] [airflow] ashb commented on a change in pull request #11566: Update bash_operator's doc_string

2020-10-16 Thread GitBox
ashb commented on a change in pull request #11566: URL: https://github.com/apache/airflow/pull/11566#discussion_r506678353 ## File path: airflow/operators/bash.py ## @@ -62,6 +62,13 @@ class BashOperator(BaseOperator): bash_command = "set -e; python3 script.py '{{

[airflow] branch master updated (84c7028 -> 4d611f2)

2020-10-16 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 84c7028 Fix tooltip typo (#11593) add 4d611f2 Clean up _trigger_dag function (#11584) No new revisions were

[GitHub] [airflow] potix2 commented on a change in pull request #11434: Type annotation for aws operators in transfers

2020-10-16 Thread GitBox
potix2 commented on a change in pull request #11434: URL: https://github.com/apache/airflow/pull/11434#discussion_r506678114 ## File path: airflow/providers/google/cloud/operators/bigquery.py ## @@ -947,9 +947,7 @@ def execute(self, context) -> None:

[GitHub] [airflow] ashb merged pull request #11584: Clean up _trigger_dag function

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

[GitHub] [airflow] kaxil commented on a change in pull request #11597: Raises a warning for provide_context instead of killing the task

2020-10-16 Thread GitBox
kaxil commented on a change in pull request #11597: URL: https://github.com/apache/airflow/pull/11597#discussion_r506677382 ## File path: airflow/operators/python.py ## @@ -85,6 +86,9 @@ def __init__( templates_exts: Optional[List[str]] = None, **kwargs

[GitHub] [airflow] mjpieters commented on pull request #11596: Allow loading plugins on Airflow start-up

2020-10-16 Thread GitBox
mjpieters commented on pull request #11596: URL: https://github.com/apache/airflow/pull/11596#issuecomment-710463154 Looks like I'll need this switch? We have dags that use plugin operators & sensors, and the scheduler fails to load our dags as the plugin is not loaded when

[GitHub] [airflow] casassg commented on a change in pull request #10587: Add @dag decorator

2020-10-16 Thread GitBox
casassg commented on a change in pull request #10587: URL: https://github.com/apache/airflow/pull/10587#discussion_r506673787 ## File path: docs/concepts.rst ## @@ -162,6 +162,31 @@ Example DAG with decorated style html_content=email_info['body'] ) +DAG

  1   2   3   4   >