[airflow] branch main updated (28e285e -> 609620a)

2021-06-18 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 28e285e Updating the DAG docstring to include render_template_as_native_obj (#16534) add 609620a Fetch

[GitHub] [airflow] mik-laj merged pull request #16535: Fetch Helm Chart inventory from remote cache

2021-06-18 Thread GitBox
mik-laj merged pull request #16535: URL: https://github.com/apache/airflow/pull/16535 -- 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. For queries about this service,

[GitHub] [airflow] mik-laj closed issue #16533: Documentation building fails if helm-chart is not being built

2021-06-18 Thread GitBox
mik-laj closed issue #16533: URL: https://github.com/apache/airflow/issues/16533 -- 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. For queries about this service, please

[airflow] branch main updated (8a090dc -> 28e285e)

2021-06-18 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 8a090dc Stop showing output of a parallel job once it has finished. (#16528) add 28e285e Updating the DAG

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #16534: Updating the DAG docstring to include `render_template_as_native_obj`

2021-06-18 Thread GitBox
boring-cyborg[bot] commented on pull request #16534: URL: https://github.com/apache/airflow/pull/16534#issuecomment-864334286 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] potiuk merged pull request #16534: Updating the DAG docstring to include `render_template_as_native_obj`

2021-06-18 Thread GitBox
potiuk merged pull request #16534: URL: https://github.com/apache/airflow/pull/16534 -- 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. For queries about this service,

[GitHub] [airflow] potiuk commented on pull request #16536: Adds hive exclusion for Python 3.9

2021-06-18 Thread GitBox
potiuk commented on pull request #16536: URL: https://github.com/apache/airflow/pull/16536#issuecomment-864332742 CC: @r-richmond -- 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] github-actions[bot] commented on pull request #16534: Updating the DAG docstring to include `render_template_as_native_obj`

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16534: URL: https://github.com/apache/airflow/pull/16534#issuecomment-864330175 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 closed pull request #16536: Adds hive exclusion for Python 3.9

2021-06-18 Thread GitBox
potiuk closed pull request #16536: URL: https://github.com/apache/airflow/pull/16536 -- 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. For queries about this service,

[GitHub] [airflow] github-actions[bot] commented on pull request #16536: Adds hive exclusion for Python 3.9

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16536: URL: https://github.com/apache/airflow/pull/16536#issuecomment-864327082 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] github-actions[bot] closed issue #12396: Scheduler identifies a zombie AFTER a task has concluded successfully --> subsequently sets the task status to failed

2021-06-18 Thread GitBox
github-actions[bot] closed issue #12396: URL: https://github.com/apache/airflow/issues/12396 -- 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. For queries about this

[GitHub] [airflow] github-actions[bot] commented on issue #12396: Scheduler identifies a zombie AFTER a task has concluded successfully --> subsequently sets the task status to failed

2021-06-18 Thread GitBox
github-actions[bot] commented on issue #12396: URL: https://github.com/apache/airflow/issues/12396#issuecomment-864325744 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] github-actions[bot] commented on issue #10080: Kubernetes Executor: Logs in UI doesn't work with Elasticsearch stdout config

2021-06-18 Thread GitBox
github-actions[bot] commented on issue #10080: URL: https://github.com/apache/airflow/issues/10080#issuecomment-864325761 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] github-actions[bot] closed issue #10080: Kubernetes Executor: Logs in UI doesn't work with Elasticsearch stdout config

2021-06-18 Thread GitBox
github-actions[bot] closed issue #10080: URL: https://github.com/apache/airflow/issues/10080 -- 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. For queries about this

[GitHub] [airflow] github-actions[bot] commented on issue #15709: Cannot delete DAG

2021-06-18 Thread GitBox
github-actions[bot] commented on issue #15709: URL: https://github.com/apache/airflow/issues/15709#issuecomment-864325707 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] github-actions[bot] closed issue #15709: Cannot delete DAG

2021-06-18 Thread GitBox
github-actions[bot] closed issue #15709: URL: https://github.com/apache/airflow/issues/15709 -- 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. For queries about this

[GitHub] [airflow] github-actions[bot] commented on pull request #16535: Fetch Helm Chart inventory from remote cache

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16535: URL: https://github.com/apache/airflow/pull/16535#issuecomment-864324479 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] potiuk commented on pull request #16535: Fetch Helm Chart inventory from remote cache

2021-06-18 Thread GitBox
potiuk commented on pull request #16535: URL: https://github.com/apache/airflow/pull/16535#issuecomment-864324467 Needs rebase :) -- 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] potiuk opened a new pull request #16536: Adds hive exclusion for Python 3.9

2021-06-18 Thread GitBox
potiuk opened a new pull request #16536: URL: https://github.com/apache/airflow/pull/16536 This is the first step to add Python 3.9 support to Airflow. Hive should be excluded in this version because it requires sasl library which for now does not support Python 3.9. Until the

[GitHub] [airflow] mik-laj opened a new pull request #16535: Fetch Helm Chart inventory from remote cache

2021-06-18 Thread GitBox
mik-laj opened a new pull request #16535: URL: https://github.com/apache/airflow/pull/16535 Close: https://github.com/apache/airflow/issues/16533 We have a mechanism that retrieves missing inventories from the remote cache if they are not present locally. This helps to prevent the

[airflow] branch constraints-main updated: Updated 3.9 constraints to non-conflicting version

2021-06-18 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk 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 0369ea5 Updated 3.9

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #16534: Updating the DAG docstring to include `render_template_as_native_obj`

2021-06-18 Thread GitBox
boring-cyborg[bot] commented on pull request #16534: URL: https://github.com/apache/airflow/pull/16534#issuecomment-864289266 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] josh-fell opened a new pull request #16534: Updating the DAG docstring to include `render_template_as_native_obj`

2021-06-18 Thread GitBox
josh-fell opened a new pull request #16534: URL: https://github.com/apache/airflow/pull/16534 The current docstring for the `airflow.models.dag.DAG` class is missing the `render_template_as_native_obj` parameter and therefore missing in the public documentation as well. This PR updates

[GitHub] [airflow] gfelot closed issue #16529: Cannot split Jinja str and render_template_as_native_obj doesn't exist

2021-06-18 Thread GitBox
gfelot closed issue #16529: URL: https://github.com/apache/airflow/issues/16529 -- 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. For queries about this service, please

[GitHub] [airflow] gfelot commented on issue #16529: Cannot split Jinja str and render_template_as_native_obj doesn't exist

2021-06-18 Thread GitBox
gfelot commented on issue #16529: URL: https://github.com/apache/airflow/issues/16529#issuecomment-864287465 For the doc issue. My bad I didn't check the version you are right. Good if I help out on something. When I try to fix the doc with the builtin button "Suggest a change on this

[GitHub] [airflow] LukeHong opened a new issue #16520: DbApiHook.get_uri() doesn't follow PostgresHook schema argument

2021-06-18 Thread GitBox
LukeHong opened a new issue #16520: URL: https://github.com/apache/airflow/issues/16520 **Apache Airflow version**: **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): **Environment**: - **Cloud provider or hardware configuration**: -

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #16494: Exclude Yarn.lock to be built into python wheel

2021-06-18 Thread GitBox
boring-cyborg[bot] commented on pull request #16494: URL: https://github.com/apache/airflow/pull/16494#issuecomment-863598713 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] potiuk opened a new issue #16533: Documentation building fails if helm-chart is not being built

2021-06-18 Thread GitBox
potiuk opened a new issue #16533: URL: https://github.com/apache/airflow/issues/16533 When you've never built `helm-chart` documentation package locally. the intersphinx repository is missing for it and it cannot be downloaded as the helm-chart package is never published as package (I

[GitHub] [airflow] pgagnon closed pull request #16512: Add extra_headers argument to LivyHook and LivyOperator

2021-06-18 Thread GitBox
pgagnon closed pull request #16512: URL: https://github.com/apache/airflow/pull/16512 -- 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. For queries about this service,

[GitHub] [airflow] github-actions[bot] commented on pull request #16515: Support extraContainers and extraVolumes in flower

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16515: URL: https://github.com/apache/airflow/pull/16515#issuecomment-863619795 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] JPonte commented on a change in pull request #16264: Add flag to delete local logs after upload

2021-06-18 Thread GitBox
JPonte commented on a change in pull request #16264: URL: https://github.com/apache/airflow/pull/16264#discussion_r654192109 ## File path: airflow/providers/microsoft/azure/log/wasb_task_handler.py ## @@ -100,8 +100,8 @@ def close(self) -> None: with

[GitHub] [airflow] josh-fell commented on issue #16529: Cannot split Jinja str and render_template_as_native_obj doesn't exist

2021-06-18 Thread GitBox
josh-fell commented on issue #16529: URL: https://github.com/apache/airflow/issues/16529#issuecomment-864280192 Hi @gfelot, Addressing the errors you are observing: - The documentation you are referring to is for version 2.1 (current stable) while you are running 2.0.2 (of which

[GitHub] [airflow] potiuk commented on pull request #16084: Added new pipeline example for the tutorial docs (Issue #11208)

2021-06-18 Thread GitBox
potiuk commented on pull request #16084: URL: https://github.com/apache/airflow/pull/16084#issuecomment-864275288 -- 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. For

[GitHub] [airflow] potiuk closed issue #16527: Request: v1 release that allows urllib3 v1.26.5

2021-06-18 Thread GitBox
potiuk closed issue #16527: URL: https://github.com/apache/airflow/issues/16527 -- 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. For queries about this service, please

[GitHub] [airflow] kaxil opened a new pull request #16513: Bugfix: Allow clearing tasks with just ``dag_id`` and empty ``subdir``

2021-06-18 Thread GitBox
kaxil opened a new pull request #16513: URL: https://github.com/apache/airflow/pull/16513 Currently if we run the following command: ``` airflow tasks clear example_bash_operator --subdir "" ``` we get the following error: ``` Traceback (most recent call

[GitHub] [airflow] o-nikolas commented on a change in pull request #16264: Add flag to delete local logs after upload

2021-06-18 Thread GitBox
o-nikolas commented on a change in pull request #16264: URL: https://github.com/apache/airflow/pull/16264#discussion_r653908548 ## File path: airflow/providers/microsoft/azure/log/wasb_task_handler.py ## @@ -100,8 +100,8 @@ def close(self) -> None: with

[GitHub] [airflow] malthe commented on pull request #16314: Add support for non-RSA type client host key

2021-06-18 Thread GitBox
malthe commented on pull request #16314: URL: https://github.com/apache/airflow/pull/16314#issuecomment-863979786 -- 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. For

[GitHub] [airflow] github-actions[bot] closed issue #9928: Statsd metrics

2021-06-18 Thread GitBox
github-actions[bot] closed issue #9928: URL: https://github.com/apache/airflow/issues/9928 -- 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. For queries about this

[GitHub] [airflow] mik-laj commented on issue #16519: [Feature] Google groups - RBAC role mapping

2021-06-18 Thread GitBox
mik-laj commented on issue #16519: URL: https://github.com/apache/airflow/issues/16519#issuecomment-863908899 -- 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. For queries

[GitHub] [airflow] damon09273 edited a comment on issue #14896: Error: 'daemonic processes are not allowed to have children', after upgradeing to airflow:2.0.1

2021-06-18 Thread GitBox
damon09273 edited a comment on issue #14896: URL: https://github.com/apache/airflow/issues/14896#issuecomment-863858852 -- 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. For

[GitHub] [airflow] jedcunningham edited a comment on pull request #16521: Implement get_uri() in PostgresHook

2021-06-18 Thread GitBox
jedcunningham edited a comment on pull request #16521: URL: https://github.com/apache/airflow/pull/16521#issuecomment-864170714 -- 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] ManiBharataraju opened a new pull request #16508: Fixes #16507 Subdags are now created with pause status based on their parent.

2021-06-18 Thread GitBox
ManiBharataraju opened a new pull request #16508: URL: https://github.com/apache/airflow/pull/16508 closes: #16507 In this PR I'm checking the pause status of the parent and assigning it to the subdag if they are newly added. -- This is an automated message from the Apache

[GitHub] [airflow] potiuk commented on pull request #16314: Add support for non-RSA type client host key

2021-06-18 Thread GitBox
potiuk commented on pull request #16314: URL: https://github.com/apache/airflow/pull/16314#issuecomment-863988505 -- 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. For

[GitHub] [airflow] potiuk merged pull request #15850: Add AWS DMS replication task operators

2021-06-18 Thread GitBox
potiuk merged pull request #15850: URL: https://github.com/apache/airflow/pull/15850 -- 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. For queries about this service,

[GitHub] [airflow] ejstembler commented on pull request #16492: Added metadata support to GCSHook.upload. Closes #16487

2021-06-18 Thread GitBox
ejstembler commented on pull request #16492: URL: https://github.com/apache/airflow/pull/16492#issuecomment-863735522 -- 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. For

[GitHub] [airflow] github-actions[bot] commented on pull request #16523: Update docs on setting up SMTP

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16523: URL: https://github.com/apache/airflow/pull/16523#issuecomment-863929903 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] potiuk merged pull request #16495: Add docs index to README.md

2021-06-18 Thread GitBox
potiuk merged pull request #16495: URL: https://github.com/apache/airflow/pull/16495 -- 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. For queries about this service,

[GitHub] [airflow] rng-spokeo opened a new issue #16519: Google groups - RBAC role mapping

2021-06-18 Thread GitBox
rng-spokeo opened a new issue #16519: URL: https://github.com/apache/airflow/issues/16519 **Description** With Google OAuth authentication enabled, users able to login without pre-creating user account in Airflow. However, AUTH_ROLES_MAPPING won't work with Google provider because

[GitHub] [airflow] github-actions[bot] commented on pull request #16512: Add extra_headers argument to LivyHook and LivyOperator

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16512: URL: https://github.com/apache/airflow/pull/16512#issuecomment-863577153 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] github-actions[bot] commented on pull request #16516: Allow null value for operator field in task_instance schema(REST API)

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16516: URL: https://github.com/apache/airflow/pull/16516#issuecomment-863619523 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] kaxil merged pull request #16494: Exclude Yarn.lock to be built into python wheel

2021-06-18 Thread GitBox
kaxil merged pull request #16494: URL: https://github.com/apache/airflow/pull/16494 -- 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. For queries about this service,

[GitHub] [airflow] ciancolo commented on a change in pull request #16365: Allow disable SSL for TableauHook

2021-06-18 Thread GitBox
ciancolo commented on a change in pull request #16365: URL: https://github.com/apache/airflow/pull/16365#discussion_r654549147 ## File path: airflow/providers/tableau/hooks/tableau.py ## @@ -61,7 +61,9 @@ def __init__(self, site_id: Optional[str] = None, tableau_conn_id: str

[GitHub] [airflow] ciancolo commented on pull request #16350: Fix Minor Bugs in Apache Sqoop Hook and Operator

2021-06-18 Thread GitBox
ciancolo commented on pull request #16350: URL: https://github.com/apache/airflow/pull/16350#issuecomment-863827006 @potiuk Sure! -- 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] ashb merged pull request #16509: Fix broken build image build after removing Py2.7/3.5 builds

2021-06-18 Thread GitBox
ashb merged pull request #16509: URL: https://github.com/apache/airflow/pull/16509 -- 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. For queries about this service, please

[GitHub] [airflow] jedcunningham commented on pull request #16518: Revert "Exclude Yarn.lock to be built into python wheel (#16494)"

2021-06-18 Thread GitBox
jedcunningham commented on pull request #16518: URL: https://github.com/apache/airflow/pull/16518#issuecomment-863629008 CC: @ernest-kr @kaxil -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] kaxil opened a new pull request #16511: Docs: Fix API verb from ``POST`` to ``PATCH``

2021-06-18 Thread GitBox
kaxil opened a new pull request #16511: URL: https://github.com/apache/airflow/pull/16511 Updating the dag, i.e. pausing/unpausing is ``PATCH``. --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] SamWheating commented on issue #16303: Replace `execution_date` in `TI.generate_command` to send run_id instead

2021-06-18 Thread GitBox
SamWheating commented on issue #16303: URL: https://github.com/apache/airflow/issues/16303#issuecomment-864181786 I think that this depends on https://github.com/apache/airflow/issues/16303 as well, since otherwise the `run_id` isn't directly available within a TaskInstance (and doing

[GitHub] [airflow-site] potiuk opened a new pull request #435: Add Provider's docs for June release

2021-06-18 Thread GitBox
potiuk opened a new pull request #435: URL: https://github.com/apache/airflow-site/pull/435 -- 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. For queries about this

[GitHub] [airflow] potiuk commented on pull request #16498: gitpodify Apache Airflow - online development workspace

2021-06-18 Thread GitBox
potiuk commented on pull request #16498: URL: https://github.com/apache/airflow/pull/16498#issuecomment-863540890 Would love to - but next week only. Having some private/family interruptions this week and do bare minimum. Hopefully next week will be better (looks like) -- This is an

[GitHub] [airflow] huozhanfeng edited a comment on pull request #15537: Fix on_failure_callback when task receive SIGKILL

2021-06-18 Thread GitBox
huozhanfeng edited a comment on pull request #15537: URL: https://github.com/apache/airflow/pull/15537#issuecomment-864013121 > > The PR will cause dags that use hive operations to fail because they can't get the right PID. > > the hive task process info like these > > 1、airflow

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #16469: Update Watchtower version to 1.0.6

2021-06-18 Thread GitBox
boring-cyborg[bot] commented on pull request #16469: URL: https://github.com/apache/airflow/pull/16469#issuecomment-863827298 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] michalc opened a new issue #16527: Request: v1 release that allows urllib3 1.26.5

2021-06-18 Thread GitBox
michalc opened a new issue #16527: URL: https://github.com/apache/airflow/issues/16527 **Description** We're still on Airflow v1.10.15 (moving to v2, but it's taking a while). However, Airflow v1 depends on requests < 2.24, i.e. the latest version that can be installed is 2.23.0,

[GitHub] [airflow] github-actions[bot] commented on issue #15485: Support for View/Export third party artifacts from Airflow

2021-06-18 Thread GitBox
github-actions[bot] commented on issue #15485: URL: https://github.com/apache/airflow/issues/15485#issuecomment-863636513 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] github-actions[bot] closed issue #10193: Scheduler get killed when the DAG doesn't parse correctly in Kubernetes Worker

2021-06-18 Thread GitBox
github-actions[bot] closed issue #10193: URL: https://github.com/apache/airflow/issues/10193 -- 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. For queries about this

[GitHub] [airflow] potiuk commented on pull request #16495: Add docs index to README.md

2021-06-18 Thread GitBox
potiuk commented on pull request #16495: URL: https://github.com/apache/airflow/pull/16495#issuecomment-863584120 -- 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. For

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #16521: Implement get_uri() in PostgresHook

2021-06-18 Thread GitBox
boring-cyborg[bot] commented on pull request #16521: URL: https://github.com/apache/airflow/pull/16521#issuecomment-863738661 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] mik-laj commented on issue #16485: auth for worker log serve app

2021-06-18 Thread GitBox
mik-laj commented on issue #16485: URL: https://github.com/apache/airflow/issues/16485#issuecomment-863923111 We can also use the `itsdangerous` library, which is already used to generate tokens in API

[GitHub] [airflow] huozhanfeng commented on pull request #15537: Fix on_failure_callback when task receive SIGKILL

2021-06-18 Thread GitBox
huozhanfeng commented on pull request #15537: URL: https://github.com/apache/airflow/pull/15537#issuecomment-864013121 > > The PR will cause dags that use hive operations to fail because they can't get the right PID. > > the hive task process info like these > > 1、airflow 20391 20170

[GitHub] [airflow] vikramkoka commented on issue #16529: Cannot split Jinja str and render_template_as_native_obj doesn't exist

2021-06-18 Thread GitBox
vikramkoka commented on issue #16529: URL: https://github.com/apache/airflow/issues/16529#issuecomment-864188566 > @vikramkoka Would you mind if I took this one? @josh-fell Go for it! -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] kaxil commented on a change in pull request #16514: Fix package version in ``LICENSE`` file

2021-06-18 Thread GitBox
kaxil commented on a change in pull request #16514: URL: https://github.com/apache/airflow/pull/16514#discussion_r654027316 ## File path: LICENSE ## @@ -229,23 +229,22 @@ MIT licenses The following components are provided under the MIT License. See project link for details.

[GitHub] [airflow] kipkoan commented on issue #15580: KubernetesExecutor not deleting worker pods on completion.

2021-06-18 Thread GitBox
kipkoan commented on issue #15580: URL: https://github.com/apache/airflow/issues/15580#issuecomment-864126661 @rpfernandezjr did you ever find a solution to this? I'm seeing the same thing with the 2.1.0 docker image. -- This is an automated message from the Apache Git Service. To

[GitHub] [airflow] malthe edited a comment on pull request #16314: Add support for non-RSA type client host key

2021-06-18 Thread GitBox
malthe edited a comment on pull request #16314: URL: https://github.com/apache/airflow/pull/16314#issuecomment-864016126 @potiuk in this case I think there is no particular need for a refactoring; the code is alright as it is – if I was going to suggest changing anything, it would be the

[GitHub] [airflow] ephraimbuddy commented on issue #14541: Airflow upgrade_check gives false positives on DbApi functions implementations check

2021-06-18 Thread GitBox
ephraimbuddy commented on issue #14541: URL: https://github.com/apache/airflow/issues/14541#issuecomment-863890164 > any luck with this?  What version of the upgrade-check are you using because I believe this was resolved in https://github.com/apache/airflow/pull/14344 -- This

[GitHub] [airflow] potiuk merged pull request #16510: Don't fail the build if we cant run `apt clean`

2021-06-18 Thread GitBox
potiuk merged pull request #16510: URL: https://github.com/apache/airflow/pull/16510 -- 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. For queries about this service,

[GitHub] [airflow] ashb commented on pull request #15537: Fix on_failure_callback when task receive SIGKILL

2021-06-18 Thread GitBox
ashb commented on pull request #15537: URL: https://github.com/apache/airflow/pull/15537#issuecomment-864111462 In that example I don't see pid 30960 anywhere -- do you know what process it was it? I suspect this might be to do with impersonation, rather than anything hive in

[GitHub] [airflow] kaxil merged pull request #16511: Docs: Fix API verb from ``POST`` to ``PATCH``

2021-06-18 Thread GitBox
kaxil merged pull request #16511: URL: https://github.com/apache/airflow/pull/16511 -- 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. For queries about this service,

[GitHub] [airflow] github-actions[bot] commented on pull request #16514: Fix package version in ``LICENSE`` file

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16514: URL: https://github.com/apache/airflow/pull/16514#issuecomment-863761037 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] uranusjr commented on a change in pull request #16521: Implement get_uri() in PostgresHook

2021-06-18 Thread GitBox
uranusjr commented on a change in pull request #16521: URL: https://github.com/apache/airflow/pull/16521#discussion_r654131855 ## File path: airflow/providers/postgres/hooks/postgres.py ## @@ -114,6 +115,25 @@ def get_conn(self) -> connection: self.conn =

[GitHub] [airflow] jhtimmins commented on pull request #2460: [AIRFLOW-1424] make the next execution date of DAGs visible

2021-06-18 Thread GitBox
jhtimmins commented on pull request #2460: URL: https://github.com/apache/airflow/pull/2460#issuecomment-863653701 Let's target 2.2 with this. @ultrabug can you handle the merge conflicts? -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] github-actions[bot] closed issue #9967: task instance is scheduled repeatedly while dag file is pretty large

2021-06-18 Thread GitBox
github-actions[bot] closed issue #9967: URL: https://github.com/apache/airflow/issues/9967 -- 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. For queries about this

[GitHub] [airflow] msumit opened a new pull request #16524: Fix bug in mark TI success api

2021-06-18 Thread GitBox
msumit opened a new pull request #16524: URL: https://github.com/apache/airflow/pull/16524 Mistakenly checking for the wrong args in TI success API. Introduced in PR https://github.com/apache/airflow/pull/16233. --- **^ Add meaningful description above** Read the

[GitHub] [airflow] potiuk commented on a change in pull request #16523: Update docs on setting up SMTP

2021-06-18 Thread GitBox
potiuk commented on a change in pull request #16523: URL: https://github.com/apache/airflow/pull/16523#discussion_r654316048 ## File path: docs/apache-airflow/howto/email-config.rst ## @@ -29,8 +29,11 @@ in the ``[email]`` section. subject_template =

[GitHub] [airflow] potiuk merged pull request #16501: Prepares documentation for rc2 release of Providers

2021-06-18 Thread GitBox
potiuk merged pull request #16501: URL: https://github.com/apache/airflow/pull/16501 -- 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. For queries about this service,

[GitHub] [airflow] pgagnon commented on a change in pull request #16512: Add extra_headers argument to LivyHook and LivyOperator

2021-06-18 Thread GitBox
pgagnon commented on a change in pull request #16512: URL: https://github.com/apache/airflow/pull/16512#discussion_r653938595 ## File path: tests/providers/apache/livy/hooks/test_livy.py ## @@ -316,6 +316,20 @@ def test_invalid_uri(self): with

[GitHub] [airflow] mik-laj closed issue #16519: [Feature] Google groups - RBAC role mapping

2021-06-18 Thread GitBox
mik-laj closed issue #16519: URL: https://github.com/apache/airflow/issues/16519 -- 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. For queries about this service, please

[GitHub] [airflow] freget opened a new pull request #16522: Tree View UI for larger DAGs & more consistent spacing in Tree View

2021-06-18 Thread GitBox
freget opened a new pull request #16522: URL: https://github.com/apache/airflow/pull/16522 closes: #16238 Currently, the space available for the tree and operator names in the tree view is calculated by substracting a quite a few things from the screen width. This does not

[GitHub] [airflow] aneesh-joseph commented on pull request #16503: Clean up Mssql data directories at the end of each CI run

2021-06-18 Thread GitBox
aneesh-joseph commented on pull request #16503: URL: https://github.com/apache/airflow/pull/16503#issuecomment-863698662 oops, failed again, we would have to tweak the directory path a bit to ``` export MSSQL_DATA_VOLUME="${HOME}/tmp-mssql-volume-${TEST_TYPE}-${MSSQL_VERSION}"

[GitHub] [airflow] ashb commented on a change in pull request #16503: Clean up Mssql data directories at the end of each CI run

2021-06-18 Thread GitBox
ashb commented on a change in pull request #16503: URL: https://github.com/apache/airflow/pull/16503#discussion_r653839806 ## File path: scripts/ci/testing/ci_run_single_airflow_test_in_docker.sh ## @@ -100,6 +100,10 @@ function run_airflow_testing_in_docker() { #

[jira] [Commented] (AIRFLOW-1424) Make the next execution date of DAGs visible

2021-06-18 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-1424?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17365725#comment-17365725 ] ASF GitHub Bot commented on AIRFLOW-1424: - jhtimmins commented on pull request #2460: URL:

[GitHub] [airflow] kaxil merged pull request #16513: Bugfix: Allow clearing tasks with just ``dag_id`` and empty ``subdir``

2021-06-18 Thread GitBox
kaxil merged pull request #16513: URL: https://github.com/apache/airflow/pull/16513 -- 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. For queries about this service,

[GitHub] [airflow] argibbs edited a comment on issue #15596: Using SLAs causes DagFileProcessorManager timeouts and prevents deleted dags from being recreated

2021-06-18 Thread GitBox
argibbs edited a comment on issue #15596: URL: https://github.com/apache/airflow/issues/15596#issuecomment-863863064 Hi there, I'd be up for making a change, but I don't know what the right solution is (as my lengthy screed above hopefully made clear). I could just comment the call

[GitHub] [airflow] github-actions[bot] commented on pull request #16513: Bugfix: Allow clearing tasks with just ``dag_id`` and empty ``subdir``

2021-06-18 Thread GitBox
github-actions[bot] commented on pull request #16513: URL: https://github.com/apache/airflow/pull/16513#issuecomment-863621097 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 opened a new pull request #16501: Prepares documentation for rc2 release of Providers

2021-06-18 Thread GitBox
potiuk opened a new pull request #16501: URL: https://github.com/apache/airflow/pull/16501 * adds clear information that the provider is for 2.1+ * adds explicit dependency to apache-airflow>=2.1.0 in dependency list * adds capability of specifying additional dependencies *

[GitHub] [airflow] mik-laj commented on a change in pull request #16495: Add docs index to README.md

2021-06-18 Thread GitBox
mik-laj commented on a change in pull request #16495: URL: https://github.com/apache/airflow/pull/16495#discussion_r654266136 ## File path: README.md ## @@ -183,7 +183,7 @@ through a more complete [tutorial](https://airflow.apache.org/docs/apache-airflo For more information

[jira] [Commented] (AIRFLOW-4374) Inherit from Enum

2021-06-18 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-4374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17365721#comment-17365721 ] ASF GitHub Bot commented on AIRFLOW-4374: - andrewgodwin commented on pull request #5302: URL:

[GitHub] [airflow] turbaszek commented on a change in pull request #16268: Change Regex For Inclusive Words

2021-06-18 Thread GitBox
turbaszek commented on a change in pull request #16268: URL: https://github.com/apache/airflow/pull/16268#discussion_r654479278 ## File path: .pre-commit-config.yaml ## @@ -340,7 +340,8 @@ repos: language: pygrep name: Check for language that we do not accept

[GitHub] [airflow] andrewgodwin commented on pull request #5302: [AIRFLOW-4374] Make enum-like-classes inherit from enum

2021-06-18 Thread GitBox
andrewgodwin commented on pull request #5302: URL: https://github.com/apache/airflow/pull/5302#issuecomment-863709135 Yes, the new version of this is https://github.com/apache/airflow/pull/15285, we should probably close this? -- This is an automated message from the Apache Git Service.

[GitHub] [airflow] ashb opened a new pull request #16509: Fix broken build image build after removing Py2.7/3.5 builds

2021-06-18 Thread GitBox
ashb opened a new pull request #16509: URL: https://github.com/apache/airflow/pull/16509 --- **^ 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] kaxil merged pull request #16518: Revert "Exclude Yarn.lock to be built into python wheel (#16494)"

2021-06-18 Thread GitBox
kaxil merged pull request #16518: URL: https://github.com/apache/airflow/pull/16518 -- 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. For queries about this service,

[GitHub] [airflow] LukeHong commented on a change in pull request #16521: Implement get_uri() in PostgresHook

2021-06-18 Thread GitBox
LukeHong commented on a change in pull request #16521: URL: https://github.com/apache/airflow/pull/16521#discussion_r654156094 ## File path: airflow/providers/postgres/hooks/postgres.py ## @@ -114,6 +115,25 @@ def get_conn(self) -> connection: self.conn =

  1   2   3   >