Project

Profile

Help

Issue #6056

closed

migration plans require a 'pulp2_importer_repository_id' but importer must have a feed url

Added by jsherril@redhat.com almost 5 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Katello
Sprint:
Sprint 65
Quarter:

Description

In katello we have some repos in pulp2 that do not have a feed associated with their improter.

If we try to leave these out, the migration plan will fail with:

["Provided Migration Plan format is invalid:'[\"'pulp2_importer_repository_id' is a required property\"]'"]}

If we try to include them, pulp3 will not 'pre-migrate' them as they don't have a feed url: https://github.com/pulp/pulp-2to3-migration/blob/master/pulp_2to3_migration/app/pre_migration.py#L322

Even though its missing a feed, and thus not pre-migrated, pulp3 will try to create a remote for it and fail (since its not premigrated).

I'm unclear what the actual fix is:
1) pulp2_importer_repository_id should not be required (its not needed)
2 pulp2_importer_repository_id should be required, but not actually migrated if it doesn't have a feed (if the importer is actually needed)

Actions #1

Updated by jsherril@redhat.com almost 5 years ago

  • Description updated (diff)
Actions #2

Updated by jsherril@redhat.com almost 5 years ago

  • Description updated (diff)
Actions #3

Updated by jsherril@redhat.com almost 5 years ago

  • Description updated (diff)
Actions #4

Updated by dalley almost 5 years ago

  • Status changed from NEW to POST
  • Assignee set to ipanova@redhat.com
Actions #5

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 #6

Updated by ttereshc almost 5 years ago

  • Status changed from POST to MODIFIED
Actions #7

Updated by ipanova@redhat.com almost 5 years ago

  • Description updated (diff)
  • Triaged changed from No to Yes
Actions #10

Updated by ttereshc over 4 years ago

  • Sprint/Milestone set to 0.1.0
Actions #11

Updated by ttereshc over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #12

Updated by ggainey over 4 years ago

  • Tags Katello added
  • Tags deleted (Katello-P1)

Also available in: Atom PDF