Name |
Last commit
|
Last update |
---|---|---|
.. | ||
djangoapps | ||
envs | ||
lib | ||
static | ||
templates | ||
tests | ||
README.rst | ||
__init__.py | ||
celery.py | ||
startup.py | ||
urls.py | ||
urls_dev.py | ||
wsgi.py |
The print_out_all_courses() routine consumes a ton of memory (2G and causes noticable mongo usage spikes). This actually causes other processes on production boxes to be memory starved and killed (such as worker children on edge when this was run recently). The behavior of this script on production is * Print several hundred courses * Ask if you want to delete the one you specified * print several hundred courses minus one On a sandbox with 5 courses, you could tell by eye that 1 is gone, but not in production (or even in stage). The original PLAT-619 ticket for this suggested printing a course listing on error, but instead it always printed the course listing. Even in the error case, hundreds of course ids is confusing and obscures the error message saying that your course_id is invalid. You should be getting the course id from the UI or from ./manage.py lms dump_course_ids, not by searching a list. Adjusted the test accordingly Remove get_courses_keys
Name |
Last commit
|
Last update |
---|---|---|
.. | ||
djangoapps | Loading commit data... | |
envs | Loading commit data... | |
lib | Loading commit data... | |
static | Loading commit data... | |
templates | Loading commit data... | |
tests | Loading commit data... | |
README.rst | Loading commit data... | |
__init__.py | Loading commit data... | |
celery.py | Loading commit data... | |
startup.py | Loading commit data... | |
urls.py | Loading commit data... | |
urls_dev.py | Loading commit data... | |
wsgi.py | Loading commit data... |