Project

Profile

Help

Issue #6056

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

Added by jsherril@redhat.com over 1 year ago. Updated 11 months 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)

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 jsherril@redhat.com over 1 year ago

  • Description updated (diff)

#2 Updated by jsherril@redhat.com over 1 year ago

  • Description updated (diff)

#3 Updated by jsherril@redhat.com over 1 year ago

  • Description updated (diff)

#4 Updated by dalley over 1 year ago

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

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

  • Sprint set to Sprint 65

#6 Updated by ttereshc over 1 year ago

  • Status changed from POST to MODIFIED

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

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

#10 Updated by ttereshc about 1 year ago

  • Sprint/Milestone set to 0.1.0

#11 Updated by ttereshc about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

#12 Updated by ggainey about 1 year ago

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

Please register to edit this issue

Also available in: Atom PDF