Project

Profile

Help

Issue #2773

closed

ISO repo does not handle updates to files on manifest during re-sync correctly

Added by mhrivnak over 7 years ago. Updated over 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
2.9.3
Platform Release:
2.13.3
OS:
Triaged:
Yes
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Pulp 2
Sprint:
Sprint 22
Quarter:

Description

Copied from Bugzilla thanks to Craig Donnelly:

Description of problem:

Currently, if you generate a PULP_MANIFEST and sync from another location via URL, you will receive the files in the Manifest and they will become locally available, but if you UPDATE a file that has already been imported and regenerate the PULP_MANIFEST, then sync again, Pulp's versions of the manifest will have a duplicate file name entry with separate sha256sums and sizes, but will not retrieve the new file (It will maintain the original in the repo).

Version-Release number of selected component (if applicable):
pulp-server-2.9.3-2.el7sat.noarch

How reproducible:
100%

Steps to Reproduce:
1. Generate PULP_MANIFEST and sync a File repo from a http based URL
2. Make a change to a file (text file, etc..), but DO NOT change the file NAME.
3. Re-Generate the PULP_MANIFEST for the remote file repo, and then re-sync on Satellite.

Actual results:
File that was modified will have two entries on the PULP_MANIFEST inside the local Satellite copy of the repo with separate sha256sums and sizes, but identical name, and only the original copy of the file prior to editing will exist in the repo.

Expected results:
We should replace original file with the newest checksum. (Act like mirror-on-sync?)

Additional info:
Perhaps the File repo type should be able to determine the time of PULP_MANIFEST generation and make a determination about what should exist based off of that? (i.e. Include a date inside the PULP_MANIFEST on creation)


Related issues

Related to RPM Support - Issue #3047: ISO repo doesn't correctly handle updates to files for content already in PulpCLOSED - CURRENTRELEASEdaviddavisActions
Actions #1

Updated by mhrivnak over 7 years ago

  • Description updated (diff)
  • Version set to 2.9.3
Actions #2

Updated by ttereshc over 7 years ago

  • Priority changed from Normal to High
  • Severity changed from 2. Medium to 3. High
  • Triaged changed from No to Yes
Actions #4

Updated by ttereshc over 7 years ago

  • Sprint Candidate changed from No to Yes
Actions #6

Updated by pcreech over 7 years ago

  • Groomed changed from No to Yes
Actions #7

Updated by mhrivnak over 7 years ago

  • Sprint/Milestone set to 40
Actions #8

Updated by daviddavis over 7 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis
Actions #9

Updated by daviddavis over 7 years ago

  • Status changed from ASSIGNED to POST

Added by daviddavis over 7 years ago

Revision 4db15dc8 | View on GitHub

Fixing dupe ISO units after file update

Fixing a bug whereby a ISO or file gets updated and then reimported into pulp by sync or upload. We're now disassociating the original content unit and replacing it with the new one. Matching on the name of the unit.

fixes #2773 https://pulp.plan.io/issues/2773

Actions #10

Updated by mhrivnak over 7 years ago

  • Sprint/Milestone changed from 40 to 41
Actions #11

Updated by daviddavis over 7 years ago

  • Status changed from POST to MODIFIED
Actions #12

Updated by pcreech over 7 years ago

  • Platform Release set to 2.13.3
Actions #13

Updated by pcreech over 7 years ago

  • Status changed from MODIFIED to 5
Actions #14

Updated by pcreech over 7 years ago

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

Updated by daviddavis about 7 years ago

  • Related to Issue #3047: ISO repo doesn't correctly handle updates to files for content already in Pulp added
Actions #16

Updated by bmbouter almost 7 years ago

  • Sprint set to Sprint 22
Actions #17

Updated by bmbouter almost 7 years ago

  • Sprint/Milestone deleted (41)
Actions #20

Updated by bmbouter over 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF