search

apache / airflow

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

Status of testing Providers that were prepared on April 10, 2024 #38904

Closed eladkal closed 7 months ago

eladkal commented 7 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.7.0rc1

All users involved in the PRs: @ferruzzi @dondaum @potiuk @sudiptob2 @eladkal @korolkevich @bolkedebruin @dolfinus @stiak @Lee-W @AlexisBRENON @molcay @dabla @dybolo @shahar1 @e-galan @CYarros10 @chrishronek @amoghrajesh @romsharon98

Committer

pdebelak commented 7 months ago

I can confirm that my change to cncf.kubernetes is working as expected.

Taragolis commented 7 months ago

Just figure out that we accidentally exclude https://github.com/apache/airflow/pull/37458 from the changelog. We could add it after the release, because changes itself presented into the released package.

gschuurman commented 7 months ago

38872 login is working as expected again with both email set and unset

ellisms commented 7 months ago

37886 is good.

eladkal commented 7 months ago

Just figure out that we accidentally exclude #37458 from the changelog. We could add it after the release, because changes itself presented into the released package.

Its a mistake in the release process. But its more than just change log. This PR is a feature while the rc1 is a bug fix release thus I will exclude yandex from this wave and cut rc2 afterwards

jliu0812 commented 7 months ago

#38022 is good. ✅

stiak commented 7 months ago

37736 is good.

Notification_Center

sudohainguyen commented 7 months ago

38223 and #38106 are good ✅

SubhamSinghal commented 7 months ago

Added a fix for #38741 in #38918. There was an extra slash in api.

dondaum commented 7 months ago

38786 and #38496 are good ✅

potiuk commented 7 months ago

Checcked all my changes : all good !

moiseenkov commented 7 months ago

My changes work as expected:

38251, #38458, #38454, #37998, #38404, #37818, #38404, #37938

pankajkoti commented 7 months ago

Checked SFTP provider for https://github.com/apache/airflow/pull/38881 and is good!

eladkal commented 7 months ago

Added a fix for https://github.com/apache/airflow/pull/38741 in https://github.com/apache/airflow/pull/38918. There was an extra slash in api.

Databricks provider will be excluded from RC1

kacpermuda commented 7 months ago

I checked the whole OpenLineage provider + my PR for dbt.cloud #38033 - all is good.

e-galan commented 7 months ago

37969 is good.

dabla commented 7 months ago

Just tested #38528 and I confirm it is working for ODBC and JDBC, we had in install issue of the rc provider hence why I dint't work at first but now I can confirm it works

amoghrajesh commented 7 months ago

Installed and RC with breeze and tested my changes:

All these changes are working as expected.

Lee-W commented 7 months ago

Verified #38390, https://github.com/apache/airflow/pull/38396/, https://github.com/apache/airflow/pull/38397/, https://github.com/apache/airflow/pull/38394/, https://github.com/apache/airflow/pull/38392/, https://github.com/apache/airflow/pull/38393/, https://github.com/apache/airflow/pull/38391/

eladkal commented 7 months ago

Thank you everyone. Providers are released. yandex and databricks providers are excluded and will followup with rc2

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