[GitHub] [airflow] ashb opened a new pull request #14606: Use built-in `cached_property` on Python 3.8 where possible

2021-03-04 Thread GitBox
ashb opened a new pull request #14606: URL: https://github.com/apache/airflow/pull/14606 Functionality is the same, this just removes one dep for Py 3.8+ --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil opened a new pull request #14605: Fix docstrings for Kubernetes code

2021-03-04 Thread GitBox
kaxil opened a new pull request #14605: URL: https://github.com/apache/airflow/pull/14605 We don't use "@param" type od docstring. This PR fixes it --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] FredericoCoelhoNunes opened a new issue #14604: Airflow CLI - triggering a DAG for a particular execution_date clears and triggers several failed dag_runs

2021-03-04 Thread GitBox
FredericoCoelhoNunes opened a new issue #14604: URL: https://github.com/apache/airflow/issues/14604 **Apache Airflow version**: 2.0.1 **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): **Environment**: - **Cloud provider or

[GitHub] [airflow] boring-cyborg[bot] commented on issue #14604: Airflow CLI - triggering a DAG for a particular execution_date clears and triggers several failed dag_runs

2021-03-04 Thread GitBox
boring-cyborg[bot] commented on issue #14604: URL: https://github.com/apache/airflow/issues/14604#issuecomment-790756868 Thanks for opening your first issue here! Be sure to follow the issue template! This is an

[GitHub] [airflow] github-actions[bot] commented on pull request #14219: Provide login endpoint for the REST API with JWT authentication method

2021-03-04 Thread GitBox
github-actions[bot] commented on pull request #14219: URL: https://github.com/apache/airflow/pull/14219#issuecomment-790754604 [The Workflow run](https://github.com/apache/airflow/actions/runs/621661103) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] github-actions[bot] commented on pull request #14219: Provide login endpoint for the REST API with JWT authentication method

2021-03-04 Thread GitBox
github-actions[bot] commented on pull request #14219: URL: https://github.com/apache/airflow/pull/14219#issuecomment-790754452 [The Workflow run](https://github.com/apache/airflow/actions/runs/621652545) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] github-actions[bot] commented on pull request #14219: Provide login endpoint for the REST API with JWT authentication method

2021-03-04 Thread GitBox
github-actions[bot] commented on pull request #14219: URL: https://github.com/apache/airflow/pull/14219#issuecomment-790754193 [The Workflow run](https://github.com/apache/airflow/actions/runs/621651041) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #14588: Fill DagBag from dag_folder setting for upgrade rules

2021-03-04 Thread GitBox
ephraimbuddy commented on a change in pull request #14588: URL: https://github.com/apache/airflow/pull/14588#discussion_r587624946 ## File path: airflow/upgrade/rules/custom_operator_metaclass_rule.py ## @@ -17,6 +17,7 @@ from __future__ import absolute_import +from

[airflow] branch constraints-master updated: Updating constraints. Build id:621389598

2021-03-04 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-master by this push: new 38119d1 Updating

[GitHub] [airflow] Gaurang033 commented on issue #14489: Add a retry with wait interval for SSH operator

2021-03-04 Thread GitBox
Gaurang033 commented on issue #14489: URL: https://github.com/apache/airflow/issues/14489#issuecomment-790746887 @mik-laj perfect. we can similar implementation for this as well, then

[GitHub] [airflow] kaxil merged pull request #14600: Remove redundant step in CodeQL GitHub Actions step

2021-03-04 Thread GitBox
kaxil merged pull request #14600: URL: https://github.com/apache/airflow/pull/14600 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: Remove redundant step in CodeQL GitHub Actions step (#14600)

2021-03-04 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 4efd584 Remove redundant step in CodeQL

[GitHub] [airflow] XD-DENG commented on a change in pull request #14588: Fill DagBag from dag_folder setting for upgrade rules

2021-03-04 Thread GitBox
XD-DENG commented on a change in pull request #14588: URL: https://github.com/apache/airflow/pull/14588#discussion_r587616725 ## File path: airflow/upgrade/rules/custom_operator_metaclass_rule.py ## @@ -17,6 +17,7 @@ from __future__ import absolute_import +from airflow

[airflow] branch master updated (69a852f -> 35175c3)

2021-03-04 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 69a852f Add link to 2.0 Blog post in Changelog (#14602) add 35175c3 docs: Capitalise & minor fixes

[airflow] branch master updated (4c1e3c8 -> 69a852f)

2021-03-04 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 4c1e3c8 Restore correct terminal with to interactive breeze usage (#14579) add 69a852f Add link to 2.0

[GitHub] [airflow] kaxil commented on issue #14283: To capitalise or not to capitalise

2021-03-04 Thread GitBox
kaxil commented on issue #14283: URL: https://github.com/apache/airflow/issues/14283#issuecomment-790742678 Closed by https://github.com/apache/airflow/pull/14534 This is an automated message from the Apache Git Service. To

[GitHub] [airflow] kaxil closed issue #14283: To capitalise or not to capitalise

2021-03-04 Thread GitBox
kaxil closed issue #14283: URL: https://github.com/apache/airflow/issues/14283 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 the

[GitHub] [airflow] kaxil merged pull request #14534: docs: capitalise & fixes (#14283)

2021-03-04 Thread GitBox
kaxil merged pull request #14534: URL: https://github.com/apache/airflow/pull/14534 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 #14602: Add links to 2.0 Blog post in Changelog

2021-03-04 Thread GitBox
kaxil merged pull request #14602: URL: https://github.com/apache/airflow/pull/14602 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] eladkal commented on issue #9373: path to file as sql parameter in PostgresOperator not working anymore

2021-03-04 Thread GitBox
eladkal commented on issue #9373: URL: https://github.com/apache/airflow/issues/9373#issuecomment-790723080 1.10.14 Where is your dag file located? Could it be that the sql file path isn't relative to the dag file? (if so did you define `template_searchpath` ?)

[GitHub] [airflow] kaxil opened a new pull request #14603: Support jinja2 native Python types

2021-03-04 Thread GitBox
kaxil opened a new pull request #14603: URL: https://github.com/apache/airflow/pull/14603 This is second attempt to https://github.com/apache/airflow/pull/4900 Docs: https://jinja.palletsprojects.com/en/2.11.x/nativetypes/ ```python >>> from jinja2 import nativetypes >>>

[GitHub] [airflow] github-actions[bot] commented on pull request #14581: Ensure last_scheduler_run is updated in DB

2021-03-04 Thread GitBox
github-actions[bot] commented on pull request #14581: URL: https://github.com/apache/airflow/pull/14581#issuecomment-790719692 [The Workflow run](https://github.com/apache/airflow/actions/runs/621474186) is cancelling this PR. Building images for the PR has failed. Follow the the workflow

[GitHub] [airflow] marcosmarxm commented on issue #13838: Setting up Airflow for local development is hard

2021-03-04 Thread GitBox
marcosmarxm commented on issue #13838: URL: https://github.com/apache/airflow/issues/13838#issuecomment-790714532 @mik-laj maybe documentation how to create/maintain providers? - create a checklist and the common commands to test for specific providers.

[GitHub] [airflow] aviau commented on issue #14167: KubernetesPodOperator Pods no unique names

2021-03-04 Thread GitBox
aviau commented on issue #14167: URL: https://github.com/apache/airflow/issues/14167#issuecomment-790709875 Not sure, but maybe fixed by https://github.com/apache/airflow/commit/809b4f9b18c7040682e17879248d714f2664273d ?

[GitHub] [airflow] ashb commented on pull request #14581: Ensure last_scheduler_run is updated in DB

2021-03-04 Thread GitBox
ashb commented on pull request #14581: URL: https://github.com/apache/airflow/pull/14581#issuecomment-790703631 Related/possibly closes https://github.com/apache/airflow/pull/11462 This is an automated message from the

[GitHub] [airflow] FredericoCoelhoNunes commented on issue #14596: Max Active Run Feature is not Running

2021-03-04 Thread GitBox
FredericoCoelhoNunes commented on issue #14596: URL: https://github.com/apache/airflow/issues/14596#issuecomment-790703426 What executor are you using? If you're using SequentialExecutor, I think you won't be able to run more than one DAG at once

[GitHub] [airflow] FredericoCoelhoNunes edited a comment on issue #9975: max_active_runs = 1 can still create multiple active execution runs

2021-03-04 Thread GitBox
FredericoCoelhoNunes edited a comment on issue #9975: URL: https://github.com/apache/airflow/issues/9975#issuecomment-790692853 @zachliu Happy to hear you were able to create a stable API! Unfortunately I am not able to: when I use Python to select a failed DAG run, and clear all its task

[GitHub] [airflow] FredericoCoelhoNunes commented on issue #9975: max_active_runs = 1 can still create multiple active execution runs

2021-03-04 Thread GitBox
FredericoCoelhoNunes commented on issue #9975: URL: https://github.com/apache/airflow/issues/9975#issuecomment-790692853 @zachliu Happy to hear you were able to create a stable API! Unfortunately I am not able to: when I use Python to select a failed DAG run, and clear all its task

[GitHub] [airflow] kaxil commented on issue #14592: Unreachable Secrets Backend Causes Web Server Crash

2021-03-04 Thread GitBox
kaxil commented on issue #14592: URL: https://github.com/apache/airflow/issues/14592#issuecomment-790691181 @fhoda I know you talked about something similar, would you want to add your description too. This is an

[GitHub] [airflow] kaxil opened a new pull request #14602: Add links to 2.0 Blog post in Changelog

2021-03-04 Thread GitBox
kaxil opened a new pull request #14602: URL: https://github.com/apache/airflow/pull/14602 We currently don't have anything mentioned about 2.0.0 in the changelog, this commit should help users get to our blog post highlighting 2.0 features. --- **^ Add meaningful

[GitHub] [airflow] ashb commented on pull request #14476: Scheduler: Remove TIs from starved pools from the critical path.

2021-03-04 Thread GitBox
ashb commented on pull request #14476: URL: https://github.com/apache/airflow/pull/14476#issuecomment-790682935 @bperson Hmmm, not right now/anymore as the logs aren't availabe and GitHub are having problems so we can't retest. It _may_ just be a transient faliure, but given both

[GitHub] [airflow] ephraimbuddy commented on pull request #14219: Provide login endpoint for the REST API with JWT authentication method

2021-03-04 Thread GitBox
ephraimbuddy commented on pull request #14219: URL: https://github.com/apache/airflow/pull/14219#issuecomment-790682313 Can someone review the authentication once more? I feel it's OK now except for the user data schemas. The way I was writing the schemas before was to emulate how

[airflow] branch master updated (061cd23 -> 4c1e3c8)

2021-03-04 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 061cd23 Fixes failing test_views tests (#14599) add 4c1e3c8 Restore correct terminal with to interactive breeze

[GitHub] [airflow] ashb merged pull request #14579: Restore correct terminal with to interactive breeze usage

2021-03-04 Thread GitBox
ashb merged pull request #14579: URL: https://github.com/apache/airflow/pull/14579 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] ashb commented on pull request #14579: Restore correct terminal with to interactive breeze usage

2021-03-04 Thread GitBox
ashb commented on pull request #14579: URL: https://github.com/apache/airflow/pull/14579#issuecomment-790679115 Will merge without CI -- this doesn't affect them anyway. This is an automated message from the Apache Git

[GitHub] [airflow] ryanahamilton commented on pull request #14595: Fix functionality of "Demo Mode", refactor/simplify naming

2021-03-04 Thread GitBox
ryanahamilton commented on pull request #14595: URL: https://github.com/apache/airflow/pull/14595#issuecomment-790659998 I would like to propose that we close this PR in favor of #14601 that will remove the feature instead. What do you all think of that? Please comment on that PR.

[GitHub] [airflow] ryanahamilton opened a new pull request #14601: Remove broken and undocumented "demo mode" feature

2021-03-04 Thread GitBox
ryanahamilton opened a new pull request #14601: URL: https://github.com/apache/airflow/pull/14601 After fixing and refactoring the "demo mode" feature in #14595, I feel that it would be more productive to replace that PR with this one that removes the feature instead. - It has

[GitHub] [airflow] kaxil commented on issue #14598: Provide shared storage between task via pluggable storage providers (akin to S3 remote logging)

2021-03-04 Thread GitBox
kaxil commented on issue #14598: URL: https://github.com/apache/airflow/issues/14598#issuecomment-790644229 You can already do this via an XComBackend: https://airflow.apache.org/blog/airflow-1.10.12/#allow-defining-custom-xcom-class There is also an example in

[GitHub] [airflow] kaxil opened a new pull request #14600: Remove redundant step in CodeQL GitHub Actions step

2021-03-04 Thread GitBox
kaxil opened a new pull request #14600: URL: https://github.com/apache/airflow/pull/14600 From https://github.com/apache/airflow/runs/2031451066#step:4:9 Warning: 1 issue was detected with this workflow: git checkout HEAD^2 is no longer necessary. Please remove this step as Code

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #14501: Fix set task instance form test

2021-03-04 Thread GitBox
ephraimbuddy commented on a change in pull request #14501: URL: https://github.com/apache/airflow/pull/14501#discussion_r587283929 ## File path: tests/api_connexion/schemas/test_task_instance_schema.py ## @@ -199,7 +199,6 @@ def test_success(self): @parameterized.expand(

[GitHub] [airflow] Naresh-sin edited a comment on pull request #13963: Bugfix: Manual DagRun trigger should not skip scheduled runs

2021-03-04 Thread GitBox
Naresh-sin edited a comment on pull request #13963: URL: https://github.com/apache/airflow/pull/13963#issuecomment-790627008 Guys Need help.Sorry by bad did not understand what is the fix to do for Manual DagRun trigger skip actual schedule? what I get from here is remove the line

[GitHub] [airflow] Naresh-sin commented on pull request #13963: Bugfix: Manual DagRun trigger should not skip scheduled runs

2021-03-04 Thread GitBox
Naresh-sin commented on pull request #13963: URL: https://github.com/apache/airflow/pull/13963#issuecomment-790627008 Guys Need help.Sorry by bad did not understand what is the fix to do for Manual DagRun trigger skip actual schedule? what I get from here is remove the line

[airflow] branch master updated (f5a35f4 -> 061cd23)

2021-03-04 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 f5a35f4 Fix some small typos also TESTING.rst (#14594) add 061cd23 Fixes failing test_views tests

[GitHub] [airflow] kaxil merged pull request #14599: Fixes failing test_views tests

2021-03-04 Thread GitBox
kaxil merged pull request #14599: URL: https://github.com/apache/airflow/pull/14599 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 #14599: Fixes failing test_views tests

2021-03-04 Thread GitBox
kaxil commented on pull request #14599: URL: https://github.com/apache/airflow/pull/14599#issuecomment-790619968 Looks like "Resource not accessible by integration" errors (GitHub's transient errors) -- going to merge it so that we don't have broken master

[GitHub] [airflow] vemikhaylov commented on issue #14591: trigger specific task in an existing DAG run

2021-03-04 Thread GitBox
vemikhaylov commented on issue #14591: URL: https://github.com/apache/airflow/issues/14591#issuecomment-790606951 @ConstantinoSchillebeeckx Will this solve your problem? https://github.com/apache/airflow/pull/14500 https://github.com/apache/airflow/issues/13225

[GitHub] [airflow] vemikhaylov commented on issue #14598: Provide shared storage between task via pluggable storage providers (akin to S3 remote logging)

2021-03-04 Thread GitBox
vemikhaylov commented on issue #14598: URL: https://github.com/apache/airflow/issues/14598#issuecomment-790604779 > * When the task starts it downloads (if exists) the `artifacts/dag_id/run_id/task_id/artifacts.zip` for each upstream task Will this be the default behaviour? If so,

[GitHub] [airflow] HaloKo4 commented on pull request #14595: Fix functionality of "Demo Mode", refactor/simplify naming

2021-03-04 Thread GitBox
HaloKo4 commented on pull request #14595: URL: https://github.com/apache/airflow/pull/14595#issuecomment-790599526 This is really cool but if the only way to activate it is from airflow.cfg it isn't very convinent This is

[airflow] branch master updated: Fix some small typos also TESTING.rst (#14594)

2021-03-04 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 f5a35f4 Fix some small typos also

[GitHub] [airflow] kaxil merged pull request #14594: Fix some small typos also TESTING.rst

2021-03-04 Thread GitBox
kaxil merged pull request #14594: URL: https://github.com/apache/airflow/pull/14594 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] ryanahamilton commented on pull request #14595: Fix functionality of "Demo Mode", refactor/simplify naming

2021-03-04 Thread GitBox
ryanahamilton commented on pull request #14595: URL: https://github.com/apache/airflow/pull/14595#issuecomment-790586929 This implementation is definitely a "demonstration mode" feature and by no means a security feature. The [description in the

[GitHub] [airflow] ecerulm commented on issue #14598: Provide shared storage between task via pluggable storage providers (akin to S3 remote logging)

2021-03-04 Thread GitBox
ecerulm commented on issue #14598: URL: https://github.com/apache/airflow/issues/14598#issuecomment-790584181 My idea was to implement with with a pluggable storage so that S3, and other storage can be used. My initial idea on how this could work is the following: * When the task

[GitHub] [airflow] ashb commented on pull request #14595: Fix functionality of "Demo Mode", refactor/simplify naming

2021-03-04 Thread GitBox
ashb commented on pull request #14595: URL: https://github.com/apache/airflow/pull/14595#issuecomment-790581682 ![image](https://user-images.githubusercontent.com/34150/109963913-de5da980-7ce4-11eb-8975-de447bfe3938.png) https://imageupscaler.com/deblurring 

[GitHub] [airflow] ashb commented on pull request #14595: Fix functionality of "Demo Mode", refactor/simplify naming

2021-03-04 Thread GitBox
ashb commented on pull request #14595: URL: https://github.com/apache/airflow/pull/14595#issuecomment-790575714 I think i t was by design that on, for example the Tree view that only the task ids were blurred, but not the tree itself. Ditto on the graph view. This lets the

[GitHub] [airflow] ashb commented on pull request #14595: Fix functionality of "Demo Mode", refactor/simplify naming

2021-03-04 Thread GitBox
ashb commented on pull request #14595: URL: https://github.com/apache/airflow/pull/14595#issuecomment-790574870 I wonder if _all_ of the code view should be blurred? Do you know what it was trying to blur before? For instance the any code of python operators is "sensitive" and

[GitHub] [airflow] ashb commented on pull request #14595: Fix functionality of "Demo Mode", refactor/simplify naming

2021-03-04 Thread GitBox
ashb commented on pull request #14595: URL: https://github.com/apache/airflow/pull/14595#issuecomment-790573172 I think this was used for producing screenshots for marketing materials, or similar, without exposing internal details of what the DAG does.

[GitHub] [airflow] ashb commented on pull request #14579: Restore correct terminal with to interactive breeze usage

2021-03-04 Thread GitBox
ashb commented on pull request #14579: URL: https://github.com/apache/airflow/pull/14579#issuecomment-790556444 @kaxil Updated -- could you confirm this works for you on OSX please? This is an automated message from the

[GitHub] [airflow] kaxil opened a new pull request #14599: Fixes failing test_views tests.

2021-03-04 Thread GitBox
kaxil opened a new pull request #14599: URL: https://github.com/apache/airflow/pull/14599 This reverts commit 49952e79b04da932242ebf3981883e591b467994. Not sure what happended. We made that change because of cPython vulnerability

[GitHub] [airflow] Dr-Denzy commented on issue #14167: KubernetesPodOperator Pods no unique names

2021-03-04 Thread GitBox
Dr-Denzy commented on issue #14167: URL: https://github.com/apache/airflow/issues/14167#issuecomment-790533319 @kaxil please assign to me. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] andreaspeters commented on a change in pull request #10977: Reinclude/Add a new Mesos Executor

2021-03-04 Thread GitBox
andreaspeters commented on a change in pull request #10977: URL: https://github.com/apache/airflow/pull/10977#discussion_r587373690 ## File path: airflow/executors/mesos_executor.py ## @@ -0,0 +1,380 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] andreaspeters commented on a change in pull request #10977: Reinclude/Add a new Mesos Executor

2021-03-04 Thread GitBox
andreaspeters commented on a change in pull request #10977: URL: https://github.com/apache/airflow/pull/10977#discussion_r587373690 ## File path: airflow/executors/mesos_executor.py ## @@ -0,0 +1,380 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] vemikhaylov commented on a change in pull request #14501: Fix set task instance form test

2021-03-04 Thread GitBox
vemikhaylov commented on a change in pull request #14501: URL: https://github.com/apache/airflow/pull/14501#discussion_r587331823 ## File path: tests/api_connexion/schemas/test_task_instance_schema.py ## @@ -199,7 +199,6 @@ def test_success(self): @parameterized.expand(

[GitHub] [airflow] ecerulm opened a new issue #14598: Provide shared storage between task via pluggable storage providers (akin to S3 remote logging)

2021-03-04 Thread GitBox
ecerulm opened a new issue #14598: URL: https://github.com/apache/airflow/issues/14598 **Description** Provide shared storage between tasks. **Use case / motivation** Today we have XCom to share small pieces of information between task, but that is not suitable to

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #14501: Fix set task instance form test

2021-03-04 Thread GitBox
ephraimbuddy commented on a change in pull request #14501: URL: https://github.com/apache/airflow/pull/14501#discussion_r587283929 ## File path: tests/api_connexion/schemas/test_task_instance_schema.py ## @@ -199,7 +199,6 @@ def test_success(self): @parameterized.expand(

[GitHub] [airflow] vemikhaylov commented on a change in pull request #14501: Fix set task instance form test

2021-03-04 Thread GitBox
vemikhaylov commented on a change in pull request #14501: URL: https://github.com/apache/airflow/pull/14501#discussion_r587259860 ## File path: tests/api_connexion/schemas/test_task_instance_schema.py ## @@ -199,7 +199,6 @@ def test_success(self): @parameterized.expand(

[GitHub] [airflow] vemikhaylov commented on a change in pull request #14501: Fix set task instance form test

2021-03-04 Thread GitBox
vemikhaylov commented on a change in pull request #14501: URL: https://github.com/apache/airflow/pull/14501#discussion_r587259860 ## File path: tests/api_connexion/schemas/test_task_instance_schema.py ## @@ -199,7 +199,6 @@ def test_success(self): @parameterized.expand(

[GitHub] [airflow] turbaszek commented on a change in pull request #14492: Adds new community provider Airbyte

2021-03-04 Thread GitBox
turbaszek commented on a change in pull request #14492: URL: https://github.com/apache/airflow/pull/14492#discussion_r587259941 ## File path: airflow/providers/airbyte/example_dags/example_airbyte_trigger_job.py ## @@ -0,0 +1,67 @@ +# +# Licensed to the Apache Software

[GitHub] [airflow] vemikhaylov commented on a change in pull request #14501: Fix set task instance form test

2021-03-04 Thread GitBox
vemikhaylov commented on a change in pull request #14501: URL: https://github.com/apache/airflow/pull/14501#discussion_r587259860 ## File path: tests/api_connexion/schemas/test_task_instance_schema.py ## @@ -199,7 +199,6 @@ def test_success(self): @parameterized.expand(

[GitHub] [airflow] turbaszek commented on a change in pull request #14492: Adds new community provider Airbyte

2021-03-04 Thread GitBox
turbaszek commented on a change in pull request #14492: URL: https://github.com/apache/airflow/pull/14492#discussion_r587259628 ## File path: airflow/providers/airbyte/example_dags/example_airbyte_trigger_job.py ## @@ -0,0 +1,67 @@ +# +# Licensed to the Apache Software

[GitHub] [airflow] vemikhaylov commented on a change in pull request #14500: Clear tasks by task ids in REST API

2021-03-04 Thread GitBox
vemikhaylov commented on a change in pull request #14500: URL: https://github.com/apache/airflow/pull/14500#discussion_r587250624 ## File path: airflow/api_connexion/openapi/v1.yaml ## @@ -2218,6 +2218,13 @@ components: type: boolean default: true +

[GitHub] [airflow] vemikhaylov commented on a change in pull request #14500: Clear tasks by task ids in REST API

2021-03-04 Thread GitBox
vemikhaylov commented on a change in pull request #14500: URL: https://github.com/apache/airflow/pull/14500#discussion_r587250624 ## File path: airflow/api_connexion/openapi/v1.yaml ## @@ -2218,6 +2218,13 @@ components: type: boolean default: true +

[GitHub] [airflow] andreaspeters commented on a change in pull request #10977: Reinclude/Add a new Mesos Executor

2021-03-04 Thread GitBox
andreaspeters commented on a change in pull request #10977: URL: https://github.com/apache/airflow/pull/10977#discussion_r587247108 ## File path: tests/executors/test_mesos_executor.py ## @@ -0,0 +1,61 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] andreaspeters commented on a change in pull request #10977: Reinclude/Add a new Mesos Executor

2021-03-04 Thread GitBox
andreaspeters commented on a change in pull request #10977: URL: https://github.com/apache/airflow/pull/10977#discussion_r587244859 ## File path: airflow/executors/mesos_executor.py ## @@ -0,0 +1,380 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] Minyus commented on issue #9610: Pod logs from KubernetesPodOperator occasionally get replaced with "Task is not able to run"

2021-03-04 Thread GitBox
Minyus commented on issue #9610: URL: https://github.com/apache/airflow/issues/9610#issuecomment-790415553 Airflow 1.10.14 with local logging and Celery 4.4.7 reproduced the same error. This is an automated message from the

[GitHub] [airflow] andreaspeters commented on a change in pull request #10977: Reinclude/Add a new Mesos Executor

2021-03-04 Thread GitBox
andreaspeters commented on a change in pull request #10977: URL: https://github.com/apache/airflow/pull/10977#discussion_r587244007 ## File path: airflow/config_templates/default_test.cfg ## @@ -102,6 +102,18 @@ default_queue = default sync_parallelism = 0 worker_precheck =

[GitHub] [airflow] kenfinster commented on a change in pull request #14588: Fill DagBag from dag_folder setting for upgrade rules

2021-03-04 Thread GitBox
kenfinster commented on a change in pull request #14588: URL: https://github.com/apache/airflow/pull/14588#discussion_r587243479 ## File path: airflow/upgrade/rules/custom_operator_metaclass_rule.py ## @@ -17,6 +17,7 @@ from __future__ import absolute_import +from airflow

<    1   2