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 05, 2023 #33132

Closed
35 of 44 tasks
eladkal opened this issue Aug 4, 2023 · 14 comments
Closed
35 of 44 tasks

Status of testing Providers that were prepared on August 05, 2023 #33132

eladkal opened this issue Aug 4, 2023 · 14 comments
Labels
kind:meta High-level information important to the community testing status Status of testing releases

Comments

@eladkal
Copy link
Contributor

eladkal commented Aug 4, 2023

Body

I have a kind request for all the contributors to the latest provider packages release.
Could you please help us to test the RC versions of the providers?

Let us know in the comment, whether the issue is addressed.

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

Provider amazon: 8.5.0rc1

Provider apache.drill: 2.4.3rc1

Provider apache.hive: 6.1.4rc1

Provider apache.spark: 4.1.3rc1

Provider apache.sqoop: 4.0.0rc1

Provider asana: 2.2.2rc1

Provider celery: 3.3.1rc1

Provider cncf.kubernetes: 7.4.1rc1

Provider common.sql: 1.6.2rc1

Provider docker: 3.7.2rc1

Provider ftp: 3.5.0rc1

Provider google: 10.6.0rc1

Provider imap: 3.3.0rc1

Provider microsoft.azure: 6.2.3rc1

Provider mysql: 5.2.1rc1

Provider openlineage: 1.0.1rc1

Provider redis: 3.3.1rc1

Provider smtp: 1.3.0rc1

Provider snowflake: 4.4.1rc1

The guidelines on how to test providers can be found in

Verify providers by contributors

All users involved in the PRs:
@pankajkoti @moiseenkov @okayhooni @hankehly @frodo2000 @VladaZakharova @hussein-awala @mahammi @mobuchowski @kimdwkimdw @ephraimbuddy @potiuk @eumiro @pegasas @rishi-kulkarni @JDarDagran

Committer

  • I acknowledge that I am a maintainer/committer of the Apache Airflow project.
@eladkal eladkal added kind:meta High-level information important to the community testing status Status of testing releases labels Aug 4, 2023
@rishi-kulkarni
Copy link
Contributor

I can confirm that #33045 makes them default timeout sufficiently long.

@hankehly
Copy link
Contributor

hankehly commented Aug 4, 2023

#32760 works on main branch (provider versions seem to match above).

Could someone please tell me how to install specific provider versions in a breeze environment? I am having some trouble following the directions in https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#provider-packages

github-actions bot pushed a commit to astronomer/astronomer-providers that referenced this issue Aug 5, 2023
@potiuk
Copy link
Member

potiuk commented Aug 5, 2023

#32760 works on main branch (provider versions seem to match above).

Could someone please tell me how to install specific provider versions in a breeze environment? I am having some trouble following the directions in https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#provider-packages

See the link posted in the issue at the top Verify providers by contributors

There are very precise instructions.

UPDATE (but the link is not precise - pointing to old chapter - this is the right link :D https://github.com/apache/airflow/blob/main/dev/README_RELEASE_PROVIDER_PACKAGES.md#verify-the-release-candidate-by-contributors

@potiuk
Copy link
Member

potiuk commented Aug 5, 2023

Checked all my changes - they all look good - except redis. I found out that the fix caused multiple "subscribe" calls for the sensor, so would likely cause problems. We should remove redis from the list, the fix for rc2 is coming.

@eladkal
Copy link
Contributor Author

eladkal commented Aug 6, 2023

google and redis provider will be excluded from RC1 and will have rc2

PR to be included in RC2 of redis:
#33139

PR to be included in RC2 of google:
#33148

@mobuchowski
Copy link
Contributor

Tested mine changes, they look good.

@hussein-awala
Copy link
Member

Tested my changes, they are included and they work as expected

@kimdwkimdw
Copy link
Contributor

Tested my changes with https://pypi.org/project/apache-airflow-providers-amazon/8.5.0rc1/

@okayhooni
Copy link
Contributor

Tested my minor changes worked well
#33057

@ephraimbuddy
Copy link
Contributor

Tested my refactor on azure #32980

@moiseenkov
Copy link
Contributor

#32960 and #32861 work as expected

apache-airflow-providers-google==10.6.0rc1
apache-airflow-providers-celery==3.3.1rc1
apache-airflow-providers-amazon==8.5.0rc1
apache-airflow-providers-cncf-kubernetes==7.4.1rc1

@frodo2000
Copy link
Contributor

frodo2000 commented Aug 7, 2023 via email

@pankajkoti
Copy link
Member

Tested #33074, #32946, #33039, #32968

Verified they are included in the RCs, ran tests and verified they work as expected.

@eladkal
Copy link
Contributor Author

eladkal commented Aug 8, 2023

Thank you everyone.
Providers are released

google and redis providers are excluded, I will cut RC2 shortly

I invite everyone to help improve providers for the next release, a list of open issues can be found here.

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