apache / airflow

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

Status of testing Providers that were prepared on May 01, 2024 #39346

Closed eladkal closed 4 months ago

eladkal commented 4 months 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 airbyte: 3.8.0rc1

All users involved in the PRs: @mobuchowski @josix @mateuslatrova @kacpermuda @dabla @Anush008 @jedcunningham @boraberke @pankajastro @Taragolis @gopidesupavan @hussein-awala @wlinamchurch @nathadfield @pankajkoti @potiuk @Kylea650

Committer

nathadfield commented 4 months ago

@eladkal #39248 and #38736 tested and working correctly.

dabla commented 4 months ago

I know there is still an issue with the MSGraphSensor in that sense that it’s not working correctly in an async way and sometimes an event loop closed error can occur. This PR will fix it so dunno how to proceed for the msgraph part with this RC?

wlinamchurch commented 4 months ago

@eladkal https://github.com/apache/airflow/pull/38911 tested and working correctly.

gopidesupavan commented 4 months ago

@eladkal https://github.com/apache/airflow/pull/39300 tested and working correctly.

Anush008 commented 4 months ago

Provider qdrant: 1.1.0rc1 https://github.com/apache/airflow/pull/39273

Tested. Works as expected.

dondaum commented 4 months ago

38961 is good. ✅

dabla commented 4 months ago

Following 3 PR suffer from the same issues with the MSGraphSenor which has been fixed in #39304: #38111 #39237 #39207

eladkal commented 4 months ago

I know there is still an issue with the MSGraphSensor in that sense that it’s not working correctly in an async way and sometimes an event loop closed error can occur. This PR will fix it so dunno how to proceed for the msgraph part with this RC?

It's not merged. Only merged PR before release are included.

dabla commented 4 months ago

I know there is still an issue with the MSGraphSensor in that sense that it’s not working correctly in an async way and sometimes an event loop closed error can occur. This PR will fix it so dunno how to proceed for the msgraph part with this RC?

It's not merged. Only merged PR before release are included.

I know there is still an issue with the MSGraphSensor in that sense that it’s not working correctly in an async way and sometimes an event loop closed error can occur. This PR will fix it so dunno how to proceed for the msgraph part with this RC?

It's not merged. Only merged PR before release are included.

I know Elad I just wanted to make you guys aware of it.

pankajastro commented 4 months ago

Tested https://github.com/apache/airflow/pull/39279 and https://github.com/apache/airflow/pull/38536 looks good

eladkal commented 4 months ago

I know Elad I just wanted to make you guys aware of it.

If there is regression we can make RC2 then it will be included. What do you recommend ?

paolo-moriello commented 4 months ago

https://github.com/apache/airflow/pull/39325 tested 👍

Kylea650 commented 4 months ago

Anybody with an Azure account able to test #39156? I can provide an example DAG I just don’t have access to my work Azure account right now and don’t want to block the release.

rawwar commented 4 months ago

Pineconepinecone: 2.0.0rc1

PR: #37307 is has bugs.

Fix is being implemented in #39365

tomrutter commented 4 months ago

Happy with #38747

dabla commented 4 months ago

I know Elad I just wanted to make you guys aware of it.

If there is regression we can make RC2 then it will be included. What do you recommend ?

Yes an RC2 would be great.

pankajkoti commented 4 months ago

Verified #39178 , works all fine

boraberke commented 4 months ago

39266 looks good.

zstrathe commented 4 months ago

38716 tested and verified #38713 is resolved

nyoungstudios commented 4 months ago

Tested https://github.com/apache/airflow/pull/39139 and looks good!

Taragolis commented 4 months ago

Tests my changes (most of them just cosmetic), look good

Lee-W commented 4 months ago

for https://github.com/apache/airflow/pull/38912, we might introduce a potential bug as commented in https://github.com/apache/airflow/issues/36090#issuecomment-2094972855

pankajkoti commented 4 months ago

for #38912, we might introduce a potential bug as commented in #36090 (comment)

Similar risk for #39130 and #39230

eladkal commented 4 months ago

Thank you everyone. Providers are released. Google, microsoft.azure and pinecone are excluded from this wave.

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