Project

Profile

Help

Refactor #3113

closed

Update Artifact instantiation code to use paths to files

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

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

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
No
Tags:
Sprint:
Sprint 28
Quarter:

Description

The Artifact model can be instantiated using an open file or a path to a file on disk. The code in the ChangeSet needs to be updated to stop opening files before instantiating the Artifact. Both of the importers in the example plugin currently open a file before instantiating an Artifact. Both importers need to stop doing this and simply pass in a path to the file when instantiating an Artifact.

The ArtifactFileFiled needs to be updated to open the file in binary mode[0].

[0] https://github.com/pulp/pulp/blob/15857fb0831c0998219a32e8d6ba52abdba20888/platform/pulpcore/app/models/fields.py#L26

Actions #1

Updated by daviddavis about 7 years ago

  • Groomed changed from No to Yes
Actions #2

Updated by mhrivnak about 7 years ago

  • Sprint/Milestone set to 47
Actions #3

Updated by dkliban@redhat.com about 7 years ago

  • Description updated (diff)
Actions #4

Updated by ttereshc about 7 years ago

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

Added by ttereshc about 7 years ago

Revision a1f85a46 | View on GitHub

Specify a path instead of an open file for Artifact instantiation

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

Added by ttereshc about 7 years ago

Revision a1f85a46 | View on GitHub

Specify a path instead of an open file for Artifact instantiation

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

Actions #6

Updated by ttereshc about 7 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #7

Updated by bmbouter almost 7 years ago

  • Sprint set to Sprint 28
Actions #8

Updated by bmbouter almost 7 years ago

  • Sprint/Milestone deleted (47)
Actions #9

Updated by daviddavis over 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #10

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #11

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF