Project

Profile

Help

Issue #2704

closed

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

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

Actions #1

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

Actions #2

Updated by mhrivnak almost 7 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

Added by jortel@redhat.com almost 7 years ago

Revision ce23d674 | View on GitHub

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

Actions #3

Updated by jortel@redhat.com almost 7 years ago

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

Updated by jortel@redhat.com almost 7 years ago

  • Status changed from ASSIGNED to POST
Actions #5

Updated by jortel@redhat.com almost 7 years ago

  • Status changed from POST to MODIFIED
Actions #6

Updated by bizhang almost 7 years ago

  • Platform Release set to 2.13.1
Actions #7

Updated by bizhang almost 7 years ago

  • Status changed from MODIFIED to 5
Actions #8

Updated by bizhang almost 7 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #9

Updated by bmbouter about 6 years ago

  • Sprint set to Sprint 18
Actions #10

Updated by bmbouter about 6 years ago

  • Sprint/Milestone deleted (37)
Actions #11

Updated by bmbouter almost 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF