Project

Profile

Help

Story #6673

Task #6672: [Epic] Upload Process Improvements

As a subclasser of the collection upload viewset, I can customize the task I'm dispatching to instead of always using import_collection

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

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

100%

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

Description

Background

Currently there are two Galaxy viewsets that provide collection import functionality.

  • One in the Galaxy V2 API here
  • Another in the Galaxy V3 API here

Goal

The subclasser wants to be able to provide their own task to perform collection import. As long as these viewsets hard-code the task that isn't possible.

Solution

  1. Add a new internal method named _dispatch_import_collection_task(repository, temp_file_path).
  2. The viewsets currently save an Artifact, but this should be a temporary file instead. Artifacts are for permanent storage, but the need here is only to save something temporarily until the backend task can analyze it and then save it.
  3. The _dispatch_import _collection_task() needs to form its own locks. The default lock should only lock on the repository it is being imported to. By default it should use the pulp_ansible.app.tasks.import_collection like it does today.

Associated revisions

Revision 6c733e34 View on GitHub
Added by Fabricio Aguiar 6 months ago

Enabled dispatching customization for collections

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

History

#1 Updated by bmbouter 6 months ago

  • Parent task set to #6672

#2 Updated by fao89 6 months ago

  • Groomed changed from No to Yes
  • Sprint set to Sprint 72

#3 Updated by fao89 6 months ago

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

#4 Updated by fao89 6 months ago

  • Status changed from ASSIGNED to POST

#5 Updated by Anonymous 6 months ago

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

#6 Updated by fao89 5 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

#7 Updated by fao89 2 months ago

  • Sprint/Milestone set to 0.2.0

Please register to edit this issue

Also available in: Atom PDF