Resolve "Add overview page for celery task results"

Closes #223 (closed)

added type::feature workflow::discussing labels

added 1 commit

• 8b943148 - Show celery task results on system status page

Compare with previous version

assigned to @hansegucker

added 45m of time spent at 2020-05-22

I thought we can show for every celery task when it was executed last time and what was its status.

added 1 commit

• e51f4a7a - Show celery task results on system status page

Compare with previous version

resolved all threads

Please review again…

removed time spent

If celery is not running, the following happens:

Traceback (most recent call last):
  File "/home/wethjo/.cache/pypoetry/virtualenvs/aleksis-9WSQE4wq-py3.8/lib/python3.8/site-packages/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/home/wethjo/.cache/pypoetry/virtualenvs/aleksis-9WSQE4wq-py3.8/lib/python3.8/site-packages/django/core/handlers/base.py", line 115, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/home/wethjo/.cache/pypoetry/virtualenvs/aleksis-9WSQE4wq-py3.8/lib/python3.8/site-packages/django/core/handlers/base.py", line 113, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/lib/python3.8/contextlib.py", line 75, in inner
    return func(*args, **kwds)
  File "/home/wethjo/.cache/pypoetry/virtualenvs/aleksis-9WSQE4wq-py3.8/lib/python3.8/site-packages/rules/contrib/views.py", line 221, in _wrapped_view
    return view_func(request, *args, **kwargs)
  File "/home/wethjo/dev/AlekSIS/aleksis/core/views.py", line 303, in system_status
    job_list = list(inspect().registered_tasks().values())[0]

Exception Type: AttributeError at /status/
Exception Value: 'NoneType' object has no attribute 'values'

Tom and I discussed whether we wanted to display only tasks that already ran and yielded a result at least once, or all registered tasks (because the first would mean to just query the TaskResult model, the latter introspecting celery).

added 8 commits

• e51f4a7a...aa288c4e - 7 commits from branch master
• 00dec64f - Merge branch 'master' into 223-add-overview-page-for-celery-task-results

Compare with previous version

added 1 commit

• b27f2321 - Check if registered_tasks() is None

Compare with previous version

unmarked as a Work In Progress

resolved all threads

Seems to be fine. I would suggest to do a last review on Dev meeting next week together with the rest.

added 1 deleted label