airflow icon indicating copy to clipboard operation
airflow copied to clipboard

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

Open eladkal opened this issue 1 year ago • 12 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 amazon: 8.22.0rc1

Provider apache.iceberg: 1.0.0rc1

Provider google: 10.18.0rc2

Provider microsoft.azure: 10.1.0rc2

Provider pinecone: 2.0.0rc2

Provider tabular: 1.5.1rc1

All users involved in the PRs: @kacpermuda @e-galan @moiseenkov @Taragolis @keygrocery @o-nikolas @romsharon98 @dondaum @rawwar @dstandish @dabla @MaksYermak @CYarros10 @Kylea650 @wlinamchurch @tatiana @idantepper @tomrutter @ferruzz

Committer

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

eladkal avatar May 12 '24 19:05 eladkal

#39156 tested and working. #39466 added as a result of testing (small documentation change). This has been approved but hasn’t yet been merged in.

Kylea650 avatar May 12 '24 20:05 Kylea650

Hi, #38894, #39233, #39171, #39234, #39434 work as expected

moiseenkov avatar May 13 '24 06:05 moiseenkov

Hello, I tested following PR and they are all working a expected:

So for me everything related to MSGraphOperator is good.

dabla avatar May 13 '24 10:05 dabla

Hi! https://github.com/apache/airflow/pull/39206 works as expected

VladaZakharova avatar May 13 '24 12:05 VladaZakharova

https://github.com/apache/airflow/pull/38911 works as expected

wlinamchurch avatar May 13 '24 16:05 wlinamchurch

#38961 is good ✅

dondaum avatar May 13 '24 20:05 dondaum

#39018 and #37693 work as expected.

System tests for #38584 are failing. CreateCustomTrainingJobOperator has a problem with training new models that are based on parent models. It seems that there is a problem with parsing and validating the parent model's resource id. The problem occurs immediately after submitting a child model's training job, when we wait for gca resource creation. It happens both for sync and deferrable modes.

Addressed in #39640

e-galan avatar May 14 '24 06:05 e-galan

#39432 and #38673 works as expected

molcay avatar May 14 '24 08:05 molcay

Hi @eladkal ! Since https://github.com/apache/airflow/pull/38584 is not working as expected, we will prepare a fix for it ASAP. Can we please discuss if we can have RC3 for google-provider in this case with that fix?

VladaZakharova avatar May 15 '24 07:05 VladaZakharova

Hi @eladkal ! Since #38584 is not working as expected, we will prepare a fix for it ASAP. Can we please discuss if we can have RC3 for google-provider in this case with that fix?

We can but I'd prefer to release RC2 and have the fix released in next wave (plan to cut it during the weekend) The current RC2 is massive. I am not sure if it's right to hold it's release due to the reported issue. I think releasing it and applying a quick followup hot fix is a good balance. WDYT?

eladkal avatar May 15 '24 07:05 eladkal

Hi @eladkal ! Since #38584 is not working as expected, we will prepare a fix for it ASAP. Can we please discuss if we can have RC3 for google-provider in this case with that fix?

We can but due I'd prefer to release RC2 and have the fix released in next wave (plan to cut it during the weekend) The current RC2 is massive. I am not sure if it's right to hold it's release due to the reported issue. I think releasing it and applying a quick followup hot fix is a good balance. WDYT?

Sounds good to me, thank you!

VladaZakharova avatar May 15 '24 07:05 VladaZakharova

Hi @VladaZakharova @eladkal !

The fix for #38584 is implemented in #39640 .

e-galan avatar May 15 '24 13:05 e-galan

#39315 works as expected (issue #39127 can be closed)

shahar1 avatar May 17 '24 11:05 shahar1

Thank you everyone. Providers are released. pinecone provider is excluded and will have rc3

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

eladkal avatar May 17 '24 16:05 eladkal