Actions
Task #5984
closedMake `pulp_importer_repo_id` optional
Start date:
Due date:
% Done:
100%
Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 65
Quarter:
Description
There are cases when a migrated repo doesn't need a remote and it should be handled.
Currently we require pulp2_importer_repository_id to be specified but we don't migrate importers without a feed.
There are 2 options:
1. make it optional and require migration plugin creator specify only importers which has a feed
- cons: responsibility is on MP creator
- pros: easier to spot a problem in migration plugin if MP is correct
-2. keep it required but handle the case when importer is not pre-migrated (for whatever reason, aka config is empty)
- cons: migration plugin has no idea why the importer is not available - bug or intention
- pros: no responsibility on MP creator.-
Option 1 met more support with a suggestion to explicitly state in the error message reasons why the importer can't be migrated.
Added by ttereshc almost 5 years ago
Added by ttereshc almost 5 years ago
Added by ttereshc almost 5 years ago
Updated by ttereshc almost 5 years ago
- Status changed from NEW to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp:pulp-2to3-migration|340572099b1ad437a0c79dd875c55cac8bbc0b9b.
Updated by ttereshc over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Make pulp2 importer optional
closes #6056 closes #5984