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 March 03, 2023 #29901

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: 7.3.0rc1

The guidelines on how to test providers can be found in Verify providers by contributors

All users involved in the PRs: @kristopherkane @sunank200 @LMnet @phanikumv @jose-lpa @pdebelak @ferruzzi @vincbeck @r-richmond @vandonr-amz @eladkal @wiktorn @maxnathaniel @pgagnon @hussein-awala @bkossakowska @Taragolis @pankajastro @fritz-astronomer @romibuzi @josh-fell @lwyszomi @potiuk @s0neq @Aakcht @FloMom @IKholopov @yehoshuadimarsky

Committer

eladkal commented 1 year ago

Contributors are also encouraged to cast non-binding vote in the mailing list thread.

s0neq commented 1 year ago

Provider yandex: 3.3.0rc1 has been tested. all is fine 👌

vandonr-amz commented 1 year ago

all good for me as it's covered by a system test that's green

fritz-astronomer commented 1 year ago

Snowflake change has unit test coverage 👍

hussein-awala commented 1 year ago

Provider google: 8.11.0rc1 : I tested #29394 and it works as expected. Provider sftp: 4.2.4rc1: Same for #29794. Provider hashicorp: 3.3.0rc1: I have tested #29734 and it works, but currently we should use the same variables, config and connections paths for all the mount points because there is a bug in the Vault secrets backend, I wonder if we can add #29908 to this release or we add it to the next one.

eladkal commented 1 year ago

@hussein-awala I will cancel RC1 for hashicorp provider and will cut RC2 after the current vote

Aakcht commented 1 year ago

Tested #29347 for SSH provider 3.5.0rc1 - works as expected. Waiting for full release!

Taragolis commented 1 year ago
potiuk commented 1 year ago

Looks good.

pdebelak commented 1 year ago

I checked apache-airflow-providers-amazon and the change in #29708 is working as expected.

maxnathaniel commented 1 year ago

Provider telegram: 4.0.0rc1 works as expected 👍

pankajastro commented 1 year ago

https://github.com/apache/airflow/pull/29657 https://github.com/apache/airflow/pull/29348 and https://github.com/apache/airflow/pull/29364 Looks good

romibuzi commented 1 year ago

29659 working as expected

raphaelauv commented 1 year ago

for the kubernetes the name of the feature is incorrect

KubernetesPodOperator._render_nested_template_fields improved by changing the conditionals for a map (#29760): @jose-lpa

->

KubernetesPodOperator._render_nested_template_fields improved by changing the conditionals (#29760): @jose-lpa

( there is no map )

eladkal commented 1 year ago

for the kubernetes the name of the feature is incorrect

KubernetesPodOperator._render_nested_template_fields improved by changing the conditionals for a map (#29760): @jose-lpa

->

KubernetesPodOperator._render_nested_template_fields improved by changing the conditionals (#29760): @jose-lpa

( there is no map )

The release mechanism currently does not allow us to change the commit messages. Probably something we need to improve on in the future. Since this is only issue with the commit title and not with the actual functionality I see no reason why we can't continue with release of this package.

josh-fell commented 1 year ago

29741, #29699, and #29549 look good.

Thanks for organizing @eladkal!

pgagnon commented 1 year ago

29623 working as expected.

IKholopov commented 1 year ago

29680 is effectively a noop (for now) and it is working as expected :)

kristopherkane commented 1 year ago

29136 Has been tested with real Google Cloud resources successfully.

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.

LMnet commented 1 year ago

29434 has a unit test and also I use the same code locally in a while.