Fix concurrent updates
What does this MR do?
In some cases, if PatchTrace takes ages,
we might send PUT with success status.
As a touchRunning we should only send running state.
The final state should be sent only as a last, last, last
request of finish().
This makes it explicit when we touch, and when we push a status update.
Why was this MR needed?
Are there points in the code the reviewer needs to double check?
Does this MR meet the acceptance criteria?
-
Documentation created/updated -
Added tests for this feature/bug -
In case of conflicts with master- branch was rebased
What are the relevant issue numbers?
Edited by Steve Xuereb