Skip to content

Commit

Permalink
Restore the celery worker --without-{gossip,mingle,heartbeat} flags (c…
Browse files Browse the repository at this point in the history
…elery#6365)

In the previously used argparse arguments framework, these three options were
used as flags.

Since 5.0.0, they are options which need to take an argument (whose only
sensible value would be "true"). The error message coming up is also (very)
hard to understand, when running the celery worker command with an odd number
of flags:

  Error: Unable to parse extra configuration from command line.
  Reason: not enough values to unpack (expected 2, got 1)

When the celery worker is run with an even number of flags, the last one is
considered as an argument of the previous one, which is a subtle bug.
  • Loading branch information
olasd authored and jeyrce committed Aug 25, 2021
1 parent e7bde84 commit 62cfd66
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions celery/bin/worker.py
Expand Up @@ -231,15 +231,15 @@ def detach(path, argv, logfile=None, pidfile=None, uid=None,
cls=CeleryOption,
help_group="Queue Options")
@click.option('--without-gossip',
default=False,
is_flag=True,
cls=CeleryOption,
help_group="Features")
@click.option('--without-mingle',
default=False,
is_flag=True,
cls=CeleryOption,
help_group="Features")
@click.option('--without-heartbeat',
default=False,
is_flag=True,
cls=CeleryOption,
help_group="Features", )
@click.option('--heartbeat-interval',
Expand Down

0 comments on commit 62cfd66

Please sign in to comment.