Actions
Story #6749
closedAs a core or plugin developer, i want to be able to share temporary files between the viewset and triggered tasks
Start date:
Due date:
% Done:
100%
Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:
Description
Current Problems¶
- Deleting the temporary Artifact can cause a race condition where the second fails or no deletes occur
- FK exception when the Artifact being deleted already exists and is associated with other content
- Backups would incorrectly include both long-term Artifacts and short-term files.
- Security concern that these temporary files are exposed to users as Artifacts and can be associated to Content, etc
Background¶
Sometimes, files must be brought forward from the viewset to the executing task, that are not meant to end up being artifacts in the end. On the other hand, different pulp services are not guaranteed to share a common filesystem (like /usr/share/pulp
).
Proposal¶
The design is to have a service that allows to create files with the same storage technology that the artifacts use, where unique names are generated, and the task to use the file has the duty to remove it again from the storage. It is necessary, that those files do neither collide with any artifact that happens to be identical, nor another such file created for a different task.
The Plan¶
- Create a new model called
PulpTemporaryFile
that has apulp_id
and afile
. - Files should be saved based on pulp_id (eg
MEDIA_ROOT/tmp/files/29/beb7f3-1a70-40da-86fc-5f9006670baa
) - Add PulpTemporaryFile to the plugin api and add documentation
Related issues
Updated by fao89 over 4 years ago
- Related to Task #6718: Stop using artifacts when importing collections added
Updated by fao89 over 4 years ago
- Related to Issue #5666: Failed imports leave orphaned artifacts blocking subsequent attempts added
Updated by fao89 over 4 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to fao89
Updated by pulpbot over 4 years ago
- Status changed from ASSIGNED to POST
Added by Fabricio Aguiar over 4 years ago
Updated by Anonymous over 4 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulpcore|6b3be3574360af559b85daec44bb2a14cebc2ee7.
Updated by lmjachky over 4 years ago
- Related to Refactor #7270: NoArtifactContentUploadViewSet should use PulpTemporaryFile instead of Artifact to store temporary data added
Updated by pulpbot over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Introducing PulpTemporaryFile
https://pulp.plan.io/issues/6749 closes #6749