Project

Profile

Help

Issue #7980

closed

The state for canceled tasks is set to failed

Added by daviddavis almost 4 years ago. Updated over 3 years 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 - DUPLICATEdalleyActions
Actions #3

Updated by dalley almost 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dalley
Actions #4

Updated by daviddavis almost 4 years ago

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

Actions #5

Updated by fao89 almost 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 87
Actions #6

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 87 to Sprint 88
Actions #7

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 88 to Sprint 89
Actions #8

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 89 to Sprint 90
Actions #9

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 90 to Sprint 91
Actions #10

Updated by rchan over 3 years ago

  • Sprint changed from Sprint 91 to Sprint 92
Actions #11

Updated by rchan over 3 years ago

  • Sprint changed from Sprint 92 to Sprint 93
Actions #12

Updated by dalley over 3 years ago

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

Updated by dalley over 3 years ago

  • Status changed from ASSIGNED to POST

Added by dalley over 3 years ago

Revision c672962e | View on GitHub

Fix another scenario where canceled tasks could be marked failed

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

Actions #14

Updated by dalley over 3 years ago

  • Status changed from POST to MODIFIED
Actions #15

Updated by mdellweg over 3 years ago

  • Sprint/Milestone set to 3.12.0
Actions #16

Updated by pulpbot over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF