search

apache / airflow

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

Status of testing Providers that were prepared on November 08, 2023 #35540

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?

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

All users involved in the PRs: @ferruzzi @ylnsnv @syedahsn @stelsemeyer-m60 @dave-pollock @eladkal @zeotuan @MaksYermak @AetherUnbound @pankajkoti @makrushin-evgenii @andrew9691 @utkarsharma2 @Joffreybvn @phi-friday @Taragolis @JDarDagran @danielvdende @bolkedebruin @dstandish @dirrao @Lee-W @oleksiidav @josh-fell @romsharon98 @Piatachock @potiuk @o-nikolas

Committer

potiuk commented 1 year ago

My change was only in "tests" and it works :)

Lee-W commented 1 year ago

Tested with default Azure credential change and a few examples dags. All look good 👍

Piatachock commented 1 year ago

✅ for Yandex provider

JDarDagran commented 1 year ago

35428 tested, working as expected

utkarsharma2 commented 1 year ago

+1 Tested my changes, working as expected.

Thanks, Utkarsh Sharma

On Thu, Nov 9, 2023 at 5:46 PM Jakub Dardzinski @.***> wrote:

35428 https://github.com/apache/airflow/pull/35428 tested, working as

expected

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

Joffreybvn commented 1 year ago

There is something wrong with the HttpOperator. The new pagination functionality doesn't behave like expected with deferrable=False. But, it works fine when deferred. I opened a PR with a fix #35560

danielvdende commented 1 year ago

Tested my change (35212), works as expected

Daniel On 9 Nov 2023 at 19:07 +0100, Joffrey Bienvenu @.***>, wrote:

There is something wrong with the HttpOperator. The new pagination functionality doesn't behave like expected with deferrable=False. But, it works fine when deferred. I opened a PR with a fix #35560 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

MaksYermak commented 1 year ago

Tested (35295), working as expected

pankajkoti commented 1 year ago

Tested #35399, working as expected. Thank you!

AetherUnbound commented 1 year ago

Tested #34773 locally - Sensors which were failing previously are now being rescheduled as expected! Looks good on my end!

dirrao commented 1 year ago

Tested #34985 locally- it is working as expected.

phi-friday commented 1 year ago

tested #35293 locally. it works as expected.

eladkal commented 1 year ago

Thank you everyone. Providers are released (http and discord providers are excluded and RC2 will follow shortly) I invite everyone to help improve providers for the next release, a list of open issues can be found here.