Project

Profile

Help

Task #5984

closed

Make `pulp_importer_repo_id` optional

Added by ttereshc almost 5 years ago. Updated over 4 years 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.

Actions #1

Updated by ipanova@redhat.com almost 5 years ago

  • Sprint set to Sprint 65

Added by ttereshc almost 5 years ago

Revision 34057209 | View on GitHub

Make pulp2 importer optional

closes #6056 closes #5984

Added by ttereshc almost 5 years ago

Revision 34057209 | View on GitHub

Make pulp2 importer optional

closes #6056 closes #5984

Added by ttereshc almost 5 years ago

Revision 34057209 | View on GitHub

Make pulp2 importer optional

closes #6056 closes #5984

Actions #2

Updated by ttereshc almost 5 years ago

  • Status changed from NEW to MODIFIED
  • % Done changed from 0 to 100
Actions #3

Updated by ttereshc over 4 years ago

  • Sprint/Milestone set to 0.1.0
Actions #4

Updated by ttereshc over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF