Project

Profile

Help

Issue #7980

The state for canceled tasks is set to failed

Added by daviddavis 10 months ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 93
Quarter:

Description

This started with the release of pulpcore 3.9. When a task is canceled, its state is 'failed' instead of 'canceled'.

Steps to reproduce

remote_href=$(http :/pulp/api/v3/remotes/file/file/ name=$RANDOM url=https://fixtures.pulpproject.org/file-perf/PULP_MANIFEST | jq -r '.pulp_href')
repo_href=$(http :/pulp/api/v3/repositories/file/file/ name=$RANDOM remote=$remote_href | jq -r '.pulp_href')
task_href=$(http POST :${repo_href}sync/ | jq -r '.task')
echo $task_href
http PATCH :${task_href} state=canceled
http :$task_href | jq -r '.state'

This will out put failed when it should output canceled. I noticed that the task response from the cancel request correctly shows canceled.


Related issues

Has duplicate Pulp - Issue #8143: Cancelling a task logs a failure message before logging the cancel messageCLOSED - DUPLICATE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision c672962e View on GitHub
Added by dalley 7 months ago

Fix another scenario where canceled tasks could be marked failed

closes: #7980 https://pulp.plan.io/issues/7980

History

#3 Updated by dalley 10 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dalley

#4 Updated by daviddavis 10 months ago

Once this issue is resolved, we'll need to revert https://github.com/pulp/pulp-cli/pull/41.

#5 Updated by fao89 10 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 87

#6 Updated by rchan 10 months ago

  • Sprint changed from Sprint 87 to Sprint 88

#7 Updated by rchan 9 months ago

  • Sprint changed from Sprint 88 to Sprint 89

#8 Updated by rchan 8 months ago

  • Sprint changed from Sprint 89 to Sprint 90

#9 Updated by rchan 8 months ago

  • Sprint changed from Sprint 90 to Sprint 91

#10 Updated by rchan 7 months ago

  • Sprint changed from Sprint 91 to Sprint 92

#11 Updated by rchan 7 months ago

  • Sprint changed from Sprint 92 to Sprint 93

#12 Updated by dalley 7 months ago

  • Has duplicate Issue #8143: Cancelling a task logs a failure message before logging the cancel message added

#13 Updated by dalley 7 months ago

  • Status changed from ASSIGNED to POST

#14 Updated by dalley 7 months ago

  • Status changed from POST to MODIFIED

#15 Updated by mdellweg 6 months ago

  • Sprint/Milestone set to 3.12.0

#16 Updated by pulpbot 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF