Project

Profile

Help

Issue #1972

migration 28 misses distribution symlinks

Added by jortel@redhat.com over 4 years ago. Updated over 1 year ago.

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

Description

The storage_path for a distribution is a directory but the distributor publishes separate symlinks to each file. The 0028 migration matches symlinks to units by matching the link target to the unit._storage_path. This result in none of the distribution links to be matched. To correct this, the migration in the platform needs to:

  • Support custom Unit classes.
  • Add a Unit.files property that defaults to an empty tuple.
  • The distribution plan needs to override Plan._new_unit() and return the custom unit.
  • The custom Unit needs to override files() to return the value of document.files along with treeinfo and .treeinfo.
    The result is, the migration in the platform will know about all of the individual links created by the distributor and that need to be migrated.

Associated revisions

Revision f54e587d View on GitHub
Added by jortel@redhat.com over 4 years ago

Fix migration 0028, distribution links to files within storage path. closes #1972

History

#1 Updated by jortel@redhat.com over 4 years ago

  • Status changed from ASSIGNED to POST

#2 Updated by jortel@redhat.com over 4 years ago

  • Description updated (diff)

#3 Updated by jortel@redhat.com over 4 years ago

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

#4 Updated by semyers over 4 years ago

  • Platform Release set to 2.8.5

#5 Updated by semyers over 4 years ago

  • Status changed from MODIFIED to 5

#7 Updated by pthomas@redhat.com over 4 years ago

  • Status changed from 5 to 6

Verified

1. Synced a rhel6 kickstart repo
2. Used the synced distribution file to kickstart a vm

#8 Updated by semyers over 4 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#11 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF