Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Status of testing of Apache Airflow 2.10.1rc1 #41956

Closed
17 of 23 tasks
utkarsharma2 opened this issue Sep 2, 2024 · 14 comments
Closed
17 of 23 tasks

Status of testing of Apache Airflow 2.10.1rc1 #41956

utkarsharma2 opened this issue Sep 2, 2024 · 14 comments
Labels
kind:meta High-level information important to the community testing status Status of testing releases

Comments

@utkarsharma2
Copy link
Contributor

utkarsharma2 commented Sep 2, 2024

Body

We are kindly requesting that contributors to Apache Airflow RC 2.10.1rc1 help test the RC.

Please let us know by commenting if the issue is addressed in the latest RC.

Thanks to all who contributed to the release (probably not a complete list!):
@gagan-bhullar-tech @obarisk @uranusjr @utkarsharma2 @bbovenzi @lh5844 @jedcunningham @amoghrajesh @jscheffl @shahar1 @potiuk @joaopamaral @collinmcnulty @Lee-W

Committer

  • I acknowledge that I am a maintainer/committer of the Apache Airflow project.
@utkarsharma2 utkarsharma2 added kind:meta High-level information important to the community testing status Status of testing releases labels Sep 2, 2024
@obarisk
Copy link
Contributor

obarisk commented Sep 2, 2024

#41591 works as expected

tested with
airflow.providers.slack.notifications.slack_webhook.send_slack_webhook_notification
and
airflow.providers.discord.notifications.discord.DiscordNotifier

@amoghrajesh
Copy link
Contributor

Tested my changes: #41783, #41680, #41694 with breeze and all of them work as expected.

@amoghrajesh
Copy link
Contributor

Also tested #41890, for which I am the original contributor. I think we should cherry pick to retain the original author because the committer who cherry picks might not be fully aware of the scope of the change and shouldn't be responsible for testing it.

@potiuk
Copy link
Member

potiuk commented Sep 3, 2024

I think that needs some tooling improvement when generating the issue. The commit itself retains the original author:

Screenshot 2024-09-03 at 06 39 03

But cherry-picked PR does not (and it cannot, because you cannot make another person author of your PR). So when we generate the issue, we should read commit information and simply add all commit authors to the issue.

Created #41969

@potiuk
Copy link
Member

potiuk commented Sep 3, 2024

All good for my changes !

@potiuk
Copy link
Member

potiuk commented Sep 3, 2024

One thing: @hadarsharon -> can you please check if your #41891 is solved in RC1 ?

@hadarsharon
Copy link

Yes @potiuk I just checked and can confirm that #41891 is resolved in v2.10.1.rc1

Cheers

@vatsrahul1001
Copy link
Collaborator

Noticed issue while using TriggerDagRunOperator

@joaopamaral
Copy link
Contributor

Tested #41809 with the following FAB version:

  • 1.2.2: permissions are synced correctly with the old access control format.
  • 1.3.0: permissions are synced correctly with new and old access control formats.

So all good for this change.

@Lee-W
Copy link
Member

Lee-W commented Sep 4, 2024

#41834 was @tirkarthi 's fix. I just tested it, and it works fine. Thanks @tirkarthi !

@tirkarthi
Copy link
Contributor

Thanks @Lee-W for the backport, it's working fine as expected.

@jrderuiter
Copy link

Confirmed that #42032 is working fine in the rc.

@utkarsharma2
Copy link
Contributor Author

Airflow 2.10.1 has been released. Thank you all for testing this release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:meta High-level information important to the community testing status Status of testing releases
Projects
None yet
Development

No branches or pull requests