Project

Profile

Help

Issue #1972

closed

migration 28 misses distribution symlinks

Added by jortel@redhat.com almost 8 years ago. Updated about 5 years 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.
Actions #1

Updated by jortel@redhat.com almost 8 years ago

  • Status changed from ASSIGNED to POST
Actions #2

Updated by jortel@redhat.com almost 8 years ago

  • Description updated (diff)

Added by jortel@redhat.com almost 8 years ago

Revision f54e587d | View on GitHub

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

Actions #3

Updated by jortel@redhat.com almost 8 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #4

Updated by semyers almost 8 years ago

  • Platform Release set to 2.8.5
Actions #5

Updated by semyers almost 8 years ago

  • Status changed from MODIFIED to 5
Actions #7

Updated by pthomas@redhat.com almost 8 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

Actions #8

Updated by semyers almost 8 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE
Actions #11

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF