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

Upgrade kombu==5.4.2 #36032

Merged
merged 1 commit into from
Mar 25, 2025
Merged

Upgrade kombu==5.4.2 #36032

merged 1 commit into from
Mar 25, 2025

Conversation

millerdev
Copy link
Contributor

@millerdev millerdev commented Mar 20, 2025

Version bump for Python 3.13 compatibility. Reviewed release notes and recent issues. This bug is the reason that this PR is not going all the way to v5.5 just yet. Given the greenness of v5.5 (released days ago), it seems prudent to wait for that to stabilize (major change: drop pycurl in favor of urllib3), even though 5.4.2 does not officially support Python 3.13. Hopefully a better solution will be available before we upgrade our production environments to Python 3.13.

There is a question about whether Celery should also be updated to 5.3 or 5.4, both of which are also in the state of not officially supporting Python 3.13 (coming in Celery 5.5).

Safety Assurance

Safety story

kombu is a dependency of celery, and not used directly by HQ code. Planning to test on staging.

Automated test coverage

Not directly.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

@millerdev millerdev added the product/invisible Change has no end-user visible impact label Mar 20, 2025
@dimagimon dimagimon added the dependencies Pull requests that update a dependency file label Mar 20, 2025
Copy link
Contributor

@AmitPhulera AmitPhulera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. Once confirmed that celery on staging is working it's GTD to me.

@millerdev millerdev marked this pull request as ready for review March 25, 2025 13:05
@millerdev millerdev requested a review from a team as a code owner March 25, 2025 13:05
@millerdev
Copy link
Contributor Author

Confirmed that Celery on staging looks good since this was deployed there.

@millerdev millerdev merged commit af17658 into master Mar 25, 2025
13 checks passed
@millerdev millerdev deleted the dm/upgrade-kombu branch March 25, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file product/invisible Change has no end-user visible impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants