airflow icon indicating copy to clipboard operation
airflow copied to clipboard

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

Open eladkal opened this issue 1 year ago • 19 comments

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

Provider amazon: 8.21.0rc1

Provider apache.beam: 5.7.0rc1

Provider apache.livy: 3.8.0rc1

Provider apache.spark: 4.8.0rc2

Provider cncf.kubernetes: 8.2.0rc1

Provider common.sql: 1.13.0rc1

Provider databricks: 6.4.0rc1

Provider datadog: 3.6.0rc1

Provider dbt.cloud: 3.8.0rc1

Provider fab: 1.1.0rc1

Provider google: 10.18.0rc1

Provider hashicorp: 3.7.0rc2

Provider http: 4.11.0rc1

Provider jdbc: 4.3.0rc1

Provider microsoft.azure: 10.1.0rc1

Provider odbc: 4.6.0rc1

Provider openai: 1.2.0rc1

Provider openlineage: 1.7.1rc1

Provider pagerduty: 3.7.0rc2

Provider pinecone: 2.0.0rc1

Provider postgres: 5.11.0rc1

Provider qdrant: 1.1.0rc1

Provider salesforce: 5.7.0rc2

Provider slack: 8.7.0rc1

Provider snowflake: 5.5.0rc1

Provider teradata: 2.1.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

  • [X] I acknowledge that I am a maintainer/committer of the Apache Airflow project.

eladkal avatar May 01 '24 13:05 eladkal

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

nathadfield avatar May 01 '24 14:05 nathadfield

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?

dabla avatar May 01 '24 15:05 dabla

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

wlinamchurch avatar May 01 '24 16:05 wlinamchurch

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

gopidesupavan avatar May 01 '24 17:05 gopidesupavan

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

Tested. Works as expected.

Anush008 avatar May 01 '24 18:05 Anush008

#38961 is good. ✅

dondaum avatar May 01 '24 21:05 dondaum

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

dabla avatar May 02 '24 06:05 dabla

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.

eladkal avatar May 02 '24 06:05 eladkal

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.

dabla avatar May 02 '24 06:05 dabla

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

pankajastro avatar May 02 '24 08:05 pankajastro

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 ?

eladkal avatar May 02 '24 08:05 eladkal

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

paolo-moriello avatar May 02 '24 10:05 paolo-moriello

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.

Kylea650 avatar May 02 '24 10:05 Kylea650

Pineconepinecone: 2.0.0rc1

PR: #37307 is has bugs.

Fix is being implemented in #39365

rawwar avatar May 02 '24 12:05 rawwar

Happy with #38747

tomrutter avatar May 02 '24 12:05 tomrutter

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.

dabla avatar May 02 '24 17:05 dabla

Verified #39178 , works all fine

pankajkoti avatar May 02 '24 17:05 pankajkoti

#39266 looks good.

boraberke avatar May 02 '24 18:05 boraberke

#38716 tested and verified #38713 is resolved

zstrathe avatar May 02 '24 21:05 zstrathe

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

nyoungstudios avatar May 03 '24 18:05 nyoungstudios

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

Taragolis avatar May 04 '24 18:05 Taragolis

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

Lee-W avatar May 06 '24 01:05 Lee-W

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

Similar risk for #39130 and #39230

pankajkoti avatar May 06 '24 06:05 pankajkoti

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.

eladkal avatar May 06 '24 08:05 eladkal