Issue #7185
closed
force_full rsync publish is done unnecessarily in some cases
Description
A full rsync publish is done when some criteria are met (defined in https://github.com/pulp/pulp/blob/2-master/server/pulp/plugins/rsync/publish.py#L373). Some of the criteria don't need a full rsync publish to make the right content published to rsync distributor:
- When there's units removal since last rsync publish
- When there's predistributor publish failure since last rsync publish
Since it takes longer time to do full publish, especially for big repos containing lots of units, unnecessary full publish should be avoided by removing above criteria.
Added by yuzheng over 4 years ago
Updated by yuzheng over 4 years ago
- Status changed from NEW to MODIFIED
Applied in changeset pulp|7fff42fc218b94cabf8140053ef393b54a427a02.
Added by yuzheng about 4 years ago
Revision c262d314 | View on GitHub
Remove unnecessary criteria of force_full rsync publish (#3997)
- Remove unnecessary criteria of force_full rsync publish
Rsync publish after units removal or failed predistributor publish doesn't have to be force_full publish, so remove them from criteria.
RepoPublishResult doesn't have distributor_config, so the criteria of predistributor_force_full doesn't work, remove it.
closes #7185 https://pulp.plan.io/issues/7185
- Remove useless variable
(cherry picked from commit 7fff42fc218b94cabf8140053ef393b54a427a02)
Updated by ggainey about 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Remove unnecessary criteria of force_full rsync publish (#3997)
Rsync publish after units removal or failed predistributor publish doesn't have to be force_full publish, so remove them from criteria.
RepoPublishResult doesn't have distributor_config, so the criteria of predistributor_force_full doesn't work, remove it.
closes #7185 https://pulp.plan.io/issues/7185