search

kartoza / geosafe

InaSAFE package for Geonode
GNU General Public License v3.0
7 stars 16 forks source link

Implement task state tracking #383

Open lucernae opened 6 years ago

lucernae commented 6 years ago

We have to know why a task is failed and at which steps.

gubuntu commented 6 years ago

same as #264?

lucernae commented 5 years ago

Proposal that I can think of now:

  1. Analysis model should track the current task result state (done) and stacktrace if failed (exception type, trace messages, error message summary)

  2. Check failed task

    a. Should implement periodic task check again to check task state. This is because if a task is failed, we can't execute callback to GeoNode.

    b. Should implement REST API for task failure callback. This might be more complicated than 2a, because we should implement oauth for communication now. Because the callback will not be via celery. But this is the ideal solution and what we should aim for. We could implement 2a first and use some of the logic to implement 2b.

  3. We should be able to query/retrieve a task progress. This needs fundamental changes in InaSAFE headless. We can't query a running task and asked for intrinsic variable values, so we had to ask Headless to update the progress periodically. This can only happen if InaSAFE supports this. We're going to use analysis progress that is being updated from Qt form in InaSAFE Desktop. Not sure how to achieve this. So, I'll probably need some time for tinkering.

  4. For the web app to query task progress to GeoSAFE, we had to enable web socket. Or plain ajax alternatively, if web socket were not available. So, it's best to start from AJAX first.

Will add more if I had additional proposals in mind.