Skip to content

added --range-prefix option to celery multi#6180

Merged
auvipy merged 3 commits into
celery:masterfrom
art-vasilyev:multi-range-prefix
Jun 20, 2020
Merged

added --range-prefix option to celery multi#6180
auvipy merged 3 commits into
celery:masterfrom
art-vasilyev:multi-range-prefix

Conversation

@art-vasilyev

@art-vasilyev art-vasilyev commented Jun 20, 2020

Copy link
Copy Markdown
Contributor

Note: Before submitting this pull request, please review our contributing
guidelines.

Description

Currently when running multiple celery workers by celery multi command and providing workers as range number, multi uses default range prefix "celery". Added option --range-prefix for overriding default range prefix.

@art-vasilyev
added --range-prefix option to celery multi
f44c54e 
Added option for overriding default range prefix when running
multiple workers prividing range with `celery multy` command.
@lgtm-com

lgtm-com Bot commented Jun 20, 2020

Copy link
Copy Markdown

This pull request introduces 2 alerts and fixes 2 when merging f44c54e into 1a0ca99 - view on LGTM.com

new alerts:

  • 1 for Non-exception in 'except' clause
  • 1 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Non-exception in 'except' clause
  • 1 for Wrong number of arguments in a call

auvipy
auvipy requested changes Jun 20, 2020
View reviewed changes
Comment thread celery/apps/multi.py
@codecov

codecov Bot commented Jun 20, 2020 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #6180 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6180   +/-   ##
=======================================
  Coverage   83.57%   83.57%           
=======================================
  Files         145      145           
  Lines       17365    17366    +1     
  Branches     2156     2156           
=======================================
+ Hits        14513    14514    +1     
  Misses       2631     2631           
  Partials      221      221
Impacted Files Coverage Δ
celery/bin/multi.py 100.00% <ø> (ø)
celery/apps/multi.py 84.45% <100.00%> (+0.04%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a0ca99...495e28e. Read the comment docs.

@lgtm-com

lgtm-com Bot commented Jun 20, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert and fixes 1 when merging 495e28e into 1a0ca99 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Wrong number of arguments in a call

@auvipy auvipy added this to the 4.4.x milestone Jun 20, 2020
@auvipy
auvipy merged commit 29444f1 into celery:master Jun 20, 2020
auvipy added a commit that referenced this pull request Jul 8, 2020
@thedrow @verhovsky
@auvipy @jainal09 @gsfish @dipAch @spengjie @chris-griffin @mhewedy @phated @tamsanh @eriktews @anilaratna2 @wyguy444 @galCohen88 @alfred-sa @paramkpr @sveniu @safwanrahman Aissaoui Anouar @qdzzyb2014 @Alireza2n @marcosmoyano @shenek @AndrewPix @fladi @dejlek @yannickschuchmann @matteius @tirkarthi @bernd-wechner @soldag @uddmorningsun @marfgold1 @Wooden-Robot @imomaliev @AlexRiina @xncbf @nixphix @martey @Seleznev-nvkz @Prodge @adyouri @Ixiodor @abhishekakamai @allanlei @M1ha-Shvn @hicay @michamos @chrisburr @tildedave @the-owl Michael Gaddis @epwalsh @TalRoni @lpsinger @neonbunny Abhishek @theirix @yukihira1992 @jpays-cnp @gward Alexa Griffith @heedong-jung @heedongsamsung @shreyanshk @georgedorn @Alphadelta14 @azimjohn @ysde @amirzia @nadflinn @nezhar @specialorange @aeijdenberg rafaelreuber @coderanger @clems4ever Mathieu Chataigner @eugeneyalansky @leonardsocialcodeinc @xinydev @watercrossing @hanchau shaoziwei @mchataigner @Anakael @ckdanny @Tirzono @dtilloy-criteo @anthonynsimon @lironhl @Darkheir @Kirade @singlaive Murphy Meng @whtsky @kwist-sgr @ldragan @nijel @radimsuckr @art-vasilyev @kakakikikeke-fork @baixuexue123 baixue @prashnts @AbdealiLoKo
Remove PY3 conditionals (#5954