[ckan-changes] [ckan/ckan] ee248d: Allow overriding celery list params

GitHub noreply at github.com
Thu Nov 9 11:40:28 UTC 2017


  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: ee248d24defeaac0ff9f31cf8060cd9b4ce6cca4
      https://github.com/ckan/ckan/commit/ee248d24defeaac0ff9f31cf8060cd9b4ce6cca4
  Author: Philippe Pepiot <philippe.pepiot at logilab.fr>
  Date:   2017-11-07 (Tue, 07 Nov 2017)

  Changed paths:
    M ckan/lib/celery_app.py

  Log Message:
  -----------
  Allow overriding celery list params

The condition 'if key in LIST_PARAMS' was never satisfied because config
keys are lower case and LIST_PARAMS is upper case.

This allow overriding the CELERY_IMPORTS, ADMINS and ROUTES in ckan
config.

Also overriding list params should extend existings values (i.e. for
CELERY_IMPORTS)


  Commit: 3198e629477f6955b0521c701b932853c6a0d80b
      https://github.com/ckan/ckan/commit/3198e629477f6955b0521c701b932853c6a0d80b
  Author: Sergey <tonnydexter at gmail.com>
  Date:   2017-11-09 (Thu, 09 Nov 2017)

  Changed paths:
    M ckan/lib/celery_app.py

  Log Message:
  -----------
  Merge pull request #3909 from philpep/override-celery-imports

Allow overriding celery list params


Compare: https://github.com/ckan/ckan/compare/4979403eff36...3198e629477f


More information about the ckan-changes mailing list