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
.
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