Project

Profile

Help

Task #5984

Make `pulp_importer_repo_id` optional

Added by ttereshc over 1 year ago. Updated about 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Sprint/Milestone:
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.

Associated revisions

Revision 34057209 View on GitHub
Added by ttereshc over 1 year ago

Make pulp2 importer optional

closes #6056 closes #5984

Revision 34057209 View on GitHub
Added by ttereshc over 1 year ago

Make pulp2 importer optional

closes #6056 closes #5984

Revision 34057209 View on GitHub
Added by ttereshc over 1 year ago

Make pulp2 importer optional

closes #6056 closes #5984

History

#1 Updated by ipanova@redhat.com over 1 year ago

  • Sprint set to Sprint 65

#2 Updated by ttereshc over 1 year ago

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

#3 Updated by ttereshc about 1 year ago

  • Sprint/Milestone set to 0.1.0

#4 Updated by ttereshc about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF