Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
27 of 45 tasks
potiuk opened this issue Aug 30, 2021 · 29 comments
Closed
27 of 45 tasks

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

potiuk opened this issue Aug 30, 2021 · 29 comments
Labels
kind:meta High-level information important to the community testing status Status of testing releases

Comments

@potiuk
Copy link
Member

potiuk commented Aug 30, 2021

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 apache.hdfs: 2.1.0rc1

Provider apache.hive: 2.0.2rc1

Provider apache.livy: 2.1.0rc1

Provider asana: 1.1.0rc1

Provider docker: 2.1.1rc1

Provider elasticsearch: 2.0.3rc1

Provider exasol: 2.0.1rc1

Provider google: 5.1.0rc1


Provider hashicorp: 2.1.0rc1 : DROPPED FROM THE WAVE


Provider jdbc: 2.0.1rc1

Provider microsoft.azure: 3.1.1rc1

Provider mongo: 2.1.0rc1

Provider neo4j: 2.0.1rc1

Provider postgres: 2.2.0rc1

Provider salesforce: 3.2.0rc1

Provider samba: 3.0.0rc1

Provider slack: 4.0.1rc1

Provider sqlite: 2.0.1rc1

Provider yandex: 2.1.0rc1

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.

@potiuk potiuk added the kind:meta High-level information important to the community label Aug 30, 2021
@nathadfield
Copy link
Collaborator

#17321 tested and looks good to me!

@Aakcht
Copy link
Contributor

Aakcht commented Aug 31, 2021

Tested #17637 - looks good

@iblaine
Copy link
Contributor

iblaine commented Aug 31, 2021

#17777 tested and looks good

@nathadfield
Copy link
Collaborator

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
Copy link
Contributor

#17281 looks good

@enima2684
Copy link
Contributor

enima2684 commented Aug 31, 2021 via email

@potiuk
Copy link
Member Author

potiuk commented Aug 31, 2021

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
Copy link
Collaborator

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
Copy link
Member Author

potiuk commented Aug 31, 2021

Yep!

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

@Gabriel39
Copy link
Contributor

alibaba 1.0.0rc1: tested and working :)

@suhanovv
Copy link
Contributor

#17551 Looks good

@peter-volkov
Copy link
Contributor

peter-volkov commented Aug 31, 2021

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
Copy link
Member Author

potiuk commented Aug 31, 2021

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
Copy link
Contributor

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

@mcreenan
Copy link
Contributor

mcreenan commented Aug 31, 2021

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
Copy link
Member Author

potiuk commented Aug 31, 2021

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
Copy link
Contributor

Goodkat commented Aug 31, 2021

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

@potiuk
Copy link
Member Author

potiuk commented Aug 31, 2021

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

@rahulraina7
Copy link
Contributor

Tested #16012 . Loading the file in s3 as expected

@dacort
Copy link
Contributor

dacort commented Sep 1, 2021

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

@potiuk
Copy link
Member Author

potiuk commented Sep 1, 2021

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
Copy link
Member Author

potiuk commented Sep 1, 2021

Tested #17423 - works!

@potiuk
Copy link
Member Author

potiuk commented Sep 1, 2021

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

@Xrampino
Copy link
Contributor

Xrampino commented Sep 1, 2021

Tested #17640. Works as expected!

@subkanthi
Copy link
Contributor

Tested #17695 . Thanks

@JavierLopezT
Copy link
Contributor

JavierLopezT commented Sep 2, 2021

Tested #17379. It's working fine

@potiuk
Copy link
Member Author

potiuk commented Sep 3, 2021

Last chance to test remaining providers :D

@subkanthi
Copy link
Contributor

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

it works, thanks

@potiuk
Copy link
Member Author

potiuk commented Sep 3, 2021

Thanks everyone for testing! You rock!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:meta High-level information important to the community testing status Status of testing releases
Projects
None yet
Development

No branches or pull requests