Project

Profile

Help

Story #6329

closed

Story #6134: [EPIC] Pulp import/export

As a user, I can create/read/update/delete PulpImporters

Added by daviddavis over 4 years ago. Updated over 4 years ago.

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

100%

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

Actions #1

Updated by daviddavis over 4 years ago

Looking at the import API in the design document[0], it looks like there's just a resource PulpImporter that's available at /pulp/api/v3/pulp_importers/. I wonder if we should make importers master/detail models though? Mostly to mirror Exporters (eg /pulp/api/v3/exporters/core/pulp/). That said, I can't think of a use case for this since most of the time, plugins import via sync.

[0] https://hackmd.io/HLptudH9R6S4PCmm8nRmmg#Import

Actions #2

Updated by daviddavis over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis
Actions #3

Updated by ggainey over 4 years ago

daviddavis wrote:

Looking at the import API in the design document[0], it looks like there's just a resource PulpImporter that's available at /pulp/api/v3/pulp_importers/. I wonder if we should make importers master/detail models though? Mostly to mirror Exporters (eg /pulp/api/v3/exporters/core/pulp/). That said, I can't think of a use case for this since most of the time, plugins import via sync.

[0] https://hackmd.io/HLptudH9R6S4PCmm8nRmmg#Import

The original intent was for import and export to mirror each other (the APIs originally matched), to among other things minimize user confusion. Also, won't MasterDetail give plugin authors an opportunity to customize/tweak the import process where it may be necessary?

Actions #4

Updated by bmbouter over 4 years ago

Making importers master detail is probably the better thing to do. sync's are great for one repo, but we could have future use cases that require us to bring in content to a group of repositories with another mechanism. @daviddavis did I understand your question correctly?

Actions #5

Updated by bmbouter over 4 years ago

ggainey wrote:

The original intent was for import and export to mirror each other (the APIs originally matched), to among other things minimize user confusion. Also, won't MasterDetail give plugin authors an opportunity to customize/tweak the import process where it may be necessary?

+1 to Master/Detail, but that's not the mechanism plugin writers use to tweak the import/export process for their content and it's associated models.

What Master/Detail allows is for plugin writers to define a whole, new import codepath that they exclusively own with it's own, new URL endpoints.

Actions #6

Updated by daviddavis over 4 years ago

We also discussed this at the rpm meeting today and I think support for using Master/Detail was unanimous. I've updated the design doc.

Actions #7

Updated by pulpbot over 4 years ago

  • Status changed from ASSIGNED to POST

Added by daviddavis over 4 years ago

Revision be696b23 | View on GitHub

Added support for importing Pulp Exports

fixes #6329 fixes #6137

Actions #8

Updated by daviddavis over 4 years ago

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

Updated by ttereshc over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
  • Sprint/Milestone set to 3.3.0

Also available in: Atom PDF