search

mitodl / lore

Learning Object Repository for Education
https://goo.gl/nDVx4D
GNU Affero General Public License v3.0
16 stars 6 forks source link

Replaced the status app with django-server-status #894

Closed giocalitri closed 8 years ago

giocalitri commented 8 years ago

What are the relevant tickets?

fixes #889

What's this PR do?

replaces the built in status app with the django-server-status library

Where should the reviewer start?

settings.py

How should this be manually tested?

rebuild the container and check that the new response from the status contain the keys celery and status_all

What gif best describes this PR or how it makes you feel?

noisecapella commented 8 years ago

This generally looks good, although the celery portion of the app isn't working because there's no USE_CELERY setting defined. Is that a blocker?

giocalitri commented 8 years ago

This generally looks good, although the celery portion of the app isn't working because there's no USE_CELERY setting defined. Is that a blocker?

Probably it is: I will look into it.

giocalitri commented 8 years ago

@noisecapella: Now everything works.

@pdpinch this needs attention before releasing to production to check if everything works fine for the imports and everything that uses celery in general.

noisecapella commented 8 years ago

Looks good now :+1: