Bug 1094523 - [RFE] Add new task state: cancel_requested
Summary: [RFE] Add new task state: cancel_requested
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Pulp
Classification: Retired
Component: async/tasks
Version: Master
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: pulp-bugs
QA Contact: pulp-qe-list
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-05-05 21:56 UTC by Randy Barlow
Modified: 2015-02-28 23:41 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-02-19 01:13:11 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Pulp Redmine 229 0 None None None Never

Description Randy Barlow 2014-05-05 21:56:23 UTC
I think it would be a worthwhile enhancement to distinguish between cancel_requested and canceled. The REST API could mark a task as cancel_requested, and once the worker successfully handles the request it can mark the task as canceled.

This would be handy for the CLI. The CLI currently exits immediately when polling tasks upon noticing that they are canceled. It could instead mark that the task has been requested to cancel, and wait to exit until it is actually exited. This would allow it to print the final progress report if there is one.

This would have aided in discovering the problem in #1082802[0], wherein it was found that cancellation was ignored.

[0] https://bugzilla.redhat.com/show_bug.cgi?id=1082802

Comment 1 Brian Bouterse 2015-02-19 01:13:11 UTC
Moved to https://pulp.plan.io/issues/229


Note You need to log in before you can comment on or make changes to this bug.