Project

Profile

Help

Task #2675

Rename 'results' field to 'error' for the Task document

Added by bmbouter over 2 years ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 18

Description

This is the state of the model at the time of this writing: https://github.com/pulp/pulp/blob/3fe96dcbf48a59613df3494d838f6c3240fb5b52/app/pulp/app/models/task.py#L155

The 'result' field currently stores the return value from a task. That occurs here It also stores a fatal exception here.

This would rename the field as described in the title and update the associated helper methods: update set_failed() and set_completed().


Checklist

Associated revisions

Revision cda52cf0 View on GitHub
Added by dalley over 2 years ago

Rename 'result' field on the Task model to 'error'.

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

Revision cda52cf0 View on GitHub
Added by dalley over 2 years ago

Rename 'result' field on the Task model to 'error'.

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

Revision cda52cf0 View on GitHub
Added by dalley over 2 years ago

Rename 'result' field on the Task model to 'error'.

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

History

#1 Updated by amacdona@redhat.com over 2 years ago

  • Groomed changed from No to Yes

#2 Updated by mhrivnak over 2 years ago

  • Sprint Candidate changed from No to Yes

#3 Updated by mhrivnak over 2 years ago

  • Sprint/Milestone set to 37

#4 Updated by dalley over 2 years ago

  • Assignee set to dalley

#5 Updated by ipanova@redhat.com over 2 years ago

  • Status changed from NEW to ASSIGNED

#6 Updated by dalley over 2 years ago

  • Status changed from ASSIGNED to MODIFIED
  • % Done changed from 0 to 100

#7 Updated by bmbouter over 1 year ago

  • Sprint set to Sprint 18

#8 Updated by bmbouter over 1 year ago

  • Sprint/Milestone deleted (37)

#9 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#10 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF