Project

Profile

Help

Refactor #2963

closed

Use the new lower case celery config settings

Added by daviddavis almost 5 years ago. Updated over 2 years 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

Actions #1

Updated by daviddavis almost 5 years ago

  • Description updated (diff)
Actions #2

Updated by daviddavis almost 5 years ago

  • Tags Easy Fix added
Actions #3

Updated by ttereshc almost 5 years ago

  • Tracker changed from Issue to Refactor
  • % Done set to 0
Actions #4

Updated by jwelborn over 4 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?

Actions #5

Updated by bizhang over 4 years ago

Actions #6

Updated by jwelborn over 4 years ago

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

Actions #7

Updated by bizhang over 4 years ago

Actions #8

Updated by jwelborn over 4 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?

Actions #9

Updated by bizhang over 4 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.

Actions #10

Updated by jwelborn over 4 years ago

  • Assignee set to jwelborn

PR incoming!

Actions #11

Updated by bizhang over 4 years ago

  • Status changed from NEW to POST

Added by werwty over 4 years ago

Revision 83aa043a

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

updated celery conf names

Actions #12

Updated by bizhang over 4 years ago

  • Status changed from POST to MODIFIED
Actions #13

Updated by daviddavis about 3 years ago

  • Sprint/Milestone set to 3.0.0
Actions #14

Updated by bmbouter about 3 years ago

  • Tags deleted (Pulp 3)
Actions #15

Updated by bmbouter over 2 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF