Project

Profile

Help

Refactor #2963

Use the new lower case celery config settings

Added by daviddavis over 3 years ago. Updated about 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Low
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix
Sprint:
Quarter:

Description

Celery 4 introduces new config settings with better naming. More info:

http://docs.celeryproject.org/en/latest/userguide/configuration.html#new-lowercase-settings

Associated revisions

Revision 83aa043a View on GitHub
Added by werwty over 3 years ago

Merge pull request #3130 from JayWelborn/update-celery-conf-names

updated celery conf names

History

#1 Updated by daviddavis over 3 years ago

  • Description updated (diff)

#2 Updated by daviddavis over 3 years ago

  • Tags Easy Fix added

#3 Updated by ttereshc over 3 years ago

  • Tracker changed from Issue to Refactor
  • % Done set to 0

#4 Updated by jwelborn over 3 years ago

Hi, I found this issue browsing looking for easy fixes.

Is this just changing the names here to the lowercase celery 4 names?

If so, can I help?

#5 Updated by bizhang over 3 years ago

#6 Updated by jwelborn over 3 years ago

Since this is a pulp 3 fix, should I pull against 3.0-dev?

#7 Updated by bizhang over 3 years ago

#8 Updated by jwelborn over 3 years ago

Just to make sure I'm not oversimplifying this:

All that's needed is changing the UPPERCASE_NAMES in celery_instance.py to the lowercase_names in the celery 4.x conf docs?

#9 Updated by bizhang over 3 years ago

jwelborn yep! Since celery will complain if there's a mix of old and new config settings names, you can be sure that everything works if it doesn't complain after you restart the pulp services by running prestart and pstatus on a vagrant environment.

#10 Updated by jwelborn over 3 years ago

  • Assignee set to jwelborn

PR incoming!

#11 Updated by bizhang over 3 years ago

  • Status changed from NEW to POST

#12 Updated by bizhang over 3 years ago

  • Status changed from POST to MODIFIED

#13 Updated by daviddavis almost 2 years ago

  • Sprint/Milestone set to 3.0.0

#14 Updated by bmbouter almost 2 years ago

  • Tags deleted (Pulp 3)

#15 Updated by bmbouter about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF