Project

Profile

Help

Issue #2787

closed

Fatal exceptions from tasks not being reported by tasks API

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

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

Description

The Task model has an error field. It is a JSON field. After a task succeeds or fails, this field is supposed to contain a dictionary representing any fatal exception encountered during the task. If no fatal exception is encountered the field should be set to None. This happens here[0] in the code, but the API for tasks does not report an 'error' field.

[0] https://github.com/pulp/pulp/blob/3.0-dev/platform/pulp/tasking/tasks.py#L322

Actions #1

Updated by dkliban@redhat.com over 7 years ago

  • Tracker changed from Story to Issue
  • Subject changed from Record fatal exceptions from tasks in 'error' field of Task object to Fatal exceptions from tasks not being reported by tasks API
  • Description updated (diff)
  • Severity set to 2. Medium
  • Triaged set to No

Added by dkliban@redhat.com over 7 years ago

Revision 12d36be1 | View on GitHub

Problem: tasks API ommits error field in response

Solution: update the serializer for Task model by renaming 'result' field to 'error'

closes #2787

Added by dkliban@redhat.com over 7 years ago

Revision 12d36be1 | View on GitHub

Problem: tasks API ommits error field in response

Solution: update the serializer for Task model by renaming 'result' field to 'error'

closes #2787

Actions #2

Updated by dkliban@redhat.com over 7 years ago

  • Status changed from NEW to MODIFIED
Actions #4

Updated by daviddavis over 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #5

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #6

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF