[GitHub] [airflow] boring-cyborg[bot] commented on issue #17714: dag import error when using sys.path.append(str(Path('.').absolute().parent))

2021-08-18 Thread GitBox
boring-cyborg[bot] commented on issue #17714: URL: https://github.com/apache/airflow/issues/17714#issuecomment-901625952 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

[GitHub] [airflow] ezerkar opened a new issue #17714: dag import error when using sys.path.append(str(Path('.').absolute().parent))

2021-08-18 Thread GitBox
ezerkar opened a new issue #17714: URL: https://github.com/apache/airflow/issues/17714 **Apache Airflow version**: v2.1.2 **OS**: Ubuntu 20.04.1 LTS **Deployment**: local **What happened**: have a file in a parent directory I want to

[GitHub] [airflow] kaojunsong commented on issue #17045: "Failed to load task run error" occur in "on_failure_callback" after upgrading from 2.0.0 to 2.1.2

2021-08-18 Thread GitBox
kaojunsong commented on issue #17045: URL: https://github.com/apache/airflow/issues/17045#issuecomment-901617158 @potiuk Hi, I still can't solve this issue, could you please kindly have a look? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] yevsh edited a comment on issue #17679: Airflow not running as root when deployed to k8s via helm

2021-08-18 Thread GitBox
yevsh edited a comment on issue #17679: URL: https://github.com/apache/airflow/issues/17679#issuecomment-901615020 thanks, so I can take the latest helms? I see it's not officialy released yet. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] yevsh commented on issue #17679: Airflow not running as root when deployed to k8s via helm

2021-08-18 Thread GitBox
yevsh commented on issue #17679: URL: https://github.com/apache/airflow/issues/17679#issuecomment-901615020 thanks, so I can take the latest helms? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] msumit commented on a change in pull request #17100: WIP: Advanced Params using json-schema

2021-08-18 Thread GitBox
msumit commented on a change in pull request #17100: URL: https://github.com/apache/airflow/pull/17100#discussion_r691753057 ## File path: airflow/serialization/serialized_objects.py ## @@ -789,6 +825,27 @@ def from_dict(cls, serialized_obj: dict) -> 'SerializedDAG':

[GitHub] [airflow] github-actions[bot] commented on pull request #17711: Doc: Fix replacing Airflow version for Docker stack

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17711: URL: https://github.com/apache/airflow/pull/17711#issuecomment-901582288 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 pull request #17710: Doc: Update Upgrade to 2 docs with Airflow 1.10.x EOL dates

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17710: URL: https://github.com/apache/airflow/pull/17710#issuecomment-901577570 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] jedcunningham commented on pull request #15385: Added table to view providers in airflow ui under admin tab

2021-08-18 Thread GitBox
jedcunningham commented on pull request #15385: URL: https://github.com/apache/airflow/pull/15385#issuecomment-901574489 @ShakaibKhan, looks like `tests/www/views/test_views_base.py::test_index` is failing due to query count. Your +1 to it probably got lost during your last rebase. --

[GitHub] [airflow] jedcunningham commented on pull request #17397: Update to Celery 5

2021-08-18 Thread GitBox
jedcunningham commented on pull request #17397: URL: https://github.com/apache/airflow/pull/17397#issuecomment-901573360 @potiuk, have at it. I've not made any progress over the last few days. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] kaxil opened a new pull request #17711: Doc: Fix replacing Airflow version for Docker stack

2021-08-18 Thread GitBox
kaxil opened a new pull request #17711: URL: https://github.com/apache/airflow/pull/17711 currently it shows up as: ``` apache/airflow:|version| - the versioned Airflow image with default Python version (3.6 currently) ``` Example:

[GitHub] [airflow] kaxil merged pull request #17709: Doc: Replace deprecated param from docstrings

2021-08-18 Thread GitBox
kaxil merged pull request #17709: URL: https://github.com/apache/airflow/pull/17709 -- 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: Doc: Replace deprecated param from docstrings (#17709)

2021-08-18 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 3b41bb4 Doc: Replace deprecated param from

[airflow] branch main updated (fbf5163 -> d26b862)

2021-08-18 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 fbf5163 Add a note about XCOMs push condition for K8sPodOperator docs (#17690) add d26b862 Add optional

[GitHub] [airflow] kaxil merged pull request #17640: redshift to S3: add optional SQL parameters

2021-08-18 Thread GitBox
kaxil merged pull request #17640: URL: https://github.com/apache/airflow/pull/17640 -- 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] boring-cyborg[bot] commented on pull request #17640: redshift to S3: add optional SQL parameters

2021-08-18 Thread GitBox
boring-cyborg[bot] commented on pull request #17640: URL: https://github.com/apache/airflow/pull/17640#issuecomment-901548299 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 commented on a change in pull request #17700: Add error check for config_file parameter in GKEStartPodOperator

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17700: URL: https://github.com/apache/airflow/pull/17700#discussion_r691719277 ## File path: airflow/providers/google/cloud/operators/kubernetes_engine.py ## @@ -292,6 +292,12 @@ def __init__( "Credentials (ADC)

[airflow] branch main updated (b99d029 -> fbf5163)

2021-08-18 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 b99d029 Add `v2-*-stable` to codecov branches (#17707) add fbf5163 Add a note about XCOMs push condition

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #17690: Add a note about XCOMs push condition for K8sPodOperator docs

2021-08-18 Thread GitBox
boring-cyborg[bot] commented on pull request #17690: URL: https://github.com/apache/airflow/pull/17690#issuecomment-901547537 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 #17690: Add a note about XCOMs push condition for K8sPodOperator docs

2021-08-18 Thread GitBox
kaxil merged pull request #17690: URL: https://github.com/apache/airflow/pull/17690 -- 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] kaxil opened a new pull request #17710: Doc: Update Upgrade to 2 docs with Airflow 1.10.x EOL dates

2021-08-18 Thread GitBox
kaxil opened a new pull request #17710: URL: https://github.com/apache/airflow/pull/17710 This PR improves the Upgrading to 2 docs and add dates when 1.10.x reached EOL and clarifies few things like Python 3.9 support etc. --- **^ Add meaningful description above**

[airflow] branch main updated: Add `v2-*-stable` to codecov branches (#17707)

2021-08-18 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 b99d029 Add `v2-*-stable` to codecov branches

[GitHub] [airflow] kaxil merged pull request #17707: Add `v2-*-stable` branches to codecov

2021-08-18 Thread GitBox
kaxil merged pull request #17707: URL: https://github.com/apache/airflow/pull/17707 -- 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] github-actions[bot] commented on pull request #17708: Rename ``task_concurrency`` to ``max_active_tis_per_dag``

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17708: URL: https://github.com/apache/airflow/pull/17708#issuecomment-901542707 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 a change in pull request #17708: Rename ``task_concurrency`` to ``max_active_tis_per_dag``

2021-08-18 Thread GitBox
potiuk commented on a change in pull request #17708: URL: https://github.com/apache/airflow/pull/17708#discussion_r691714139 ## File path: tests/serialization/test_dag_serialization.py ## @@ -887,6 +887,7 @@ def test_no_new_fields_added_to_base_operator(self):

[GitHub] [airflow] sudohainguyen commented on issue #13487: DockerOperator Could not serialize the XCom value into JSON

2021-08-18 Thread GitBox
sudohainguyen commented on issue #13487: URL: https://github.com/apache/airflow/issues/13487#issuecomment-901541912 > > > > > enable_xcom_pickling = True > > > > > > > > > > > > This worked for me too. > > > > > > > > > Did not work for me. Any alternatives? > >

[GitHub] [airflow] github-actions[bot] commented on pull request #17709: Doc: Replace deprecated param from docstrings

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17709: URL: https://github.com/apache/airflow/pull/17709#issuecomment-901541723 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] commented on pull request #17707: Add `v2-*-stable` branches to codecov

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17707: URL: https://github.com/apache/airflow/pull/17707#issuecomment-901541550 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 #17456: Adding TaskGroup support in chain()

2021-08-18 Thread GitBox
potiuk commented on pull request #17456: URL: https://github.com/apache/airflow/pull/17456#issuecomment-901540998 > Happy to take a look and help any way I can. Is this test always failing for the same exception? If not, is there a set of logs I can look through to see any failure trends?

[GitHub] [airflow] josh-fell edited a comment on pull request #17456: Adding TaskGroup support in chain()

2021-08-18 Thread GitBox
josh-fell edited a comment on pull request #17456: URL: https://github.com/apache/airflow/pull/17456#issuecomment-901537912 > > Those two (`test_mark_success_on_success_callback` and `test_verify_integrity_if_dag_not_changed`) have been flaky recently; they are likely not related to your

[GitHub] [airflow] kaxil edited a comment on pull request #17496: deduplicate running jobs on BigQueryInsertJobOperator

2021-08-18 Thread GitBox
kaxil edited a comment on pull request #17496: URL: https://github.com/apache/airflow/pull/17496#issuecomment-901540606 Following test is failing, can you fix it please @blcksrx ``` FAILED

[GitHub] [airflow] kaxil commented on pull request #17496: deduplicate running jobs on BigQueryInsertJobOperator

2021-08-18 Thread GitBox
kaxil commented on pull request #17496: URL: https://github.com/apache/airflow/pull/17496#issuecomment-901540606 Following test is failing, can you fix it please @blcksrx ``` FAILED

[GitHub] [airflow] kaxil opened a new pull request #17709: Doc: Replace deprecated param from docstrings

2021-08-18 Thread GitBox
kaxil opened a new pull request #17709: URL: https://github.com/apache/airflow/pull/17709 Follow-up of https://github.com/apache/airflow/pull/16267 . Docstrings should show new values instead of deprecated. (`max_active_tasks` instead of concurency) --- **^ Add

[GitHub] [airflow] potiuk commented on pull request #17339: Adding support for multiple task-ids in the external task sensor

2021-08-18 Thread GitBox
potiuk commented on pull request #17339: URL: https://github.com/apache/airflow/pull/17339#issuecomment-901538878 Sorry it took so long. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [airflow] kaxil commented on a change in pull request #17690: Add a note about XCOMs push condition for K8sPodOperator docs

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17690: URL: https://github.com/apache/airflow/pull/17690#discussion_r691710485 ## File path: docs/apache-airflow-providers-cncf-kubernetes/operators.rst ## @@ -119,6 +119,9 @@ See the following example on how this occurs:

[airflow] branch main updated (32475fa -> 6040125)

2021-08-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 32475fa Sync DB Migrations with Airflow 2.1.3 (#17703) add 6040125 Adding support for multiple task-ids in the

[GitHub] [airflow] potiuk closed issue #16705: Ability to add multiple task_ids in the ExternalTaskSensor

2021-08-18 Thread GitBox
potiuk closed issue #16705: URL: https://github.com/apache/airflow/issues/16705 -- 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 merged pull request #17339: Adding support for multiple task-ids in the external task sensor

2021-08-18 Thread GitBox
potiuk merged pull request #17339: URL: https://github.com/apache/airflow/pull/17339 -- 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] boring-cyborg[bot] commented on pull request #17339: Adding support for multiple task-ids in the external task sensor

2021-08-18 Thread GitBox
boring-cyborg[bot] commented on pull request #17339: URL: https://github.com/apache/airflow/pull/17339#issuecomment-901538811 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 opened a new pull request #17708: Rename ``task_concurrency`` to ``max_active_tis_per_dag``

2021-08-18 Thread GitBox
kaxil opened a new pull request #17708: URL: https://github.com/apache/airflow/pull/17708 Follow-up of https://github.com/apache/airflow/pull/16267 Renames `task_concurrency` to `max_active_tis_per_dag` Some of Airflow's concurrency settings have been a source of confusion for

[GitHub] [airflow] josh-fell commented on pull request #17456: Adding TaskGroup support in chain()

2021-08-18 Thread GitBox
josh-fell commented on pull request #17456: URL: https://github.com/apache/airflow/pull/17456#issuecomment-901537912 > > Those two (`test_mark_success_on_success_callback` and `test_verify_integrity_if_dag_not_changed`) have been flaky recently; they are likely not related to your

[GitHub] [airflow] kaxil opened a new pull request #17707: Add `v2-*-stable` branches to codecov

2021-08-18 Thread GitBox
kaxil opened a new pull request #17707: URL: https://github.com/apache/airflow/pull/17707 Remove `v1-*` branches and adds `v2-*` branches --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil commented on a change in pull request #17700: Add error check for config_file parameter in GKEStartPodOperator

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17700: URL: https://github.com/apache/airflow/pull/17700#discussion_r691709121 ## File path: tests/providers/google/cloud/operators/test_kubernetes_engine.py ## @@ -182,6 +182,19 @@ def test_execute(self, file_mock,

[airflow] branch main updated (87e64fa -> 32475fa)

2021-08-18 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 87e64fa Clearly document no breaking change for 2.1.2 and 2.1.3 (#17706) add 32475fa Sync DB Migrations

[GitHub] [airflow] kaxil merged pull request #17703: Sync DB Migrations with Airflow 2.1.3

2021-08-18 Thread GitBox
kaxil merged pull request #17703: URL: https://github.com/apache/airflow/pull/17703 -- 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] thesuperzapper commented on issue #11707: Manage pools in Helm Chart

2021-08-18 Thread GitBox
thesuperzapper commented on issue #11707: URL: https://github.com/apache/airflow/issues/11707#issuecomment-901531822 I want to highlight that the ["user-community" chart](https://github.com/airflow-helm/charts/tree/main/charts/airflow) (this is separate from the "official") supports this.

[GitHub] [airflow] thesuperzapper commented on issue #11708: Git sync for plugins in the Helm Chart

2021-08-18 Thread GitBox
thesuperzapper commented on issue #11708: URL: https://github.com/apache/airflow/issues/11708#issuecomment-901530913 The [user-community chart](https://github.com/airflow-helm/charts/tree/main/charts/airflow) (not the official) has implemented support for using plugins from within your

[GitHub] [airflow] thesuperzapper commented on issue #16684: Helm Chart Allow for Empty data settings

2021-08-18 Thread GitBox
thesuperzapper commented on issue #16684: URL: https://github.com/apache/airflow/issues/16684#issuecomment-901529141 Those watching might be interested in how the ["user community" chart](https://github.com/airflow-helm/charts/tree/main/charts/airflow) (not the "official" one) manages

[airflow-site] tag 0.0.6 created (now 790089c)

2021-08-18 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to tag 0.0.6 in repository https://gitbox.apache.org/repos/asf/airflow-site.git. at 790089c (commit) No new revisions were added by this update.

[airflow-site] branch gh-pages updated: Deploying to gh-pages from @ 790089c6b51c60f7e31f3f3c2fe4f3afa0737b1b 

2021-08-18 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch gh-pages in repository https://gitbox.apache.org/repos/asf/airflow-site.git The following commit(s) were added to refs/heads/gh-pages by this push: new d24fe6e Deploying to gh-pages

[airflow] branch main updated (d5a6dc7 -> 87e64fa)

2021-08-18 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 d5a6dc7 Minor doc formatting fix (#17704) add 87e64fa Clearly document no breaking change for 2.1.2 and

[GitHub] [airflow] kaxil merged pull request #17706: Clearly document no breaking change for 2.1.2 and 2.1.3

2021-08-18 Thread GitBox
kaxil merged pull request #17706: URL: https://github.com/apache/airflow/pull/17706 -- 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] thesuperzapper commented on issue #17188: Chart: Uses deprecated v1beta1 Ingress Resource. Incompatible with Kubernetes 1.22

2021-08-18 Thread GitBox
thesuperzapper commented on issue #17188: URL: https://github.com/apache/airflow/issues/17188#issuecomment-901526009 If you want some reference, here is the PR that made this change in the "user-community" (not "official") helm chart: https://github.com/airflow-helm/charts/pull/381

[GitHub] [airflow-site] kaxil merged pull request #467: Airflow theme v0.0.6

2021-08-18 Thread GitBox
kaxil merged pull request #467: URL: https://github.com/apache/airflow-site/pull/467 -- 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-site] branch main updated: Airflow theme v0.0.6 (#467)

2021-08-18 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-site.git The following commit(s) were added to refs/heads/main by this push: new 790089c Airflow theme v0.0.6 (#467)

[GitHub] [airflow] github-actions[bot] commented on pull request #17706: Clearly document no breaking change for 2.1.2 and 2.1.3

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17706: URL: https://github.com/apache/airflow/pull/17706#issuecomment-901525275 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] kaxil opened a new pull request #17706: Clearly document no breaking change for 2.1.2 and 2.1.3

2021-08-18 Thread GitBox
kaxil opened a new pull request #17706: URL: https://github.com/apache/airflow/pull/17706 Similar to https://github.com/apache/airflow/pull/6240 --- **^ Add meaningful description above** Read the **[Pull Request

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

2021-08-18 Thread GitBox
potiuk commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691694600 ## File path: airflow/providers/apache/druid/provider.yaml ## @@ -51,6 +51,10 @@ hooks: hook-class-names: -

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

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691692384 ## File path: airflow/providers/apache/druid/provider.yaml ## @@ -51,6 +51,10 @@ hooks: hook-class-names: -

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

2021-08-18 Thread GitBox
potiuk commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691692380 ## File path: airflow/provider_info.schema.json ## @@ -16,11 +16,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-08-18 Thread GitBox
potiuk commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691691883 ## File path: airflow/providers/apache/druid/provider.yaml ## @@ -51,6 +51,10 @@ hooks: hook-class-names: -

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

2021-08-18 Thread GitBox
potiuk commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691691883 ## File path: airflow/providers/apache/druid/provider.yaml ## @@ -51,6 +51,10 @@ hooks: hook-class-names: -

[GitHub] [airflow] kaxil closed issue #17646: Scheduler is not working

2021-08-18 Thread GitBox
kaxil closed issue #17646: URL: https://github.com/apache/airflow/issues/17646 -- 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] kaxil commented on a change in pull request #17682: Import Hooks lazily individually in providers manager

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691690929 ## File path: airflow/providers/apache/druid/provider.yaml ## @@ -51,6 +51,10 @@ hooks: hook-class-names: -

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

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

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

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691690190 ## File path: airflow/provider_info.schema.json ## @@ -16,11 +16,33 @@ }, "hook-class-names": { "type": "array", - "description":

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

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17682: URL: https://github.com/apache/airflow/pull/17682#discussion_r691689974 ## File path: airflow/provider_info.schema.json ## @@ -16,11 +16,33 @@ }, "hook-class-names": { "type": "array", - "description":

[airflow] branch main updated (b6c20a8 -> d5a6dc7)

2021-08-18 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 b6c20a8 docs(dagowner): describe dag owner more carefully (#17699) add d5a6dc7 Minor doc formatting fix

[GitHub] [airflow] kaxil merged pull request #17704: Minor doc formatting fix

2021-08-18 Thread GitBox
kaxil merged pull request #17704: URL: https://github.com/apache/airflow/pull/17704 -- 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: docs(dagowner): describe dag owner more carefully (#17699)

2021-08-18 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 b6c20a8 docs(dagowner): describe dag

[GitHub] [airflow] ephraimbuddy merged pull request #17699: docs(dagowner): describe dag owner more carefully

2021-08-18 Thread GitBox
ephraimbuddy merged pull request #17699: URL: https://github.com/apache/airflow/pull/17699 -- 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] kaxil commented on pull request #17704: Minor doc formatting fix

2021-08-18 Thread GitBox
kaxil commented on pull request #17704: URL: https://github.com/apache/airflow/pull/17704#issuecomment-901512851 > So the competition continues @kaxil ;) ![https://media.giphy.com/media/e7FOBuKCDtwWI/giphy.gif](https://media.giphy.com/media/e7FOBuKCDtwWI/giphy.gif) -- This is an

[GitHub] [airflow-site] kaxil opened a new pull request #467: Airflow theme v0.0.6

2021-08-18 Thread GitBox
kaxil opened a new pull request #467: URL: https://github.com/apache/airflow-site/pull/467 Release Airflow theme 0.0.6 that removes Airflow Summit banner #466 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[airflow-site] 01/01: Airflow theme v0.0.6

2021-08-18 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch rel-0.0.6 in repository https://gitbox.apache.org/repos/asf/airflow-site.git commit d8c9c8aa8515c3111b2f65a2709466d6f602b77f Author: Kaxil Naik AuthorDate: Thu Aug 19 01:12:15 2021 +0100

[airflow-site] branch rel-0.0.6 created (now d8c9c8a)

2021-08-18 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch rel-0.0.6 in repository https://gitbox.apache.org/repos/asf/airflow-site.git. at d8c9c8a Airflow theme v0.0.6 This branch includes the following new commits: new d8c9c8a

[GitHub] [airflow] github-actions[bot] commented on pull request #17704: Minor doc formatting fix

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17704: URL: https://github.com/apache/airflow/pull/17704#issuecomment-901511457 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] kaxil opened a new pull request #17704: Minor doc formatting fix

2021-08-18 Thread GitBox
kaxil opened a new pull request #17704: URL: https://github.com/apache/airflow/pull/17704 Just a minor doc formatting fix to remove space --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] github-actions[bot] closed pull request #12925: [Loki log handler] - Integration with Grafana Loki

2021-08-18 Thread GitBox
github-actions[bot] closed pull request #12925: URL: https://github.com/apache/airflow/pull/12925 -- 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] github-actions[bot] closed issue #13391: MySqlToHiveOperator , ”invalid path“ while loding the extracted csv to hive

2021-08-18 Thread GitBox
github-actions[bot] closed issue #13391: URL: https://github.com/apache/airflow/issues/13391 -- 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] github-actions[bot] commented on issue #13391: MySqlToHiveOperator , ”invalid path“ while loding the extracted csv to hive

2021-08-18 Thread GitBox
github-actions[bot] commented on issue #13391: URL: https://github.com/apache/airflow/issues/13391#issuecomment-901510398 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] kaxil commented on a change in pull request #17676: Mark DAG.param as a decorated field

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17676: URL: https://github.com/apache/airflow/pull/17676#discussion_r691684818 ## File path: airflow/serialization/serialized_objects.py ## @@ -651,7 +651,7 @@ class SerializedDAG(DAG, BaseSerialization): not pickle-able.

[GitHub] [airflow] github-actions[bot] commented on issue #15563: airflow dag can't be deleted cleanly

2021-08-18 Thread GitBox
github-actions[bot] commented on issue #15563: URL: https://github.com/apache/airflow/issues/15563#issuecomment-901510366 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] kaxil commented on a change in pull request #17683: rename none_failed_or_skipped by none_failed_and_at_least_one_succeed

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17683: URL: https://github.com/apache/airflow/pull/17683#discussion_r691682079 ## File path: airflow/utils/trigger_rule.py ## @@ -32,6 +32,7 @@ class TriggerRule: NONE_SKIPPED = 'none_skipped' DUMMY = 'dummy' ALWAYS =

[GitHub] [airflow] kaxil commented on a change in pull request #17683: rename none_failed_or_skipped by none_failed_and_at_least_one_succeed

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17683: URL: https://github.com/apache/airflow/pull/17683#discussion_r691316424 ## File path: airflow/utils/trigger_rule.py ## @@ -32,6 +32,7 @@ class TriggerRule: NONE_SKIPPED = 'none_skipped' DUMMY = 'dummy' ALWAYS =

[GitHub] [airflow] kaxil commented on a change in pull request #17683: rename none_failed_or_skipped by none_failed_and_at_least_one_succeed

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17683: URL: https://github.com/apache/airflow/pull/17683#discussion_r691682079 ## File path: airflow/utils/trigger_rule.py ## @@ -32,6 +32,7 @@ class TriggerRule: NONE_SKIPPED = 'none_skipped' DUMMY = 'dummy' ALWAYS =

[airflow] branch main updated (09828a5 -> 79c1a0e)

2021-08-18 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 09828a5 docs(impersonation): update note so avoid misintrepretation (#17701) add 79c1a0e Add ``Scribd`` to

[GitHub] [airflow] kaxil merged pull request #17685: Update INTHEWILD.md

2021-08-18 Thread GitBox
kaxil merged pull request #17685: URL: https://github.com/apache/airflow/pull/17685 -- 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] boring-cyborg[bot] commented on pull request #17685: Update INTHEWILD.md

2021-08-18 Thread GitBox
boring-cyborg[bot] commented on pull request #17685: URL: https://github.com/apache/airflow/pull/17685#issuecomment-901501734 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] github-actions[bot] commented on pull request #17685: Update INTHEWILD.md

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17685: URL: https://github.com/apache/airflow/pull/17685#issuecomment-901501683 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

[airflow] branch main updated (c698204 -> 09828a5)

2021-08-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 c698204 Add steps for building release package without breeze. (#17702) add 09828a5 docs(impersonation):

[GitHub] [airflow] potiuk merged pull request #17701: docs(impersonation): update note so avoid misintrepretation

2021-08-18 Thread GitBox
potiuk merged pull request #17701: URL: https://github.com/apache/airflow/pull/17701 -- 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] github-actions[bot] commented on pull request #17703: Sync DB Migrations with Airflow 2.1.3

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17703: URL: https://github.com/apache/airflow/pull/17703#issuecomment-901477162 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] benjcabalona1029 closed pull request #17687: Docker Operator with docker-socket-proxy

2021-08-18 Thread GitBox
benjcabalona1029 closed pull request #17687: URL: https://github.com/apache/airflow/pull/17687 -- 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] kaxil opened a new pull request #17703: Sync DB Migrations with Airflow 2.1.3

2021-08-18 Thread GitBox
kaxil opened a new pull request #17703: URL: https://github.com/apache/airflow/pull/17703 Since https://github.com/apache/airflow/pull/16401 was backported to Airflow 2.1.3, the order of DB migrations needs to be changed in the `main` branch as the other PRs with DB migrations weren't

[GitHub] [airflow] github-actions[bot] commented on pull request #17700: Add error check for config_file parameter in GKEStartPodOperator

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17700: URL: https://github.com/apache/airflow/pull/17700#issuecomment-901474189 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

[airflow] branch main updated (0016007 -> c698204)

2021-08-18 Thread jhtimmins
This is an automated email from the ASF dual-hosted git repository. jhtimmins pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 0016007 hdfs provider: allow SSL webhdfs connections (#17637) add c698204 Add steps for building release

[GitHub] [airflow] jhtimmins merged pull request #17702: Add steps for building release package without breeze.

2021-08-18 Thread GitBox
jhtimmins merged pull request #17702: URL: https://github.com/apache/airflow/pull/17702 -- 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] kaxil commented on a change in pull request #17702: Add steps for building release package without breeze.

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17702: URL: https://github.com/apache/airflow/pull/17702#discussion_r691640282 ## File path: dev/README_RELEASE_AIRFLOW.md ## @@ -115,14 +115,19 @@ The Release Candidate artifacts we vote upon should be the exact ones we vote ag

[GitHub] [airflow] github-actions[bot] commented on pull request #17702: Add steps for building release package without breeze.

2021-08-18 Thread GitBox
github-actions[bot] commented on pull request #17702: URL: https://github.com/apache/airflow/pull/17702#issuecomment-901464130 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] kaxil commented on a change in pull request #17702: Add steps for building release package without breeze.

2021-08-18 Thread GitBox
kaxil commented on a change in pull request #17702: URL: https://github.com/apache/airflow/pull/17702#discussion_r691640009 ## File path: dev/README_RELEASE_AIRFLOW.md ## @@ -115,14 +115,19 @@ The Release Candidate artifacts we vote upon should be the exact ones we vote ag

  1   2   3   4   5   6   >