Actions
Issue #4559
closed
Cancelling a task logs a failure message before logging the cancel message
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:
Description
Mar 21 14:04:47 pulp3-source-fedora29.localhost.example.com rq[10066]: pulp: rq.worker:INFO: Cleaning registries for queue: reserved-resource-worker-1@pulp3-source-fedora29.localhost.example.com
Mar 21 14:04:47 pulp3-source-fedora29.localhost.example.com rq[10066]: pulp: rq.worker:INFO: reserved-resource-worker-1@pulp3-source-fedora29.localhost.example.com: pulpcore.tasking.tasks._release_resources('f8b2be25-a8c1-4995-9f62-99cbe0df961d') (c3257770-db36-4d6d-a6a9-8cf47f5e0431)
Mar 21 14:04:47 pulp3-source-fedora29.localhost.example.com rq[10066]: pulp: pulpcore.tasking.tasks:ERROR: The task f8b2be25-a8c1-4995-9f62-99cbe0df961d exited immediately for some reason. Marking as failed. Check the logs for more details
Mar 21 14:04:47 pulp3-source-fedora29.localhost.example.com rq[10066]: pulp: rq.worker:INFO: reserved-resource-worker-1@pulp3-source-fedora29.localhost.example.com: Job OK (c3257770-db36-4d6d-a6a9-8cf47f5e0431)
Mar 21 14:04:48 pulp3-source-fedora29.localhost.example.com gunicorn[10069]: pulp: pulpcore.tasking.util:INFO: Task canceled: f8b2be25-a8c1-4995-9f62-99cbe0df961d.
This message should not be logged, since the task was cancelled, not failed.
"pulp: pulpcore.tasking.tasks:ERROR: The task f8b2be25-a8c1-4995-9f62-99cbe0df961d exited immediately for some reason. Marking as failed. Check the logs for more details"
The task status is correct, e.g. "cancelled", not failed.
Related issues
Actions
.
Get rid of failure message on task cancellation
_release_resources() task checks whether task status is still set to "running". In the case of cancelation, that is still true if the transaction hasn't been committed yet. We need to set the task status to 'canceled' beforehand so that we kill the workhorse and the resource cleanup task runs, it doesn't think the workhorse died erroneously.
closes: #4559 https://pulp.plan.io/issues/4559