apache / airflow

Apache Airflow - A platform to programmatically author, schedule, and monitor workflows
https://airflow.apache.org/
Apache License 2.0
35.37k stars 13.81k forks source link

Status of testing Providers that were prepared on August 30, 2021 #17922

Closed potiuk closed 2 years ago

potiuk commented 2 years ago

I have a kind request for all the contributors to the latest provider packages release. Could you help us to test the RC versions of the providers and let us know in the comment, if the issue is addressed there.

Providers that need testing

Those are providers that require testing as there were some substantial changes introduced:

Provider amazon: 2.2.0rc1


Provider hashicorp: 2.1.0rc1 : DROPPED FROM THE WAVE

Provider jdbc: 2.0.1rc1

New providers

Providers that do not need testing

Those are providers that were either doc-only or only got the meta-data updated including preparing for connection import optimisation that will be available in Airflow 2.2.0. Those provider do not contain changes that need additional testing.

nathadfield commented 2 years ago

17321 tested and looks good to me!

Aakcht commented 2 years ago

Tested #17637 - looks good

iblaine commented 2 years ago

17777 tested and looks good

nathadfield commented 2 years ago

I've just tried to start a version of our Astronomer based Airflow (v2.1.3) using the v2.1.0rc1 Hashicorp provider but I'm seeing the following on startup.

File "/usr/local/lib/python3.7/site-packages/airflow/configuration.py", line 470, in getimport
2021-08-31T07:58:31.887552500Z     f'The object could not be loaded. Please check "{key}" key in "{section}" section. '
2021-08-31T07:58:31.887570500Z airflow.exceptions.AirflowConfigException: The object could not be loaded. Please check "backend" key in "secrets" section. Current value: "airflow.providers.hashicorp.secrets.vault.VaultBackend".
pavelhlushchanka commented 2 years ago

17281 looks good

enima2684 commented 2 years ago

17474 https://github.com/apache/airflow/pull/17474 tested and looks good

On Tue, Aug 31, 2021 at 10:22 AM Pavel Hlushchanka @.***> wrote:

17281 https://github.com/apache/airflow/pull/17281 looks good

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/apache/airflow/issues/17922#issuecomment-909013081, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF6ZXBPXLR4W23HCIICVIMLT7SGNTANCNFSM5DCZC7YQ .

potiuk commented 2 years ago

I've just tried to start a version of our Astronomer based Airflow (v2.1.3) using the v2.1.0rc1 Hashicorp provider but I'm seeing the following on startup.

Did it work with previous version of the package (2.0.0) ?

nathadfield commented 2 years ago

I've just tried to start a version of our Astronomer based Airflow (v2.1.3) using the v2.1.0rc1 Hashicorp provider but I'm seeing the following on startup.

Did it work with previous version of the package (2.0.0) ?

Yep!

potiuk commented 2 years ago

Yep!

All right. I think I know. We'll drop Hashicorp from this wave and release it separately. Good spot.

Gabriel39 commented 2 years ago

alibaba 1.0.0rc1: tested and working :)

suhanovv commented 2 years ago

#17551 Looks good

peter-volkov commented 2 years ago

checked Provider yandex: 2.1.0rc1 with apache-airflow==2.1.3 works well.

Is it expected that apache-airflow-backport-providers-yandex for Airflow 1.* does not have new rc version? Will it be released?

potiuk commented 2 years ago

Is it excepted that apache-airflow-backport-providers-yandex for Airflow 1.* does not have new rc version? Will it be released?

According to the earlier announcements, we stopped releasing backports in March. Airflow 1.10 reached end of life in June and we encourage everyone to migrate to Airflow 2. There will be no more releases (not even critical security fixes) for Airflow 1.10 nor backports.

josh-fell commented 2 years ago

17399 Tested and verified. Thanks for putting this together @potiuk!

mcreenan commented 2 years ago

Tested amazon: 2.2.0rc1 for #17210 and it looks good.

I was able to run my test dag and validate it fails with amazon provider 1.2.0 and succeeded with 2.2.0rc1.

I love that the providers are separate and we can simply update the version in requirements.txt instead of having to update to a newer version of Airflow. Great work on that!

potiuk commented 2 years ago

I love that the providers are separate and we can simply update the version in requirements.txt instead of having to update to a newer version of Airflow. Great work on that!

Glad you like it! That was quite an effort of quite many people but I do agree it's been worth!

Goodkat commented 2 years ago

17850 tested and verified. Now it returns the pandas dataframe as expected.

potiuk commented 2 years ago

Cool! . We are at 1/3 of the fixes confirmed after 1/3 of the time. Great progress :) . Thanks everyone who are helping!

rahulraina7 commented 2 years ago

Tested https://github.com/apache/airflow/pull/16012 . Loading the file in s3 as expected

dacort commented 2 years ago

Tested #16766 with apache-airflow==2.1.3 - used the example DAG to trigger an EMR on EKS job to completion! 🎉

potiuk commented 2 years ago

FYI. I also marked some of the providers (jdbc, neo4j) that had only "type annotation" and doc changes on top of the "2.2.0" optimisation. There are no substantial changes and they are installing, importing and they are discovered properly, So no need to test them separately either.

BTW. For everyone watching - all the providers are now "Warning free" as well. There were a few providers in which just importing operators generated unnecessary warnings and some examples used deprecated operators. After merging #17900 w have no warnings (except those generated in 3rd-party libraries) and all examples are also using non-deprecated operators.

potiuk commented 2 years ago

Tested #17423 - works!

potiuk commented 2 years ago

We have majority of providers tested and working now- I would approciate some test on the ones that have not been touched yet,

Xrampino commented 2 years ago

Tested #17640. Works as expected!

subkanthi commented 2 years ago

Tested #17695 . Thanks

JavierLopezT commented 2 years ago

Tested #17379. It's working fine

potiuk commented 2 years ago

Last chance to test remaining providers :D

subkanthi commented 2 years ago

@potiuk tested both Example dag slackfile (#17400): Fixed SlackAPIFileOperator to upload file and file content (#17247):

it works, thanks

potiuk commented 2 years ago

Thanks everyone for testing! You rock!