Project

Profile

Help

Refactor #7505

Drop --local-sqlite for createrepo_c to avoid thrashing /tmp

Added by mhaluza about 1 month ago. Updated 16 days ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
2.21.4
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

Commit ca24ae2 referenced via https://pulp.plan.io/issues/1157 as internal need for the former RCM team introduced --local-sqlite parameter for createrepo_c to speed up publish process in a scenario where pulp filesystems were located on NFS. This need is no longer valid.

Currently, the --local-sqlite causes createrepo_c to write to TMPDIR instead of its usual output directory, which causes huge sqlite files to be left under /tmp after cancelled or failed Pulp tasks. /tmp is oftentimes fairly small and this may cause the filesystem to fill up quickly. Pulp already has the concept of creating a temporary working directory for each task under /var/lib/pulp/working and cleaning it up at the end. Usage of this working directory should be preferred.

Suggested change: https://github.com/pulp/pulp_rpm/pull/1833

Associated revisions

Revision fa4413bd View on GitHub
Added by mhaluza about 1 month ago

Drop --local-sqlite for createrepo_c to avoid thrashing /tmp

This PR reverts commit ca24ae299 which has been introduced in https://pulp.plan.io/issues/1157 as internal (and no longer valid) need for the former RCM team. Currently, the --local-sqlite causes createrepo_c to write to TMPDIR instead of its usual output directory, which causes huge sqlite files to be left under /tmp after cancelled or failed Pulp tasks. /tmp is oftentimes fairly small and this may cause the filesystem to fill up quickly. Pulp already has the concept of creating a temporary working directory for each task under /var/lib/pulp/working and cleaning it up at the end.

closes #7505 https://pulp.plan.io/issues/7505

Revision 8cd3eba2 View on GitHub
Added by mhaluza 10 days ago

Drop --local-sqlite for createrepo_c to avoid thrashing /tmp

This PR reverts commit ca24ae299 which has been introduced in https://pulp.plan.io/issues/1157 as internal (and no longer valid) need for the former RCM team. Currently, the --local-sqlite causes createrepo_c to write to TMPDIR instead of its usual output directory, which causes huge sqlite files to be left under /tmp after cancelled or failed Pulp tasks. /tmp is oftentimes fairly small and this may cause the filesystem to fill up quickly. Pulp already has the concept of creating a temporary working directory for each task under /var/lib/pulp/working and cleaning it up at the end.

closes #7505 https://pulp.plan.io/issues/7505

(cherry picked from commit fa4413bd13b60587c173f36573899bcc653ab8c8)

History

#1 Updated by ipanova@redhat.com about 1 month ago

  • Status changed from NEW to POST

#2 Updated by mhaluza about 1 month ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#3 Updated by ggainey 16 days ago

  • Platform Release set to 2.21.4

Please register to edit this issue

Also available in: Atom PDF