Skip to content

Update min version of Celery library to 5.5.0 - #43777

Merged
potiuk merged 1 commit into
apache:mainfrom
eladkal:celery
Apr 1, 2025
Merged

Update min version of Celery library to 5.5.0#43777
potiuk merged 1 commit into
apache:mainfrom
eladkal:celery

Conversation

@eladkal

@eladkaleladkal commented Nov 7, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

Closes: #41359
Closes: #26542

I'm not sure if it fixes the problem described but it prevents other related bugs associated with the older versions

Celery 5.5.0 hasn't been released yet.
https://pypi.org/project/celery/#history
waiting for it has been released

@eladkal
eladkal marked this pull request as draft November 7, 2024 10:19
@potiuk

potiuk commented Nov 9, 2024

Copy link
Copy Markdown
Member

nice

@github-actions

github-actionsBot commented Dec 25, 2024

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actionsgithub-actionsBot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Dec 25, 2024
@eladkaleladkal removed the stale Stale PRs per the .github/workflows/stale.yml policy file label Dec 25, 2024
@eladkaleladkal mentioned this pull request Jan 23, 2025
Closed
2 tasks
@github-actions

github-actionsBot commented Mar 10, 2025

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actionsgithub-actionsBot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Mar 10, 2025
@eladkaleladkal removed the stale Stale PRs per the .github/workflows/stale.yml policy file label Mar 10, 2025
@eladkal
eladkal marked this pull request as ready for review April 1, 2025 06:37
@potiuk

potiuk commented Apr 1, 2025

Copy link
Copy Markdown
Member

Still the same stability tests we alredy know about !

@potiuk

potiuk commented Apr 1, 2025

Copy link
Copy Markdown
Member

BTW. Since this is celery and we already know that celery executor is not 100% tested -> that one will need at the very least running airflow with CeleryExecutor in breeze successfully. Just "tests passing" is not enough.

potiuk
potiuk reviewed Apr 1, 2025
View reviewed changes
Comment threadproviders/celery/pyproject.toml
@eladkal
eladkal requested a review from potiukApril 1, 2025 12:40
@potiuk
potiuk merged commit 836f869 into apache:mainApr 1, 2025
@alex-trm

alex-trm commented Apr 1, 2025

Copy link
Copy Markdown

Yesss love it! They finally got it released! Only took 9 mo haha

@eladkal
eladkal deleted the celery branch April 1, 2025 17:23
aaron-y-chen pushed a commit to aaron-y-chen/airflow that referenced this pull request Apr 4, 2025
@eladkal@aaron-y-chen
Update min version of Celery library to 5.5.0 (apache#43777)
2863c1c
@eladkaleladkal mentioned this pull request Apr 6, 2025
Closed
diogotrodrigues pushed a commit to diogotrodrigues/airflow that referenced this pull request Apr 6, 2025
@eladkal@diogotrodrigues
Update min version of Celery library to 5.5.0 (apache#43777)
0e0272a
simonprydden pushed a commit to simonprydden/airflow that referenced this pull request Apr 8, 2025
@eladkalSimon Prydden
Update min version of Celery library to 5.5.0 (apache#43777)
dfa388e
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kaxilkaxilkaxil approved these changes
@hussein-awalahussein-awalahussein-awala approved these changes
@romsharon98romsharon98Awaiting requested review from romsharon98
@potiukpotiukAwaiting requested review from potiuk

Assignees

No one assigned

Labels

area:providersprovider:celery

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Race condition Airflow's Celery executor timeout and import redis leave a broken import Celery worker enters a catatonic state after redis restart

5 participants

@eladkal@potiuk@alex-trm@kaxil@hussein-awala