Project

Profile

Help

Issue #1789

closed

Sync fills up /var/cache/pulp/ with rpms

Added by mhrivnak about 8 years ago. Updated about 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
2.7.1
Platform Release:
2.8.1
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

Originally reported by a user here: https://www.redhat.com/archives/pulp-list/2016-March/msg00041.html

In 2.6 (and earlier), at the completion of each rpm download, the file would be moved from its temporary location to a permanent location. That means for any given sync, you would never have more files in that temporary location than you did concurrent downloads. Using the default setting of 5 concurrent downloads, that's a maximum of 5 rpms. No problem so far.

This change turned the "move" into a "copy":

https://github.com/pulp/pulp_rpm/commit/de1895ede2f24731dcf6f52321beaee6b71c15a4

Now all of the rpm files pile up, and they don't get removed until the end of the task. On quick glance, it appears that 2.8 suffers the same problem, although the code has been refactored.

Actions #1

Updated by mhrivnak about 8 years ago

  • Status changed from ASSIGNED to POST

Added by mhrivnak about 8 years ago

Revision 4b132682 | View on GitHub

RPMs and DRPMs get removed from /var/cache/pulp immediately after import

https://pulp.plan.io/issues/1789

fixes #1789

Actions #2

Updated by mhrivnak about 8 years ago

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

Updated by semyers about 8 years ago

  • Status changed from MODIFIED to 5
Actions #4

Updated by pthomas@redhat.com about 8 years ago

  • Status changed from 5 to 6

verified that the /var/cache/pulp is not filling up during sync.

Actions #5

Updated by semyers about 8 years ago

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

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF