[GitHub] [airflow] boring-cyborg[bot] commented on issue #18316: Use numeric sorting while listing DAGs

2021-09-16 Thread GitBox
boring-cyborg[bot] commented on issue #18316: URL: https://github.com/apache/airflow/issues/18316#issuecomment-921547319 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] siddharthvp opened a new issue #18316: Use numeric sorting while listing DAGs

2021-09-16 Thread GitBox
siddharthvp opened a new issue #18316: URL: https://github.com/apache/airflow/issues/18316 ### Apache Airflow version 2.0.2 ### Operating System Linux ### Versions of Apache Airflow Providers NA ### Deployment MWAA ### Deployment detail

[GitHub] [airflow] Jorricks commented on pull request #16634: Require can_edit on DAG privileges to modify TaskInstances and DagRuns

2021-09-16 Thread GitBox
Jorricks commented on pull request #16634: URL: https://github.com/apache/airflow/pull/16634#issuecomment-921531599 Btw: corresponding with this change, we will need to update the airflow documentation. I will create a PR for this but I would like to list the Airflow version for which t

[GitHub] [airflow] mik-laj commented on a change in pull request #18309: task logs for dummy operators

2021-09-16 Thread GitBox
mik-laj commented on a change in pull request #18309: URL: https://github.com/apache/airflow/pull/18309#discussion_r710781654 ## File path: airflow/utils/log/log_reader.py ## @@ -54,6 +57,8 @@ def read_log_chunks( contain information about the task log which can enable

[GitHub] [airflow] ignaski commented on a change in pull request #18042: Fixing ses email backend

2021-09-16 Thread GitBox
ignaski commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r710775770 ## File path: airflow/config_templates/config.yml ## @@ -1351,6 +1351,20 @@ example: "/path/to/my_html_content_template_file" default: ~

[GitHub] [airflow] uranusjr commented on a change in pull request #18042: Fixing ses email backend

2021-09-16 Thread GitBox
uranusjr commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r710774643 ## File path: airflow/config_templates/config.yml ## @@ -1351,6 +1351,20 @@ example: "/path/to/my_html_content_template_file" default: ~

[GitHub] [airflow] ignaski commented on a change in pull request #18042: Fixing ses email backend

2021-09-16 Thread GitBox
ignaski commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r710774449 ## File path: airflow/config_templates/config.yml ## @@ -1351,6 +1351,20 @@ example: "/path/to/my_html_content_template_file" default: ~

[GitHub] [airflow] github-actions[bot] commented on pull request #18310: Fix mini scheduler not respecting `wait_for_downstream` dependency

2021-09-16 Thread GitBox
github-actions[bot] commented on pull request #18310: URL: https://github.com/apache/airflow/pull/18310#issuecomment-921521835 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and tak

[GitHub] [airflow] uranusjr opened a new pull request #18315: Backport isoformat None guard

2021-09-16 Thread GitBox
uranusjr opened a new pull request #18315: URL: https://github.com/apache/airflow/pull/18315 This was fixed in main in #17732, but the commit was not backported to 2.1 since the PR also contains a new feature and does not otherwise make sense on 2.1. Fix #18286 #17814. Not su

[GitHub] [airflow] uranusjr commented on issue #18286: Last Run not loading for DAGs on the listing page

2021-09-16 Thread GitBox
uranusjr commented on issue #18286: URL: https://github.com/apache/airflow/issues/18286#issuecomment-921512794 This is duplicate to #17814 and fixed in main in #17732. Note that the PR is currently not backported to 2.1. -- This is an automated message from the Apache Git Service. To res

[GitHub] [airflow] gargrishabh75 edited a comment on issue #18286: Last Run not loading for DAGs on the listing page

2021-09-16 Thread GitBox
gargrishabh75 edited a comment on issue #18286: URL: https://github.com/apache/airflow/issues/18286#issuecomment-920981829 > CAn you please take a look at webserver logs and paste the error you see there? I think that must be something specific to your setting (I have some guesses but they

[GitHub] [airflow] uranusjr commented on issue #18312: Scheduler TypeError: unsupported operand type(s) for +: 'datetime.datetime' and 'dict'

2021-09-16 Thread GitBox
uranusjr commented on issue #18312: URL: https://github.com/apache/airflow/issues/18312#issuecomment-921511544 Please provide a DAG that reproduces the issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [airflow] uranusjr commented on issue #18314: After upgrading to v2.1.3, the scheduler won't start because `queued_at` is missing in `airflow_db.dag_run`

2021-09-16 Thread GitBox
uranusjr commented on issue #18314: URL: https://github.com/apache/airflow/issues/18314#issuecomment-921508986 Did you run `airflow db upgrade`? -- 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 t

[GitHub] [airflow] fredthomsen commented on a change in pull request #18257: Ability to access http k8s via multiple hostnames

2021-09-16 Thread GitBox
fredthomsen commented on a change in pull request #18257: URL: https://github.com/apache/airflow/pull/18257#discussion_r710751039 ## File path: chart/templates/webserver/webserver-ingress.yaml ## @@ -39,31 +39,38 @@ spec: {{- if .Values.ingress.web.tls.enabled }} tls:

[GitHub] [airflow] tau-mask opened a new issue #18314: After upgrading to v2.1.3, the scheduler won't start because `queued_at` is missing in `airflow_db.dag_run`

2021-09-16 Thread GitBox
tau-mask opened a new issue #18314: URL: https://github.com/apache/airflow/issues/18314 ### Apache Airflow version 2.1.3 (latest released) ### Operating System Ubuntu 20.04 ### Versions of Apache Airflow Providers _No response_ ### Deployment

[GitHub] [airflow] boring-cyborg[bot] commented on issue #18314: After upgrading to v2.1.3, the scheduler won't start because `queued_at` is missing in `airflow_db.dag_run`

2021-09-16 Thread GitBox
boring-cyborg[bot] commented on issue #18314: URL: https://github.com/apache/airflow/issues/18314#issuecomment-921445320 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] yuanke7 opened a new issue #18313: No response from request api(longer than 30min) while using SimpleHttpOperator

2021-09-16 Thread GitBox
yuanke7 opened a new issue #18313: URL: https://github.com/apache/airflow/issues/18313 ### Apache Airflow version 2.1.2 ### Operating System CentOS Linux 7 ### Versions of Apache Airflow Providers apache-airflow-providers-amazon==2.0.0 apache-airflow-pro

[GitHub] [airflow] boring-cyborg[bot] commented on issue #18313: No response from request api(longer than 30min) while using SimpleHttpOperator

2021-09-16 Thread GitBox
boring-cyborg[bot] commented on issue #18313: URL: https://github.com/apache/airflow/issues/18313#issuecomment-921442949 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [airflow] fredthomsen commented on a change in pull request #18257: Ability to access http k8s via multiple hostnames

2021-09-16 Thread GitBox
fredthomsen commented on a change in pull request #18257: URL: https://github.com/apache/airflow/pull/18257#discussion_r710727363 ## File path: chart/templates/webserver/webserver-ingress.yaml ## @@ -39,31 +39,38 @@ spec: {{- if .Values.ingress.web.tls.enabled }} tls:

[GitHub] [airflow] kodeine commented on issue #13679: SQL Syntax errors on startup

2021-09-16 Thread GitBox
kodeine commented on issue #13679: URL: https://github.com/apache/airflow/issues/13679#issuecomment-921432337 Hello @kaxil , I am still getting this error, ``` ostgresql_1 | 2021-09-17 03:45:45.666 GMT [252] LOG: incomplete startup packet postgresql_1 | 2021-09-17 03

[GitHub] [airflow] imamdigmi opened a new issue #18312: Scheduler TypeError: unsupported operand type(s) for +: 'datetime.datetime' and 'dict'

2021-09-16 Thread GitBox
imamdigmi opened a new issue #18312: URL: https://github.com/apache/airflow/issues/18312 ### Apache Airflow version 2.1.3 (latest released) ### Operating System Debian GNU/Linux 10 (buster) ### Versions of Apache Airflow Providers _No response_ ### D

[GitHub] [airflow] josh-fell commented on issue #9415: [AIP-31] Update DAG examples to use Functional DAG style

2021-09-16 Thread GitBox
josh-fell commented on issue #9415: URL: https://github.com/apache/airflow/issues/9415#issuecomment-921402673 > What do you think about this? I feel it's good: > > ```python > from airflow.example_dags.libs.helper import print_stuff > > @task > def start_task(): >

[GitHub] [airflow] imamdigmi commented on issue #18286: Last Run not loading for DAGs on the listing page

2021-09-16 Thread GitBox
imamdigmi commented on issue #18286: URL: https://github.com/apache/airflow/issues/18286#issuecomment-921397047 yes, I encounter the same problem ![image](https://user-images.githubusercontent.com/10998240/133712122-7aee68ee-dee5-4428-b2de-7b889b492aca.png) -- This is an automated

[GitHub] [airflow] bhavaniravi commented on a change in pull request #18226: Add start date to trigger_dagrun operator

2021-09-16 Thread GitBox
bhavaniravi commented on a change in pull request #18226: URL: https://github.com/apache/airflow/pull/18226#discussion_r710122853 ## File path: airflow/api/common/experimental/trigger_dag.py ## @@ -82,6 +82,7 @@ def _trigger_dag( for _dag in dags_to_trigger: trigg

[GitHub] [airflow] bigyan-kc commented on issue #18283: db+ string in result backend but not metadata secret

2021-09-16 Thread GitBox
bigyan-kc commented on issue #18283: URL: https://github.com/apache/airflow/issues/18283#issuecomment-921384894 @EliMor sure I will also add comment to the values file. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [airflow] ephraimbuddy commented on pull request #18310: Fix mini scheduler not respecting `wait_for_downstream` dependency

2021-09-16 Thread GitBox
ephraimbuddy commented on pull request #18310: URL: https://github.com/apache/airflow/pull/18310#issuecomment-921380276 You can reproduce the behaviour with this DAG: ```python import datetime from airflow import DAG from airflow.operators.bash import BashOperator defa

[GitHub] [airflow] ephraimbuddy commented on issue #18011: Task stuck in upstream_failed

2021-09-16 Thread GitBox
ephraimbuddy commented on issue #18011: URL: https://github.com/apache/airflow/issues/18011#issuecomment-921379642 @WattsInABox. If you can get scheduler logs when this happens, that would be very helpful. -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [airflow] ephraimbuddy opened a new pull request #18310: Fix mini scheduler not respecting `wait_for_downstream` dependency

2021-09-16 Thread GitBox
ephraimbuddy opened a new pull request #18310: URL: https://github.com/apache/airflow/pull/18310 When wait_for_downstream is set on a task, mini scheduler doesn't respect it and goes ahead to schedule unrunnable task instances. This PR fixes it by checking the dependency in mini sc

[GitHub] [airflow] WattsInABox commented on issue #18011: Task stuck in upstream_failed

2021-09-16 Thread GitBox
WattsInABox commented on issue #18011: URL: https://github.com/apache/airflow/issues/18011#issuecomment-921371431 We're also seeing this. Most of our DAGs have well under 100 tasks, a few just under 200 tasks, 673 active DAGs, 179 paused DAGs. We do not use `wait_for_downstream` anywhere.

[GitHub] [airflow] jaketf opened a new pull request #18309: task logs for dummy operators

2021-09-16 Thread GitBox
jaketf opened a new pull request #18309: URL: https://github.com/apache/airflow/pull/18309 add dummy task logs for dummy operators --- **^ Add meaningful description above** Currently if a user clicks in the webserver on a node that is a `DummyOperator` and tries to rea

[GitHub] [airflow] github-actions[bot] commented on pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
github-actions[bot] commented on pull request #18282: URL: https://github.com/apache/airflow/pull/18282#issuecomment-921344885 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide

[GitHub] [airflow] github-actions[bot] commented on issue #17343: Dags added by DagBag interrupt randomly

2021-09-16 Thread GitBox
github-actions[bot] commented on issue #17343: URL: https://github.com/apache/airflow/issues/17343#issuecomment-921344500 This issue has been automatically marked as stale because it has been open for 30 days with no response from the author. It will be closed in next 7 days if no further

[GitHub] [airflow] mik-laj commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
mik-laj commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710590117 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,317 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or mo

[GitHub] [airflow] mik-laj commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
mik-laj commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710589010 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,317 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or mo

[GitHub] [airflow] mik-laj commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
mik-laj commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710588518 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,317 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or mo

[airflow] branch main updated (7e132a0 -> 11621ce)

2021-09-16 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 7e132a0 Fix spelling on chart docs (#18308) add 11621ce Build docs when chart changelog or values schema

[GitHub] [airflow] jedcunningham merged pull request #18307: Build docs when chart changelog or values schema changes

2021-09-16 Thread GitBox
jedcunningham merged pull request #18307: URL: https://github.com/apache/airflow/pull/18307 -- 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 specific comment. To unsubscribe, e-mail: commit

[airflow] branch main updated (246e5b4 -> 7e132a0)

2021-09-16 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 246e5b4 Pass exception to ``run_finished_callback`` for Debug Executor (#17983) add 7e132a0 Fix spelling on ch

[GitHub] [airflow] potiuk merged pull request #18308: Fix spelling on chart docs

2021-09-16 Thread GitBox
potiuk merged pull request #18308: URL: https://github.com/apache/airflow/pull/18308 -- 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 specific comment. To unsubscribe, e-mail: commits-unsub

[GitHub] [airflow] jedcunningham opened a new pull request #18308: Fix spelling on chart docs

2021-09-16 Thread GitBox
jedcunningham opened a new pull request #18308: URL: https://github.com/apache/airflow/pull/18308 There is a spelling error in the chart docs on main (my bad, #18307 should prevent it in the future). -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] potiuk commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710557478 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,316 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or mor

[GitHub] [airflow] github-actions[bot] commented on pull request #18307: Build docs when chart changelog or values schema changes

2021-09-16 Thread GitBox
github-actions[bot] commented on pull request #18307: URL: https://github.com/apache/airflow/pull/18307#issuecomment-921319763 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and tak

[GitHub] [airflow] jedcunningham opened a new pull request #18307: Build docs when chart changelog or values schema changes

2021-09-16 Thread GitBox
jedcunningham opened a new pull request #18307: URL: https://github.com/apache/airflow/pull/18307 Thought of another couple of chart files that should lead to a docs build. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] jedcunningham commented on a change in pull request #18284: Display flash messages on dashboard from local settings

2021-09-16 Thread GitBox
jedcunningham commented on a change in pull request #18284: URL: https://github.com/apache/airflow/pull/18284#discussion_r710545867 ## File path: airflow/settings.py ## @@ -519,3 +520,13 @@ def initialize(): # By default this is off, but is automatically configured on when run

[GitHub] [airflow] ridwantahir edited a comment on issue #17507: Task processes killed with WARNING - Recorded pid does not match the current pid

2021-09-16 Thread GitBox
ridwantahir edited a comment on issue #17507: URL: https://github.com/apache/airflow/issues/17507#issuecomment-921305515 one tick that worked for me was to delete the task instance(or all task instances of a particular dag run) from airflow ui -- This is an automated message from the Apa

[GitHub] [airflow] ridwantahir commented on issue #17507: Task processes killed with WARNING - Recorded pid does not match the current pid

2021-09-16 Thread GitBox
ridwantahir commented on issue #17507: URL: https://github.com/apache/airflow/issues/17507#issuecomment-921305515 one tick that worked for me was to delete the task instance(or all task instances of a particular dag run) -- This is an automated message from the Apache Git Service. To res

[GitHub] [airflow] kaxil commented on a change in pull request #17819: Properly handle ti state difference between executor and scheduler

2021-09-16 Thread GitBox
kaxil commented on a change in pull request #17819: URL: https://github.com/apache/airflow/pull/17819#discussion_r710539698 ## File path: tests/models/test_taskinstance.py ## @@ -1683,6 +1683,21 @@ def test_handle_failure(self, create_dummy_dag, session=None): assert

[GitHub] [airflow] kaxil commented on a change in pull request #17819: Properly handle ti state difference between executor and scheduler

2021-09-16 Thread GitBox
kaxil commented on a change in pull request #17819: URL: https://github.com/apache/airflow/pull/17819#discussion_r710539698 ## File path: tests/models/test_taskinstance.py ## @@ -1683,6 +1683,21 @@ def test_handle_failure(self, create_dummy_dag, session=None): assert

[GitHub] [airflow] potiuk commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710528880 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,316 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or mor

[GitHub] [airflow] potiuk commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710527820 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,316 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or mor

[GitHub] [airflow] potiuk commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710527820 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,316 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or mor

[airflow] branch main updated (475fcd1 -> 246e5b4)

2021-09-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 475fcd1 Added Bentego to INTHEWILD.md (#18301) add 246e5b4 Pass exception to ``run_finished_callback`` for D

[GitHub] [airflow] kaxil merged pull request #17983: Pass exception to run_finished_callback for Debug Executor

2021-09-16 Thread GitBox
kaxil merged pull request #17983: URL: https://github.com/apache/airflow/pull/17983 -- 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 specific comment. To unsubscribe, e-mail: commits-unsubs

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #17983: Pass exception to run_finished_callback for Debug Executor

2021-09-16 Thread GitBox
boring-cyborg[bot] commented on pull request #17983: URL: https://github.com/apache/airflow/pull/17983#issuecomment-921282379 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] kaxil commented on a change in pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
kaxil commented on a change in pull request #18282: URL: https://github.com/apache/airflow/pull/18282#discussion_r710520015 ## File path: docs/apache-airflow/installation/index.rst ## @@ -0,0 +1,316 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more

[airflow] branch main updated (5e9a877 -> 475fcd1)

2021-09-16 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 5e9a877 Add information about differencies of deps in eager-upgrade (#18293) add 475fcd1 Added Bentego to INTH

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #18301: Added Bentego to INTHEWILD.md

2021-09-16 Thread GitBox
boring-cyborg[bot] commented on pull request #18301: URL: https://github.com/apache/airflow/pull/18301#issuecomment-921273701 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] potiuk merged pull request #18301: Added Bentego to INTHEWILD.md

2021-09-16 Thread GitBox
potiuk merged pull request #18301: URL: https://github.com/apache/airflow/pull/18301 -- 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 specific comment. To unsubscribe, e-mail: commits-unsub

[GitHub] [airflow] potiuk commented on pull request #18282: Refactor installation pages

2021-09-16 Thread GitBox
potiuk commented on pull request #18282: URL: https://github.com/apache/airflow/pull/18282#issuecomment-921272711 Anyone :) ? I really think it's going to help new users. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[airflow] branch main updated (3ddb389 -> 5e9a877)

2021-09-16 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 3ddb389 Run Docs build when updating `chart/UPDATING.rst` (#18303) add 5e9a877 Add information about differenci

[GitHub] [airflow] potiuk commented on pull request #18293: Add information about differences of deps in eager-upgrade

2021-09-16 Thread GitBox
potiuk commented on pull request #18293: URL: https://github.com/apache/airflow/pull/18293#issuecomment-921272083 Better now :) ![image](https://user-images.githubusercontent.com/595491/133688908-6bfc574d-20ab-49cb-bbfa-42d4201847c5.png) -- This is an automated message from th

[GitHub] [airflow] potiuk merged pull request #18293: Add information about differences of deps in eager-upgrade

2021-09-16 Thread GitBox
potiuk merged pull request #18293: URL: https://github.com/apache/airflow/pull/18293 -- 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 specific comment. To unsubscribe, e-mail: commits-unsub

[GitHub] [airflow] ephraimbuddy commented on issue #9415: [AIP-31] Update DAG examples to use Functional DAG style

2021-09-16 Thread GitBox
ephraimbuddy commented on issue #9415: URL: https://github.com/apache/airflow/issues/9415#issuecomment-921271184 What do you think about this? I feel it's good: ```python from airflow.example_dags.libs.helper import print_stuff @task def start_task(): print_stuff(

[GitHub] [airflow] jedcunningham opened a new pull request #18306: Chart: warn when webserver secret key isn't set

2021-09-16 Thread GitBox
jedcunningham opened a new pull request #18306: URL: https://github.com/apache/airflow/pull/18306 We will start warning chart admins when they deploy with the default random webserver secret key, as it can lead to unnecessary restarts of the Airflow components. Message on the da

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #18305: add sla_miss_callback section to the documentation

2021-09-16 Thread GitBox
boring-cyborg[bot] commented on pull request #18305: URL: https://github.com/apache/airflow/pull/18305#issuecomment-921227161 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Con

[GitHub] [airflow] alex-astronomer opened a new pull request #18305: add sla_miss_callback section to the documentation

2021-09-16 Thread GitBox
alex-astronomer opened a new pull request #18305: URL: https://github.com/apache/airflow/pull/18305 Closes: #15641 Adds documentation for `sla_miss_callback`. Also links from `DAG` class's docstring to the `sla_miss_callback` section of `tasks.rst` where required. ![ima

[GitHub] [airflow] josh-fell commented on issue #9415: [AIP-31] Update DAG examples to use Functional DAG style

2021-09-16 Thread GitBox
josh-fell commented on issue #9415: URL: https://github.com/apache/airflow/issues/9415#issuecomment-921226065 In other PRs there was discussion of removing `PythonOperator` use in the example DAGs but I'm wondering what everyone's thoughts are on handling imported functions that are called

[GitHub] [airflow] josh-fell commented on a change in pull request #18278: Updating miscellaneous provider DAGs to use TaskFlow API where applicable

2021-09-16 Thread GitBox
josh-fell commented on a change in pull request #18278: URL: https://github.com/apache/airflow/pull/18278#discussion_r710448830 ## File path: airflow/providers/apache/kylin/example_dags/example_kylin_dag.py ## @@ -21,110 +21,92 @@ The tasks below include kylin build, refresh,

[GitHub] [airflow] wolfier opened a new issue #18304: User induced deadlock for DAGs with `depends_on_past`

2021-09-16 Thread GitBox
wolfier opened a new issue #18304: URL: https://github.com/apache/airflow/issues/18304 ### Apache Airflow version main (development) ### Operating System Debian GNU/Linux 10 (buster) ### Versions of Apache Airflow Providers _No response_ ### Deployme

[GitHub] [airflow] MhmtErsy commented on pull request #18301: Added Bentego to INTHEWILD.md

2021-09-16 Thread GitBox
MhmtErsy commented on pull request #18301: URL: https://github.com/apache/airflow/pull/18301#issuecomment-921214140 Hi @mik-laj , I fixed the order numbers to 1. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] kaxil commented on pull request #18293: Add information about differences of deps in eager-upgrade

2021-09-16 Thread GitBox
kaxil commented on pull request #18293: URL: https://github.com/apache/airflow/pull/18293#issuecomment-921184651 > But I need to reverse it :) Why, n! :-D -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[airflow] branch main updated (8692cbf -> 3ddb389)

2021-09-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 8692cbf Fix Chart doc build (#18302) add 3ddb389 Run Docs build when updating `chart/UPDATING.rst` (#18303)

[GitHub] [airflow] kaxil merged pull request #18303: Run Docs build when updating `chart/UPDATING.rst`

2021-09-16 Thread GitBox
kaxil merged pull request #18303: URL: https://github.com/apache/airflow/pull/18303 -- 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 specific comment. To unsubscribe, e-mail: commits-unsubs

[GitHub] [airflow] potiuk edited a comment on pull request #18293: Add information about differences of deps in eager-upgrade

2021-09-16 Thread GitBox
potiuk edited a comment on pull request #18293: URL: https://github.com/apache/airflow/pull/18293#issuecomment-921109314 But I need to reverse it :) -- 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

[airflow] branch main updated (0527624 -> 8692cbf)

2021-09-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 0527624 Chart: Fix applying labels on Triggerer (#18299) add 8692cbf Fix Chart doc build (#18302) No new re

[GitHub] [airflow] kaxil merged pull request #18302: Fix Chart doc build

2021-09-16 Thread GitBox
kaxil merged pull request #18302: URL: https://github.com/apache/airflow/pull/18302 -- 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 specific comment. To unsubscribe, e-mail: commits-unsubs

[GitHub] [airflow] github-actions[bot] commented on pull request #18303: Run Docs build when updating `chart/UPDATING.rst`

2021-09-16 Thread GitBox
github-actions[bot] commented on pull request #18303: URL: https://github.com/apache/airflow/pull/18303#issuecomment-921156453 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and tak

[GitHub] [airflow] kaxil opened a new pull request #18303: Run Docs build when updating `chart/UPDATING.rst`

2021-09-16 Thread GitBox
kaxil opened a new pull request #18303: URL: https://github.com/apache/airflow/pull/18303 Run Docs build when updating `chart/UPDATING.rst` since we generate docs based on it https://github.com/apache/airflow/blob/main/docs/helm-chart/updating.rst --- **^ Add meaningful d

[GitHub] [airflow] github-actions[bot] commented on pull request #18302: Fix Chart doc build

2021-09-16 Thread GitBox
github-actions[bot] commented on pull request #18302: URL: https://github.com/apache/airflow/pull/18302#issuecomment-921154331 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not m

[GitHub] [airflow] kaxil opened a new pull request #18302: Fix Chart doc build

2021-09-16 Thread GitBox
kaxil opened a new pull request #18302: URL: https://github.com/apache/airflow/pull/18302 Error on main: ``` WARNING: Title underline too short. ``` --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apac

[GitHub] [airflow] Jorricks commented on pull request #16634: Require can_edit on DAG privileges to modify TaskInstances and DagRuns

2021-09-16 Thread GitBox
Jorricks commented on pull request #16634: URL: https://github.com/apache/airflow/pull/16634#issuecomment-921142840 > That’s what happens when you have a file containing ~5k lines of source code 😞 One day I’m going to sit down and split `www/views.py` into a directory with like 20 files.

[GitHub] [airflow] ephraimbuddy commented on issue #18011: Task stuck in upstream_failed

2021-09-16 Thread GitBox
ephraimbuddy commented on issue #18011: URL: https://github.com/apache/airflow/issues/18011#issuecomment-921135750 @Tonkonozhenko Do you have `wait_for_downstream` args set? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] kaxil merged pull request #18299: Chart: Fix applying labels on Triggerer

2021-09-16 Thread GitBox
kaxil merged pull request #18299: URL: https://github.com/apache/airflow/pull/18299 -- 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 specific comment. To unsubscribe, e-mail: commits-unsubs

[airflow] branch main updated (73044af -> 0527624)

2021-09-16 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 73044af Chart: Use stable API versions where available (#17211) add 0527624 Chart: Fix applying labels on Tr

[GitHub] [airflow] potiuk commented on a change in pull request #18298: Missing command for Python3 users

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #18298: URL: https://github.com/apache/airflow/pull/18298#discussion_r710361149 ## File path: docs/apache-airflow/start/local.rst ## @@ -48,6 +48,8 @@ constraint files to enable reproducible installation, so using ``pip`` and const

[GitHub] [airflow] potiuk closed pull request #18298: Missing command for Python3 users

2021-09-16 Thread GitBox
potiuk closed pull request #18298: URL: https://github.com/apache/airflow/pull/18298 -- 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 specific comment. To unsubscribe, e-mail: commits-unsub

[GitHub] [airflow] kaxil commented on a change in pull request #18042: Fixing ses email backend

2021-09-16 Thread GitBox
kaxil commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r710355525 ## File path: airflow/utils/email.py ## @@ -68,6 +73,8 @@ def send_email( def send_email_smtp( +from_email: str, +from_name: str, Review com

[GitHub] [airflow] kaxil commented on a change in pull request #18042: Fixing ses email backend

2021-09-16 Thread GitBox
kaxil commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r710355300 ## File path: airflow/utils/email.py ## @@ -68,6 +73,8 @@ def send_email( def send_email_smtp( +from_email: str, +from_name: str, Review com

[GitHub] [airflow] kaxil commented on a change in pull request #18042: Fixing ses email backend

2021-09-16 Thread GitBox
kaxil commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r710353998 ## File path: airflow/config_templates/config.yml ## @@ -1351,6 +1351,20 @@ example: "/path/to/my_html_content_template_file" default: ~

[GitHub] [airflow] jedcunningham edited a comment on pull request #17211: Chart: Use stable API versions where available

2021-09-16 Thread GitBox
jedcunningham edited a comment on pull request #17211: URL: https://github.com/apache/airflow/pull/17211#issuecomment-921112562 Thanks @cablespaghetti! Congrats on your first commit. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [airflow] jedcunningham commented on pull request #17211: Chart: Use stable API versions where available

2021-09-16 Thread GitBox
jedcunningham commented on pull request #17211: URL: https://github.com/apache/airflow/pull/17211#issuecomment-921112562 Thanks @cablespaghetti! -- 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 t

[airflow] branch main updated (2dac083 -> 73044af)

2021-09-16 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 2dac083 Fixed wasb hook attempting to create container when getting a blob client (#18287) add 73044af

[GitHub] [airflow] jedcunningham closed issue #17188: Chart: Uses deprecated v1beta1 Ingress Resource. Incompatible with Kubernetes 1.22

2021-09-16 Thread GitBox
jedcunningham closed issue #17188: URL: https://github.com/apache/airflow/issues/17188 -- 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 specific comment. To unsubscribe, e-mail: commits-uns

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #17211: Chart: Use stable API versions where available

2021-09-16 Thread GitBox
boring-cyborg[bot] commented on pull request #17211: URL: https://github.com/apache/airflow/pull/17211#issuecomment-921112371 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] jedcunningham merged pull request #17211: Chart: Use stable API versions where available

2021-09-16 Thread GitBox
jedcunningham merged pull request #17211: URL: https://github.com/apache/airflow/pull/17211 -- 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 specific comment. To unsubscribe, e-mail: commit

[GitHub] [airflow] potiuk commented on pull request #18293: Add information about differences of deps in eager-upgrade

2021-09-16 Thread GitBox
potiuk commented on pull request #18293: URL: https://github.com/apache/airflow/pull/18293#issuecomment-921109314 But I need to revert it :) -- 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] potiuk commented on pull request #18293: Add information about differences of deps in eager-upgrade

2021-09-16 Thread GitBox
potiuk commented on pull request #18293: URL: https://github.com/apache/airflow/pull/18293#issuecomment-921108886 Nice: @uranusjr @kaxil - you will like it :) ![image](https://user-images.githubusercontent.com/595491/133660750-406db83a-3a7e-45b5-8a5e-771338aa7afb.png) -- This

[GitHub] [airflow] potiuk commented on a change in pull request #17682: Import Hooks lazily individually in providers manager

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r710342369 ## File path: airflow/provider.yaml.schema.json ## @@ -195,11 +195,33 @@ }, "hook-class-names": { "type": "array", - "description":

[GitHub] [airflow] potiuk commented on a change in pull request #17682: Import Hooks lazily individually in providers manager

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r710340282 ## File path: airflow/provider.yaml.schema.json ## @@ -195,11 +195,33 @@ }, "hook-class-names": { "type": "array", - "description":

[GitHub] [airflow] potiuk commented on a change in pull request #17682: Import Hooks lazily individually in providers manager

2021-09-16 Thread GitBox
potiuk commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r710340282 ## File path: airflow/provider.yaml.schema.json ## @@ -195,11 +195,33 @@ }, "hook-class-names": { "type": "array", - "description":

  1   2   3   >