Project

Profile

Help

Issue #2704

Catalog entries for existing content units with old-style storage path get created with new-style storage path.

Added by jortel@redhat.com almost 5 years ago. Updated almost 3 years ago.

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

Description

Catalog entries for existing content units with old-style storage path get created with new-style storage path. The code creating the catalog entry blindly calls set_storage_path() on the unit to ensure that the path is set. For RPMs and ISOs that already exist and are fetched from the DB, this overwrites the _storage_path of the unit (in memory) just before inserting the catalog entry.

This only affects installation that do not run the storage-path migration introduced in 2.8.

Associated revisions

Revision ce23d674 View on GitHub
Added by jortel@redhat.com almost 5 years ago

Ensure path in catalog entry matches unit storage path for existing units. closes #2704

History

#1 Updated by jortel@redhat.com almost 5 years ago

Testing:
1. Create and sync a repo with download-policy=on_demand.
2. Sync the repo.
3. Using mongo shell, query for the catalog entry and make a note of the path.
4. Using mongo shell, update the unit for one of the units and set the _storage_path=<something else>
5. Run the sync w/ --force_full
6. Using mongo shell, query for the catalog entry and note, it still has the same path but should not because the unit was updated.

patched:

Repeat 1-6 but note the path in the catalog entry matches the change in the unit.

#2 Updated by mhrivnak almost 5 years ago

  • Priority changed from Normal to High
  • Sprint/Milestone set to 37
  • Severity changed from 2. Medium to 3. High
  • Triaged changed from No to Yes

#3 Updated by jortel@redhat.com almost 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to jortel@redhat.com

#4 Updated by jortel@redhat.com almost 5 years ago

  • Status changed from ASSIGNED to POST

#5 Updated by jortel@redhat.com almost 5 years ago

  • Status changed from POST to MODIFIED

#6 Updated by bizhang over 4 years ago

  • Platform Release set to 2.13.1

#7 Updated by bizhang over 4 years ago

  • Status changed from MODIFIED to 5

#8 Updated by bizhang over 4 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#9 Updated by bmbouter almost 4 years ago

  • Sprint set to Sprint 18

#10 Updated by bmbouter almost 4 years ago

  • Sprint/Milestone deleted (37)

#11 Updated by bmbouter almost 3 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF