apache / airflow

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

Status of testing Providers that were prepared on August 26, 2023 #33751

Closed eladkal closed 1 year ago

eladkal commented 1 year ago

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?

The guidelines on how to test providers can be found in

Verify providers by contributors

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.6.0rc1

All users involved in the PRs: @amoghrajesh @mobuchowski @zbstof @johnbrandborg @vincbeck @JDarDagran @benchoncy @ellisms @SamWheating @pankajastro @alexbegg @sunank200 @zinal @LipuFei @functicons @hussein-awala @ephraimbuddy @Lee-W @arjunanan6 @erdos2n @vandonr-amz @fdemiane @Owen-CH-Leung @phanikumv @Taragolis @mati-o @syedahsn @tiranux @covidium @yermalov-here @potiuk

Committer

alexbegg commented 1 year ago

Hi, https://github.com/apache/airflow/pull/33427, https://github.com/apache/airflow/pull/33425, https://github.com/apache/airflow/pull/33359 works as expected, even tested using an external database and all looks good.

alexbegg commented 1 year ago

I also tested https://github.com/apache/airflow/pull/33601 and it no longer has the ImportError warning, as expected

arjunanan6 commented 1 year ago

I have just tested #33314, works as expected.

LipuFei commented 1 year ago

I haved tested https://github.com/apache/airflow/pull/33652, looks good.

johnbrandborg commented 1 year ago

I have tested #33005 this morning. Messages indicate Service Principal Token is being used, and data was retrieved from a Databricks Workspace.

potiuk commented 1 year ago

Checked all my changes. All look good!

ephraimbuddy commented 1 year ago

Tested https://github.com/apache/airflow/pull/33457 and it works fine

Owen-CH-Leung commented 1 year ago

Tested #33135 and it works as expected. For #33306 , it's not working as expected so I have created another PR which just got merged.

https://github.com/apache/airflow/pull/33779

pankajastro commented 1 year ago

https://github.com/apache/airflow/pull/33598 and https://github.com/apache/airflow/pull/33627 looks good. thanks!

hussein-awala commented 1 year ago

Checked #33483 and it is present in the different providers.

Lee-W commented 1 year ago

Tested https://github.com/apache/airflow/pull/33716 and https://github.com/apache/airflow/pull/33467. Both works fine. Thanks!

amoghrajesh commented 1 year ago

Tested my changes:

Things do not seem to have regressed and it works as expected. Ran a few dags to target the cncf provider too. It looks good to me.

Thanks!

mobuchowski commented 1 year ago

Tested my changes, everything looks good.

zinal commented 1 year ago

Hi! #29811 looks fine for me, processed the test dag successfully.

phanikumv commented 1 year ago

32883 looks good

JDarDagran commented 1 year ago

32910 works as expected.

yermalov-here commented 1 year ago

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

ellisms commented 1 year ago

tested #33219 and it looks good.

fdemiane commented 1 year ago

tested #33500, works as expected

vandonr-amz commented 1 year ago

good for me

vincbeck commented 1 year ago

33327 and #33673 work as expected

eladkal commented 1 year ago

Thank you everyone. microsoft.azure and cncf.kubernetes are excluded from rc1. I will cut rc2 shortly. The rest of the providers are released.

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