[airflow] branch main updated (258451c -> 406b38b)

2021-10-14 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 258451c MySQLToS3Operator add support for parquet format (#18755) add 406b38b Replace default api_version of

[airflow] branch constraints-main updated: Updating constraints. Build id:1344389256

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

[airflow] branch main updated (3f6d9b6 -> 258451c)

2021-10-14 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 3f6d9b6 Handle case of nonexistent file when preparing file path queue (#18998) add 258451c

[airflow] branch main updated: Handle case of nonexistent file when preparing file path queue (#18998)

2021-10-14 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 3f6d9b6 Handle case of nonexistent file when

[airflow] branch constraints-main updated: Updating constraints. Build id:1344096896

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

[airflow] branch main updated (cc627b3 -> fb00240)

2021-10-14 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 cc627b3 Add decription on how you can customize image entrypoint (#18915) add fb00240 Update CSV ingest code

[airflow] 09/10: Try to move "dangling" rows in upgradedb (#18953)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 327b83967881b5c5a8973b8d2dbb1e8c39c051d6 Author: Tzu-ping Chung AuthorDate: Fri Oct 15 01:09:54 2021 +0800

[airflow] 07/10: Sentry before send fallback (#18980)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 1a45e1ead6f5a76b4e21b2ea49c1ba6ce03c72f3 Author: Leon Smith <_...@leonmarksmith.com> AuthorDate: Thu Oct 14

[airflow] 06/10: Fix XCom.delete error in Airflow 2.2.0 (#18956)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit b329f8d3d71ead96f8ef7d5f6eb51f0525993b9d Author: jordanjeremy

[airflow] 08/10: Row lock TI query in SchedulerJob._process_executor_events (#18975)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 5f85eef2cc252affc7049278f507579e2f6f6814 Author: Ephraim Anierobi AuthorDate: Thu Oct 14 16:09:38 2021 +0100

[airflow] 10/10: BugFix: Null execution date on insert to ``task_fail`` violating NOT NULL (#18979)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit e35588c6cbf0cb75ab914d8f31252c4ff87fde89 Author: Robin Edwards AuthorDate: Thu Oct 14 20:46:51 2021 +0100

[airflow] 01/10: Group PATCH DAGrun together with other DAGRun endpoints (#18885)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 032eda12ed7d70b5ab3f06fecf626716ecd025bf Author: Bas Harenslak AuthorDate: Mon Oct 11 20:55:09 2021 +0200

[airflow] 02/10: Doc: Add Callbacks Section to Logging & Monitoring (#18842)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit bc551c146efbaecb385357c664a905eb0a437d75 Author: john-jac <75442233+john-...@users.noreply.github.com>

[airflow] 05/10: Changes related to PR #16634 (#18644)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit f3dd5b50513fde34c3dc547a9b68a6ac36f1b8d6 Author: Jorrick Sleijster AuthorDate: Thu Oct 14 12:53:02 2021

[airflow] 03/10: Add information about keepalives for managed Postgres (#18850)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 85ec578d33ec7be29f3c42ada85232267a684358 Author: Jarek Potiuk AuthorDate: Tue Oct 12 22:54:51 2021 +0200

[airflow] 04/10: Check python version before starting triggerer (#18926)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit 7183ad097ffa182428c3b8cd3266dac192bd3d34 Author: Jed Cunningham

[airflow] branch v2-2-test updated (e3d07e9 -> e35588c)

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a change to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git. from e3d07e9 Bump version to 2.2.1 new 032eda1 Group PATCH DAGrun together with other DAGRun endpoints

[airflow] branch main updated: Add decription on how you can customize image entrypoint (#18915)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new cc627b3 Add decription on how you can customize

[airflow] branch v2-2-test updated: Bump version to 2.2.1

2021-10-14 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch v2-2-test in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/v2-2-test by this push: new e3d07e9 Bump version to 2.2.1

[airflow] branch main updated: BugFix: Null execution date on insert to ``task_fail`` violating NOT NULL (#18979)

2021-10-14 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 2966e3c BugFix: Null execution date on insert

[airflow] branch potiuk-patch-1 updated (559b9ec -> 7e391f4)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from 559b9ec Update docs/docker-stack/entrypoint.rst add 7e391f4 Update docs/docker-stack/entrypoint.rst

[airflow] branch potiuk-patch-1 updated (c19cb70 -> 559b9ec)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from c19cb70 Update docs/docker-stack/entrypoint.rst add 559b9ec Update docs/docker-stack/entrypoint.rst

[airflow] branch potiuk-patch-1 updated (afa06c3 -> c19cb70)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from afa06c3 Update docs/docker-stack/entrypoint.rst add c19cb70 Update docs/docker-stack/entrypoint.rst

[airflow] branch potiuk-patch-1 updated (b90f0fc -> afa06c3)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from b90f0fc Update docs/docker-stack/entrypoint.rst add afa06c3 Update docs/docker-stack/entrypoint.rst

[airflow] branch potiuk-patch-1 updated (50e9894 -> b90f0fc)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from 50e9894 Update docs/docker-stack/entrypoint.rst add b90f0fc Update docs/docker-stack/entrypoint.rst

[airflow] branch potiuk-patch-1 updated (bb4fd13 -> 50e9894)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from bb4fd13 Add decription on how you can customize image entrypoint add 50e9894 Update

[airflow] branch main updated: Add pre-commit hook for common misspelling check in files (#18964)

2021-10-14 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 1571f80 Add pre-commit hook for common

[airflow] branch main updated (f967ca9 -> 141d9f2)

2021-10-14 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 f967ca9 Try to move "dangling" rows in upgradedb (#18953) add 141d9f2 Adds Pendulum 1.x -> 2.x upgrade

[airflow] branch main updated (2ba722d -> f967ca9)

2021-10-14 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 2ba722d Fix typo in ``tutorial.rst`` (#18983) add f967ca9 Try to move "dangling" rows in upgradedb (#18953)

[GitHub] [airflow] stroykova commented on issue #14672: Tasks intermittently get terminated with SIGTERM on K8s executor

2021-10-14 Thread GitBox
stroykova commented on issue #14672: URL: https://github.com/apache/airflow/issues/14672#issuecomment-943508653 I use airflow in a data science project with lots of multiprocessing. I also have the same issue with SIGTERM signal. Here is a code example that shows using multiprocessing

[GitHub] [airflow] kaxil closed pull request #18985: Restoring additional context in Slack operators how-to guide

2021-10-14 Thread GitBox
kaxil closed pull request #18985: URL: https://github.com/apache/airflow/pull/18985 -- 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:

[airflow] branch main updated (52cc84c -> 2ba722d)

2021-10-14 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 52cc84c Row lock TI query in SchedulerJob._process_executor_events (#18975) add 2ba722d Fix typo in

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #18983: Update tutorial.rst

2021-10-14 Thread GitBox
boring-cyborg[bot] commented on pull request #18983: URL: https://github.com/apache/airflow/pull/18983#issuecomment-943502887 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

[GitHub] [airflow] kaxil merged pull request #18983: Update tutorial.rst

2021-10-14 Thread GitBox
kaxil merged pull request #18983: URL: https://github.com/apache/airflow/pull/18983 -- 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:

[GitHub] [airflow] 30blay commented on pull request #16338: SFTP hook to prefer the SSH paramiko key over the key file path

2021-10-14 Thread GitBox
30blay commented on pull request #16338: URL: https://github.com/apache/airflow/pull/16338#issuecomment-943492539 @eladkal Thanks, done! https://github.com/apache/airflow/pull/18988 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #18988: SFTP hook to prefer the SSH paramiko key over the key file path

2021-10-14 Thread GitBox
boring-cyborg[bot] commented on pull request #18988: URL: https://github.com/apache/airflow/pull/18988#issuecomment-943491951 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

[GitHub] [airflow] 30blay opened a new pull request #18988: SFTP hook to prefer the SSH paramiko key over the key file path

2021-10-14 Thread GitBox
30blay opened a new pull request #18988: URL: https://github.com/apache/airflow/pull/18988 Related issue: [#16323](https://github.com/apache/airflow/issues/16323) The SFTP hook does not allow for a SSH private key string to be used, only a private key file path. As it is based on

[GitHub] [airflow] bbovenzi edited a comment on issue #18587: vertical spacing between tasks has increased significantly in Graph View

2021-10-14 Thread GitBox
bbovenzi edited a comment on issue #18587: URL: https://github.com/apache/airflow/issues/18587#issuecomment-929892856 @ryanahamilton You updated the styling for the graph view a little while back. Was the extra spacing for labels? I'm curious to hear your opinion. From the commit

[airflow] branch potiuk-patch-1 updated (46519ef -> bb4fd13)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. discard 46519ef Add decription on how you can customize image entrypoint add bb4fd13 Add decription on how

[GitHub] [airflow] bbovenzi edited a comment on issue #18587: vertical spacing between tasks has increased significantly in Graph View

2021-10-14 Thread GitBox
bbovenzi edited a comment on issue #18587: URL: https://github.com/apache/airflow/issues/18587#issuecomment-929892856 @ryanahamilton You updated the styling for the graph view a little while back. Was the extra spacing for labels? I'm curious to hear your opinion. From the commit

[GitHub] [airflow] potiuk commented on a change in pull request #18915: Add decription on how you can customize image entrypoint

2021-10-14 Thread GitBox
potiuk commented on a change in pull request #18915: URL: https://github.com/apache/airflow/pull/18915#discussion_r729116384 ## File path: docs/docker-stack/entrypoint.rst ## @@ -194,6 +194,69 @@ If there are any other arguments - they are simply passed to the "airflow" comma

[airflow] branch potiuk-patch-1 updated (6ee760b -> 46519ef)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. discard 6ee760b Update docs/docker-stack/entrypoint.rst discard 1db1958 Update docs/docker-stack/entrypoint.rst

[GitHub] [airflow] eladkal commented on a change in pull request #18987: Document `hdfs_namenode_principal` for HDFS connections

2021-10-14 Thread GitBox
eladkal commented on a change in pull request #18987: URL: https://github.com/apache/airflow/pull/18987#discussion_r729110911 ## File path: docs/apache-airflow-providers-apache-hdfs/connections.rst ## @@ -44,6 +44,8 @@ Extra (optional, connection parameters) parameters

[GitHub] [airflow] eladkal commented on pull request #16338: SFTP hook to prefer the SSH paramiko key over the key file path

2021-10-14 Thread GitBox
eladkal commented on pull request #16338: URL: https://github.com/apache/airflow/pull/16338#issuecomment-943478497 @30blay since the PR is stale you can take over and open a new PR. You can preserve the original commits so it will be co-authored by the original contributor and you. --

[GitHub] [airflow] potiuk commented on a change in pull request #18915: Add custom entrypoint documentation

2021-10-14 Thread GitBox
potiuk commented on a change in pull request #18915: URL: https://github.com/apache/airflow/pull/18915#discussion_r729107401 ## File path: docs/docker-stack/entrypoint.rst ## @@ -194,6 +194,69 @@ If there are any other arguments - they are simply passed to the "airflow" comma

[airflow] branch potiuk-patch-1 updated (1db1958 -> 6ee760b)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from 1db1958 Update docs/docker-stack/entrypoint.rst add 6ee760b Update docs/docker-stack/entrypoint.rst

[GitHub] [airflow] uranusjr commented on pull request #18953: Try to move "dangling" rows in upgradedb

2021-10-14 Thread GitBox
uranusjr commented on pull request #18953: URL: https://github.com/apache/airflow/pull/18953#issuecomment-943466398 Also I somehow forgot to push the fix to the db query count test before :( Waiting for CI to pass... -- This is an automated message from the Apache Git Service. To

[GitHub] [airflow] jedcunningham commented on a change in pull request #18915: Add custom entrypoint documentation

2021-10-14 Thread GitBox
jedcunningham commented on a change in pull request #18915: URL: https://github.com/apache/airflow/pull/18915#discussion_r729096305 ## File path: docs/docker-stack/entrypoint.rst ## @@ -194,6 +194,69 @@ If there are any other arguments - they are simply passed to the

[GitHub] [airflow] uranusjr commented on a change in pull request #18953: Try to move "dangling" rows in upgradedb

2021-10-14 Thread GitBox
uranusjr commented on a change in pull request #18953: URL: https://github.com/apache/airflow/pull/18953#discussion_r729094722 ## File path: airflow/www/templates/airflow/dags.html ## @@ -51,6 +51,14 @@ {% for m in dashboard_alerts %} {{ message(m.message, m.category)

[airflow] branch main updated (1edcd42 -> 52cc84c)

2021-10-14 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 1edcd42 Sentry before send fallback (#18980) add 52cc84c Row lock TI query in

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

2021-10-14 Thread GitBox
ephraimbuddy commented on issue #17507: URL: https://github.com/apache/airflow/issues/17507#issuecomment-943454316 For those having this issue while using multiple schedulers, I added a fix of an issue I got while running two schedulers. You can try the patch. See

[GitHub] [airflow] ephraimbuddy merged pull request #18975: Row lock TI query in SchedulerJob._process_executor_events

2021-10-14 Thread GitBox
ephraimbuddy merged pull request #18975: URL: https://github.com/apache/airflow/pull/18975 -- 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:

[GitHub] [airflow] jedcunningham commented on a change in pull request #18953: Try to move "dangling" rows in upgradedb

2021-10-14 Thread GitBox
jedcunningham commented on a change in pull request #18953: URL: https://github.com/apache/airflow/pull/18953#discussion_r729076205 ## File path: airflow/www/templates/airflow/dags.html ## @@ -51,6 +51,14 @@ {% for m in dashboard_alerts %} {{ message(m.message,

[GitHub] [airflow] ephraimbuddy commented on pull request #18975: Row lock TI query in SchedulerJob._process_executor_events

2021-10-14 Thread GitBox
ephraimbuddy commented on pull request #18975: URL: https://github.com/apache/airflow/pull/18975#issuecomment-943449013 > Looks green! BTW. We have LOT MORE GREEN nowadays after all those small flaky test fixes :D. Yay!!  Thanks for the good works -- This is an automated

[GitHub] [airflow] bbovenzi commented on a change in pull request #16931: #16692 show cron description in UI

2021-10-14 Thread GitBox
bbovenzi commented on a change in pull request #16931: URL: https://github.com/apache/airflow/pull/16931#discussion_r729075558 ## File path: airflow/www/templates/airflow/dag.html ## @@ -91,11 +91,15 @@ Schedule: {{ dag.schedule_interval }} + {%

[GitHub] [airflow] 30blay commented on pull request #16338: SFTP hook to prefer the SSH paramiko key over the key file path

2021-10-14 Thread GitBox
30blay commented on pull request #16338: URL: https://github.com/apache/airflow/pull/16338#issuecomment-943440885 @hx-markterry I made this [PR](https://github.com/hx-markterry/airflow/pull/1) on your fork a week ago, to move this forward. Have you seen it? -- This is an automated

[GitHub] [airflow] github-actions[bot] commented on pull request #18938: Jinja templates should be rendered in dict keys

2021-10-14 Thread GitBox
github-actions[bot] commented on pull request #18938: URL: https://github.com/apache/airflow/pull/18938#issuecomment-943435358 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

[GitHub] [airflow] potiuk commented on pull request #18938: Jinja templates should be rendered in dict keys

2021-10-14 Thread GitBox
potiuk commented on pull request #18938: URL: https://github.com/apache/airflow/pull/18938#issuecomment-943435474 Works for me then - some static checks need to be fixed though (and I think we should hold-off with merging that until we release 2.2.1 as there are potential conflicts I

[GitHub] [airflow] pgagnon opened a new pull request #18987: Document `hdfs_namenode_principal` for HDFS connections

2021-10-14 Thread GitBox
pgagnon opened a new pull request #18987: URL: https://github.com/apache/airflow/pull/18987 Documents `hdfs_namenode_principal` for HDFS connections which is supported by `HDFSHook`. --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] github-actions[bot] commented on pull request #18986: Upgrade old DAG/task param format when deserializing from the DB

2021-10-14 Thread GitBox
github-actions[bot] commented on pull request #18986: URL: https://github.com/apache/airflow/pull/18986#issuecomment-943427471 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

[GitHub] [airflow] ashb opened a new pull request #18986: Upgrade old DAG/task param format when deserializing from the DB

2021-10-14 Thread GitBox
ashb opened a new pull request #18986: URL: https://github.com/apache/airflow/pull/18986 Reported in https://github.com/apache/airflow/issues/18909#issuecomment-942362731 (doesn't close that issue, there are two points there) -- This is an automated message from the Apache Git

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

2021-10-14 Thread GitBox
josh-fell commented on issue #9415: URL: https://github.com/apache/airflow/issues/9415#issuecomment-943422016 I'll scan the current documentation and see where updates can be made re: TaskFlow API that doesn't already exist. If anyone has any particular pointers though feel free to shoot

[GitHub] [airflow] github-actions[bot] commented on pull request #18964: Add pre-commit hook for common misspelling check in files

2021-10-14 Thread GitBox
github-actions[bot] commented on pull request #18964: URL: https://github.com/apache/airflow/pull/18964#issuecomment-943421771 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

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

2021-10-14 Thread GitBox
josh-fell commented on issue #9415: URL: https://github.com/apache/airflow/issues/9415#issuecomment-943420848 Not sure why #18278 (Updating miscellaneous provider DAGs to use TaskFlow API where applicable) didn't appear as a mention for this issue so adding it here. -- This is an

[GitHub] [airflow] potiuk commented on a change in pull request #18964: Add pre-commit hook for common misspelling check in files

2021-10-14 Thread GitBox
potiuk commented on a change in pull request #18964: URL: https://github.com/apache/airflow/pull/18964#discussion_r729049203 ## File path: airflow/providers/microsoft/azure/operators/wasb_delete_blob.py ## @@ -65,7 +65,7 @@ def __init__( self.ignore_if_missing =

[airflow] branch main updated (d5aebf1 -> 1edcd42)

2021-10-14 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from d5aebf1 Align the default version with Facebook business SDK (#18883) add 1edcd42 Sentry before send

[GitHub] [airflow] mik-laj merged pull request #18980: Sentry before send fallback

2021-10-14 Thread GitBox
mik-laj merged pull request #18980: URL: https://github.com/apache/airflow/pull/18980 -- 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:

[GitHub] [airflow] mik-laj closed issue #18963: Missing value for new before_send config causes airflow.exceptions.AirflowConfigException: section/key [sentry/before_send] not found in config

2021-10-14 Thread GitBox
mik-laj closed issue #18963: URL: https://github.com/apache/airflow/issues/18963 -- 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:

[GitHub] [airflow] potiuk commented on pull request #18975: Row lock TI query in SchedulerJob._process_executor_events

2021-10-14 Thread GitBox
potiuk commented on pull request #18975: URL: https://github.com/apache/airflow/pull/18975#issuecomment-943415425 Looks green! BTW. We have LOT MORE GREEN nowadays after all those small flaky test fixes :D. -- This is an automated message from the Apache Git Service. To respond to the

[airflow] branch main updated (0a82a42 -> d5aebf1)

2021-10-14 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 0a82a42 Fix --github-image-id flag for Breeze (#18882) (#18946) add d5aebf1 Align the default version with

[GitHub] [airflow] potiuk merged pull request #18883: Align the default version with Facebook business SDK

2021-10-14 Thread GitBox
potiuk merged pull request #18883: URL: https://github.com/apache/airflow/pull/18883 -- 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:

[GitHub] [airflow] potiuk closed issue #18845: Facebook Ads Provider uses a deprecated version of the API

2021-10-14 Thread GitBox
potiuk closed issue #18845: URL: https://github.com/apache/airflow/issues/18845 -- 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:

[airflow] branch potiuk-patch-1 updated (9b30b90 -> 1db1958)

2021-10-14 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch potiuk-patch-1 in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9b30b90 Update entrypoint.rst add 1db1958 Update docs/docker-stack/entrypoint.rst No new revisions

[GitHub] [airflow] potiuk commented on a change in pull request #18915: Add custom entrypoint documentation

2021-10-14 Thread GitBox
potiuk commented on a change in pull request #18915: URL: https://github.com/apache/airflow/pull/18915#discussion_r729040844 ## File path: docs/docker-stack/entrypoint.rst ## @@ -194,6 +194,69 @@ If there are any other arguments - they are simply passed to the "airflow" comma

[GitHub] [airflow] potiuk commented on a change in pull request #18915: Add custom entrypoint documentation

2021-10-14 Thread GitBox
potiuk commented on a change in pull request #18915: URL: https://github.com/apache/airflow/pull/18915#discussion_r729040844 ## File path: docs/docker-stack/entrypoint.rst ## @@ -194,6 +194,69 @@ If there are any other arguments - they are simply passed to the "airflow" comma

[airflow] branch main updated (8e19ed6 -> 0a82a42)

2021-10-14 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 8e19ed6 Add description about `not a directory` errors in WSL2 (#18976) add 0a82a42 Fix --github-image-id

[GitHub] [airflow] potiuk merged pull request #18946: Fix --github-image-id flag for Breeze (#18882)

2021-10-14 Thread GitBox
potiuk merged pull request #18946: URL: https://github.com/apache/airflow/pull/18946 -- 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:

[GitHub] [airflow] potiuk commented on pull request #18974: Allows to disable built-in secret variables individually in chart

2021-10-14 Thread GitBox
potiuk commented on pull request #18974: URL: https://github.com/apache/airflow/pull/18974#issuecomment-943409814 Just FYI: I am going for holidays tomorrow @jedcunningham @kaxil and likely won't be too active for the next 2 weeks (I hope I won't have to at least). But if a new release of

[GitHub] [airflow] github-actions[bot] commented on pull request #18946: Fix --github-image-id flag for Breeze (#18882)

2021-10-14 Thread GitBox
github-actions[bot] commented on pull request #18946: URL: https://github.com/apache/airflow/pull/18946#issuecomment-943408624 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

[GitHub] [airflow] potiuk commented on pull request #18949: Fix very rare test_fast_follow failure.

2021-10-14 Thread GitBox
potiuk commented on pull request #18949: URL: https://github.com/apache/airflow/pull/18949#issuecomment-943407022 Converted it into draft for now (going for holidays for next 2 weeks, so won't have time to do more digging). This is certainly not crucial, and something that happens

[GitHub] [airflow] dstandish commented on pull request #18800: Make the salesforce connection initiation more robust

2021-10-14 Thread GitBox
dstandish commented on pull request #18800: URL: https://github.com/apache/airflow/pull/18800#issuecomment-943406004 The workaround is add all the keys to extra as None But still nice I think to make the hook tolerant of omitted keys in extra. -- This is an automated message

[airflow] branch main updated (eb2c7af -> 8e19ed6)

2021-10-14 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 eb2c7af Changes related to PR #16634 (#18644) add 8e19ed6 Add description about `not a directory` errors in

[GitHub] [airflow] potiuk merged pull request #18976: Add description about `not a directory` errors in WSL2

2021-10-14 Thread GitBox
potiuk merged pull request #18976: URL: https://github.com/apache/airflow/pull/18976 -- 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:

[GitHub] [airflow] Ayushsunny commented on a change in pull request #18610: fixed default value of execution_timeout in base operator

2021-10-14 Thread GitBox
Ayushsunny commented on a change in pull request #18610: URL: https://github.com/apache/airflow/pull/18610#discussion_r729032457 ## File path: airflow/models/baseoperator.py ## @@ -502,7 +502,7 @@ def __init__( pool: Optional[str] = None, pool_slots: int = 1,

[GitHub] [airflow] github-actions[bot] commented on pull request #18985: Restoring additional context in Slack operators how-to guide

2021-10-14 Thread GitBox
github-actions[bot] commented on pull request #18985: URL: https://github.com/apache/airflow/pull/18985#issuecomment-943402073 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

[GitHub] [airflow] josh-fell opened a new pull request #18985: Restoring additional context in Slack operators how-to guide

2021-10-14 Thread GitBox
josh-fell opened a new pull request #18985: URL: https://github.com/apache/airflow/pull/18985 A recent update to the Slack example DAG removed some context of using operators that users may find useful. Some of the args were moved to `default_args` to simplify authoring of the DAG but

[GitHub] [airflow] uranusjr commented on a change in pull request #16931: #16692 show cron description in UI

2021-10-14 Thread GitBox
uranusjr commented on a change in pull request #16931: URL: https://github.com/apache/airflow/pull/16931#discussion_r729024231 ## File path: airflow/timetables/interval.py ## @@ -221,6 +223,20 @@ def infer_manual_data_interval(self, *, run_after: DateTime) -> DataInterval:

[GitHub] [airflow] uranusjr commented on a change in pull request #16931: #16692 show cron description in UI

2021-10-14 Thread GitBox
uranusjr commented on a change in pull request #16931: URL: https://github.com/apache/airflow/pull/16931#discussion_r729024495 ## File path: airflow/timetables/base.py ## @@ -94,6 +95,15 @@ def logical_date(self) -> DateTime: class Timetable(Protocol): """Protocol that

[GitHub] [airflow] uranusjr commented on a change in pull request #16931: #16692 show cron description in UI

2021-10-14 Thread GitBox
uranusjr commented on a change in pull request #16931: URL: https://github.com/apache/airflow/pull/16931#discussion_r729024231 ## File path: airflow/timetables/interval.py ## @@ -221,6 +223,20 @@ def infer_manual_data_interval(self, *, run_after: DateTime) -> DataInterval:

[GitHub] [airflow] uranusjr commented on a change in pull request #16931: #16692 show cron description in UI

2021-10-14 Thread GitBox
uranusjr commented on a change in pull request #16931: URL: https://github.com/apache/airflow/pull/16931#discussion_r729022127 ## File path: airflow/models/dag.py ## @@ -2521,6 +2521,10 @@ def __init__(self, concurrency=None, **kwargs): def __repr__(self): return

[GitHub] [airflow] uranusjr commented on a change in pull request #16931: #16692 show cron description in UI

2021-10-14 Thread GitBox
uranusjr commented on a change in pull request #16931: URL: https://github.com/apache/airflow/pull/16931#discussion_r729020028 ## File path: airflow/timetables/base.py ## @@ -94,6 +95,15 @@ def logical_date(self) -> DateTime: class Timetable(Protocol): """Protocol that

[GitHub] [airflow] github-actions[bot] commented on pull request #18883: Align the default version with Facebook business SDK

2021-10-14 Thread GitBox
github-actions[bot] commented on pull request #18883: URL: https://github.com/apache/airflow/pull/18883#issuecomment-943388476 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

[GitHub] [airflow] potiuk commented on pull request #18915: Add custom entrypoint documentation

2021-10-14 Thread GitBox
potiuk commented on pull request #18915: URL: https://github.com/apache/airflow/pull/18915#issuecomment-943387096 Cool. Now I need just an approve from another committer :). Anyone? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] yiga2 commented on pull request #18915: Add custom entrypoint documentation

2021-10-14 Thread GitBox
yiga2 commented on pull request #18915: URL: https://github.com/apache/airflow/pull/18915#issuecomment-943383271 @potiuk great ! Thanks so much for the detailed documentation. I am all set ! Btw, your PR pretty much closes https://github.com/apache/airflow/issues/14346 in my mind. --

[GitHub] [airflow] potiuk commented on pull request #18956: Fix XCom.delete error in Airflow 2.2.0

2021-10-14 Thread GitBox
potiuk commented on pull request #18956: URL: https://github.com/apache/airflow/pull/18956#issuecomment-943380914 Cool! Great to confirm that - I thought so, but I was not entirely sure :) -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk commented on issue #18835: Status of testing Amzon Provider 2.3.0rc2 that was prepared on October 08, 2021

2021-10-14 Thread GitBox
potiuk commented on issue #18835: URL: https://github.com/apache/airflow/issues/18835#issuecomment-943376572 > sorry i'm late (aws hasn't had an exception until today joy) #18733 works fine *JUST* in time then! -- This is an automated message from the Apache Git Service. To

[GitHub] [airflow] pateash commented on pull request #18984: #16692 show cron schedule description in UI v3

2021-10-14 Thread GitBox
pateash commented on pull request #18984: URL: https://github.com/apache/airflow/pull/18984#issuecomment-943376067 Duplicate -- 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

[GitHub] [airflow] pateash closed pull request #18984: #16692 show cron schedule description in UI v3

2021-10-14 Thread GitBox
pateash closed pull request #18984: URL: https://github.com/apache/airflow/pull/18984 -- 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:

[GitHub] [airflow] pateash opened a new pull request #18984: #16692 show cron schedule description in UI v3

2021-10-14 Thread GitBox
pateash opened a new pull request #18984: URL: https://github.com/apache/airflow/pull/18984 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more

[GitHub] [airflow] github-actions[bot] commented on pull request #18953: Try to move "dangling" rows in upgradedb

2021-10-14 Thread GitBox
github-actions[bot] commented on pull request #18953: URL: https://github.com/apache/airflow/pull/18953#issuecomment-943375300 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

  1   2   3   >