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

Remove pid arg from celery option to fix duplicate pid issue, Move celery command to provider package #36794

Merged
merged 25 commits into from
Feb 25, 2024
Merged
Changes from 6 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
739f9e3
reset stats instance in daemon context
pavansharma36 Oct 25, 2023
3a4ef10
Merge branch 'main' into main
pavansharma36 Oct 25, 2023
242b8aa
Merge branch 'main' into main
pavansharma36 Oct 25, 2023
d40994c
Fix for CI
pavansharma36 Oct 25, 2023
bd97ae1
Merge branch 'apache:main' into main
pavansharma36 Jan 15, 2024
c0d98aa
add celery pid suffix to fix duplicate pid file error
pavansharma36 Jan 15, 2024
c1b213f
make copy of celery command in provider
pavansharma36 Jan 25, 2024
3cb52a2
added deprecated
pavansharma36 Jan 25, 2024
a62aa5d
removed celery pid arg as its handled from daemon context
pavansharma36 Jan 26, 2024
df928da
cleanup
pavansharma36 Jan 26, 2024
7d1e449
Merge branch 'main' into main
pavansharma36 Jan 26, 2024
58659fc
fix static check
pavansharma36 Jan 30, 2024
bd097f4
fix test case on removal of pid
pavansharma36 Jan 30, 2024
cd30cf8
copied test also
pavansharma36 Jan 30, 2024
3a7c77c
fix static check
pavansharma36 Jan 31, 2024
69539e7
Merge branch 'main' into main
pavansharma36 Jan 31, 2024
85776f1
fix test cases
pavansharma36 Feb 1, 2024
3995910
copy daemon utils func for backward compaitibility
pavansharma36 Feb 1, 2024
68c0efd
lazy import new func
pavansharma36 Feb 2, 2024
5ba587b
wrapped providers_configuration_loaded to fix compaitibility
pavansharma36 Feb 2, 2024
c6d9ed9
Merge branch 'main' into main
pavansharma36 Feb 3, 2024
9a14d36
update deprecated msg with info
pavansharma36 Feb 8, 2024
b019557
added warning at module level
pavansharma36 Feb 25, 2024
ad1608c
use warnings
pavansharma36 Feb 25, 2024
95fa468
Merge branch 'main' into main
pavansharma36 Feb 25, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion airflow/cli/commands/celery_command.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ def worker(args):
"--loglevel",
celery_log_level,
"--pidfile",
worker_pid_file_path,
worker_pid_file_path + ".celery.pid",
potiuk marked this conversation as resolved.
Show resolved Hide resolved
]
if autoscale:
options.extend(["--autoscale", autoscale])
Expand Down
Loading