Project

Profile

Help

Issue #1704

Download task marked as succesful when download shows as failed

Added by dkliban@redhat.com almost 5 years ago. Updated over 1 year ago.

Status:
CLOSED - WONTFIX
Priority:
High
Assignee:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

[jenkins@rhel7-vanilla-np-qeos-74379 ~]$ pulp-admin repo download --repo-id zoo
+----------------------------------------------------------------------+
                      Downloading Repository [zoo]
+----------------------------------------------------------------------+

This command may be exited via ctrl+c without affecting the request.

Download Repository Content
[==================================================] 100%
32 of 32 items
... failed

Task Succeeded

History

#1 Updated by bmbouter almost 5 years ago

Consider that the task itself did complete, but units it was downloading failed. What happens if just 1 units fails? Is "all units" a special enough case to handle specially?

#2 Updated by mhrivnak over 4 years ago

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

#3 Updated by mhrivnak over 4 years ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (mhrivnak)

The consensus is to continue with the established convention that a download failure does not mark the whole task as failed. We should re-visit task statuses as they pertain to the work done in the task in pulp 3.

#4 Updated by mhrivnak over 4 years ago

  • Priority changed from Normal to High
  • Severity changed from 2. Medium to 1. Low
  • Triaged changed from No to Yes
  • Tags Easy Fix added

#5 Updated by bmbouter over 1 year ago

  • Status changed from NEW to CLOSED - WONTFIX

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

#6 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF