-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Release 5.3 #7605
Comments
@auvipy Please allow me to manage the release. There are a couple of bugs in stamping I need to fix before we reach GA. |
FYI, I am not going to release the final version. just the alphas/beta's |
Please don't release another beta before we finalize the Header Stamping API. Also note that documentation points to the beta instead of the latest GA version. |
OK no worries. I will be focusing on other stuffs |
This is just a comment on timelines as I know the project is very busy and as always doing great work. I was looking at releases and noticed that the due date for this milestone for 5.3 is past now. It looks like there are a good deal more tasks in this 5.3 milestone that previous releases. According to the progress meter this milestone is 50% complete and it seems like it could be (guessing) ~6 months before the release. I'm wondering if it is possible for the team to reduce the scope of 5.3 to make a release? Alternatively, might you change the due date of this milestone? |
I think we might release 5.3 by December 2022. And yes we already covered lots of stuff. what is halting another release is some bug in newly introduced Stamping API in the canvas. they are regularly ironed out. I will check and re milestone some issues to next version to reduce the remaining stuff. |
@thedrow @Nusnus should we expect a new beta by the end of December? or that will take little longer? Another thing I want your opinion is, python 3.7 is going to be EOL by june 2023, pypy already dropped support for python 3.7. should we keep python 3.7 for 5.3? or python 3.8-3.11 should be the supported version? I don't have problem if you want to keep 3.7 support for final release. just want to know your opinions/views on this. |
Before the 5.3 release, I want to improve the testing to deliver a stable version with many bugs crushed. So, in my opinion, until the plugin provides new capabilities that are used for adding more tests to Celery, it would be unwise to release. We cannot even attend attention now due to the need for testing facilities for production-like edge cases. We want to increase our quality where real production environments are failing.
I think 5.3 should come with Python 3.7 support. |
I'd suggest a new tentative deadline: Feb 2023 |
OK fair enough. will plan accordingly. in the mean time we can improve CI and other stuff as well. |
Awesome, I also want to attend celery/kombu#1590 which is listed as a release blocker on the checklist above. I don't know when I'll get to it, but I got it on my mind. |
OK. also ping on pytest-celery works as well. |
@auvipy It would be useful to stabilize the "contribution pipeline" meaning all of the noise when opening a new PR. That + CI adjustments/upgrades (like I see you're doing recently 💯 ) would be efficient improvements as they are relatively small and infra-related so fixes are "searchable" (in opposed to features impl etc.) If you could prioritize the above I would appreciate it a lot! |
I'll start by stabilizing the To keep on topic, the effort on |
|
Can you comment on the possibility to reduce or re-milestone this 5.3 to achieve a release? Since Dec when the new date was set to Feb (now passed) there have only been a few percent of the current tasks (now at 64% complete). It seems like an additional 6 month wait time till some release is not unreasonable unless some things can be restructured/removed from this scope. |
I agree, we should update the official 5.3 to match the actual delta for release. To be more specific, the current |
it would be great if we at least can release another beta ASAP |
I released a new beta version. |
Thank you @thedrow ! |
I've updated the milestones and 5.3 project. Missing Steps from the official 5.3 Milestone:
Once we have a new passing smoke testing layer with new tests for edge cases that we cannot test in the existing infrastructure, we'll be able to sign the 5.3 as "Stable" and allow a proper final release. None of the above is listed in the 5.3 Milestone, but its my top-priority at the moment, including fixing any bug I find on the way. During the time I'll be invested in the above, other contributors might address more issues from which we will focus on bug fixes and enhancements. No breaking API changes, no complicated additions, though common sense would be applied per case. So the new milestone might be very close to 100%, but it (partially) lacks the above and as the P.S |
@thedrow @auvipy @auvipy - Let's focus on marking 5.3 more closely. I've updated the project so the columns should mean what they say with better accuracy to what's under it. Let's use the Project property on new PRs to mark their progress/status. I've also updated the 5.3 milestone date to May 2023. I believe this makes sense per my previous comment. @auvipy - The main purpose is to do some cleanup for better understanding of actual critical open tickets for 5.3, improve transparency with the community and organize the 5.3 project per actual real status for improved clarity on our current state. @auvipy , if you may, please invest some time cleaning up per the above, I will also try to monitor new issues and mark them accordingly. |
@auvipy btw was this handled? I think you fixed something regarding |
Thank you ❤️
Short answer: Yes.
There are many production-relevant use-cases that have never been automatically tested due to the limitations of the current testing infrastructure, so the time has come to cover that aspect of our testing coverage.
The benefits of improved testing apply to all celery users, but yes, enterprise audience may have a higher level of acceptance standards, and releasing 5.3 with improved QA coverage, specifically for production-relevant cases, may improve engagement and motivate upgrading to 5.3 by a wider range of users.
Basically, besides the stamping feature, Lastly, the new pytest plugin will allow adding smoke tests that might discover actual real critical new bugs that we also want to fix before the final release, to the best effort we can under a reasonable time frame. -- I hope I answered all of your questions @rgangopadhya 🙏 |
There are many issues of type "I have this setup, using this config/running this canvas, and I get XYZ". We don't know how critical are the existing unknown bugs, but as we progress into reproducing such architectures in the (upcoming) smoke tests, we'll be able to have a better definition of our stability to allow releasing 5.3 based on automatic tests and not a hunch / deadline. |
As this thread has evolved into an updates feed for the 5.3 release, I'd like to share some recent developments.
Finally, as per @thedrow's instructions, we can only release 5.3 once the stamping feature functions as expected. In retrospect, we'll have to avoid reaching a single release blocker that is long and complicated to finalize. Still, for 5.3, we'll use this buffer to accept additional contributions, refinements, and bug fixes that will go into 5.3. Lastly, from 5.3 forward, it will be much easier and simpler to advance, and our release cycles will be able to shrink significantly. |
@Nusnus lets re target at may 30. I also need to cross check some stuff and release kombu |
Thanks for all the work that's going into the 5.3 release - we've been running 5.3.0rc1 on python3.11 in production for ~10 days now and things seem mostly fine. The biggest challenge is this regression breaks error tracking with Sentry, Datadog, etc I'm not sure where to look for status of the stamping feature, so depending on how much feedback you'd like from folks testing this release - I'd suggest maybe considering an rc2 if the stamping feature ends up taking longer than this week (based on May 30 estimate) |
Hey @jbergler, thanks for the feedback. Regarding RC2 - I am currently investigating #8105 & #8086 which I wish to get fixed before we release RC2 as this would be a significant fix that I want to include in the next release candidate, as I expect challenges with this task. Lastly, per @thedrow last given instruction, we cannot release without the stamping feature working, but as the release date has been postponed too many times, I will discuss with @thedrow the possibility of making cuts and allowing releasing 5.3.0 with known issues and having 5.3.1 to close the known gaps which are just slow to close but are actively being worked on. This also means the smoke tests will also be ready only for 5.3.1, but I think it's bearable. @auvipy does the above sit well with you? |
it does. |
Ok, so I've fixed #8105 and #8086 and also talked with @thedrow regarding 5.3 release. Kombu Celery Lastly
P.S |
@Nusnus the thing you are not seeing are still on https://github.com/celery/celery/releases/tag/v5.3.0rc1. we had some missing dependencies issues in both celery & kombu and redis + sqlalchemy upgrade issues fixed so for compatibilty check and early feedback I just released another RC/RC2. I keep checking remaining kombu issues and release a stable release and test it in celery CI. so that you can move to stable release in next 72 hours. |
Ok cool, thanks for the update @auvipy ! Let's aim for getting Kombu released today if possible - we also need to see if there are any remaining Python 3.7 in the docs/readme etc. before we release Celery/Kombu 5.3.0 so as to avoid confusion (I will announce in the GitHub release we've officially dropped Python 3.7 support). Once you're done with Kombu @auvipy , I will open a PR for 5.3.0 for Celery with the version bump + changelog etc., so we can properly review the last step before release and avoid "oops we forgot this and that". LMK what you think @auvipy ! |
Tomorrow morning and will ping you. [Updated due date to 5th June 2023] |
Just noticed that - ok, thanks for the update. I'll move on with the Celery 5.3.0 release (GitHub / PyPI) once you let me know you're ready from your side @auvipy. |
Hey @auvipy , unfortunately I am dealing with an urgent personal issue and have had to get any work on hold. I will get back to partial availability tomorrow (6/June) and hopefully as the urgent issue passes, to full capacity. Family first ❤️ |
I will be available to get Celery 5.3.0 out and finish the PR we have open during the day (6/June) @auvipy I don't want to postpone the release any further if we can. |
OK I will try to improve the doc now. don't worry. |
@Nusnus I have updated the whats-new part & release notes. also tagged https://github.com/celery/celery/releases/tag/v5.3.0 and released the stable version as you are busy with emergencies. I tried to cover as much detail as i can. I completed it as I got some free time and not sure when again get that in 1 weeks. we can improve the release highlights and related docs as needed. thanks for all you effort and heads up. |
backport branch created for when we need that https://github.com/celery/celery/tree/v5.3 |
I have been a bit off these days, but I'm coming back slowly. I'll catch up on what I missed and move forwards. P.S P.P.S |
you can ping me on twitter / email whatever easy for you. |
Version: 5.3
Release PR:
Checklist
Release PR drafted
Release PR reviewed
The
main
branch build passesRelease Notes
What's New
Process
Alphas
Betas
Release Candidates
Release Blockers
Potential Release Blockers
The text was updated successfully, but these errors were encountered: