apache / airflow

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

Status of testing Providers that were prepared on June 20, 2023 #32030

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?

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

The guidelines on how to test providers can be found in

Verify providers by contributors

All users involved in the PRs: @o-nikolas @utkarsharma2 @phanikumv @nathadfield @simonprydden @vargacypher @svdimchenko @longslvr @dstandish @uranusjr @dimonchik-suvorov @vincbeck @melugoyal @eladkal @Joffreybvn @sunank200 @moiseenkov @pankajastro @potiuk @killua1zoldyck @hussein-awala @raphaelauv @enchant3dmango @Lee-W @Adaverse @vandonr-amz @syedahsn @ying-w @bkossakowska @VladaZakharova @mahammi @pankajkoti @pgagnon

Committer

raphaelauv commented 1 year ago

apache/airflow:slim-2.6.2-python3.11 with apache-airflow-providers-cncf-kubernetes==7.1.0rc1

on a kubernetes-sigs/kind

KubernetesResourceOperator - KubernetesDeleteResourceOperator & KubernetesCreateResourceOperator (#29930):

:ok_hand:

pankajkoti commented 1 year ago

Verified that my changes in https://github.com/apache/airflow/pull/31201 and https://github.com/apache/airflow/pull/31849 are included in the RCs and that they look good. Tested both with corresponding example DAGs and they yield expected results ✅

Thank you for including this! 🙂

enchant3dmango commented 1 year ago

apache-airflow-providers-apache-spark==4.1.1rc1. ✅

Thank you! 🖖🏼

simonprydden commented 1 year ago

Verified my change in #31574 is included in the RC and working.

Thank you.

Adaverse commented 1 year ago

Tested #31765 works as expected.

Can we include #31783 in this release @eladkal @phanikumv?

Joffreybvn commented 1 year ago

JiraOperator (#31672) works like a charm, thanks !

vandonr-amz commented 1 year ago

good for me 👍

vargacypher commented 1 year ago

Tested apache-airflow-providers-discord==3.3.0rc1 ✅ image

killua1zoldyck commented 1 year ago

31837 Works as expected

longslvr commented 1 year ago

https://github.com/apache/airflow/pull/31171 is good to go!

vincbeck commented 1 year ago

31622 works as expected. My other changes are style related

Lee-W commented 1 year ago

https://github.com/apache/airflow/pull/31482 works as expected and we have successfully tested other providers using our sample DAGs without any issues.

eladkal commented 1 year ago

Can we include #31783 in this release @eladkal @phanikumv?

This PR is not merged yet. Once it's merged it will be released in a the following wave

phanikumv commented 1 year ago

Tested #30279, #31685, #31081,#31317 , #31249, #31758 as part of below RC and they are working as expected.

apache-airflow-providers-amazon==8.2.0rc1 pip install apache-airflow-providers-apache-livy==3.5.1rc1 pip install apache-airflow-providers-google==10.2.0rc1

ying-w commented 1 year ago

please remove https://github.com/apache/airflow/pull/31338 it is not working as expected, I just pushed up a fix for it

eladkal commented 1 year ago

please remove https://github.com/apache/airflow/pull/31338 it is not working as expected, I just pushed up a fix for it

We can not remove. The choices are cancel RC1 and cut RC2 or release anyway. Assuming this is the only issue I prefer to release anyway since this is a new small enhancement which won't cause any user pain if it's not working as expected. I will cut a followup release for the google provider with the fix as soon as this wave is released.

utkarsharma2 commented 1 year ago

Tested #31207, https://github.com/apache/airflow/pull/31207 #31359 https://github.com/apache/airflow/pull/31359 #30698 https://github.com/apache/airflow/pull/30698 and they are working as expected.

Thanks, Utkarsh Sharma

On Thu, Jun 22, 2023 at 11:31 AM eladkal @.***> wrote:

please remove #31338 https://github.com/apache/airflow/pull/31338 it is not working as expected, I just pushed up a fix for it

We can not remove. The choices are cancel RC1 and cut RC2 or release anyway. Assuming this is the only issue I prefer to release anyway since this is a new small enhancement which won't cause any user pain if it's not working as expected. I will cut a followup release for the google provider with the fix as soon as this wave is released.

— Reply to this email directly, view it on GitHub https://github.com/apache/airflow/issues/32030#issuecomment-1602054263, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDLAHNDYBA6G5EMZJ56FO3XMPNT5ANCNFSM6AAAAAAZNI7N2Y . You are receiving this because you were mentioned.Message ID: @.***>

pankajastro commented 1 year ago

https://github.com/apache/airflow/pull/30244, https://github.com/apache/airflow/pull/30865 and https://github.com/apache/airflow/pull/30945 looks good. https://github.com/apache/airflow/pull/31703 We reverted in https://github.com/apache/airflow/pull/31985. So all good. thank you!

hussein-awala commented 1 year ago

I tested:

I cannot test https://github.com/apache/airflow/pull/31991 because I don't have a GCP setup, but it's present in the release and it should work according to the GCP documentation

potiuk commented 1 year ago

Checked if #27264 , #31983 , #31780, #31984 are included. All looks good.

melugoyal commented 1 year ago

validated #31659, looks good

eladkal commented 1 year ago

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