Refactor #7505
closed
Drop --local-sqlite for createrepo_c to avoid thrashing /tmp
100%
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
Added by mhaluza over 3 years ago
Updated by ipanova@redhat.com over 3 years ago
- Status changed from NEW to POST
Updated by mhaluza over 3 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset fa4413bd13b60587c173f36573899bcc653ab8c8.
Added by mhaluza over 3 years ago
Revision 8cd3eba2 | View on GitHub
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)
Updated by ggainey over 3 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
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