Backport #8962
closedBackport 8944 to 3.13.1
0%
Related issues
Updated by dalley over 3 years ago
- Related to Issue #8944: Cannot sync RHEL6 repos from cdn.redhat.com added
Added by dalley over 3 years ago
Updated by dalley over 3 years ago
- Status changed from NEW to MODIFIED
Applied in changeset e3b0e47d29aab5125792f1f2fd8c959cddb06c01.
Updated by pulpbot over 3 years ago
Added by dalley over 3 years ago
Revision 884a1d1c | View on GitHub
Handle duplicate packages in upstream repos
Some repositories have some packages listed in the metadata twice. This shouldn't happen (but it does). createrepo_c deduplicates by virtue of parsing everything into a dict keyed by pkgid, but the iterative parser does not. This eventually results in a mismatch once the iterative parser comes across a package that the createrepo_c primary parser already handled. So we keep a list of pkgid's we've already written out in order to skip them once the iterative parser hits them a 2nd (or 3rd, etc.) time.
backports: #8944 https://pulp.plan.io/issues/8944
fixes #8962
(cherry picked from commit 8393a60695dd28a38d515cd7376396734626ae16)
Updated by dalley over 3 years ago
- Tracker changed from Issue to Backport
- % Done set to 0
- Severity deleted (
2. Medium) - Groomed deleted (
No)
Updated by pulpbot over 3 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Handle duplicate packages in upstream repos
Some repositories have some packages listed in the metadata twice. This shouldn't happen (but it does). createrepo_c deduplicates by virtue of parsing everything into a dict keyed by pkgid, but the iterative parser does not. This eventually results in a mismatch once the iterative parser comes across a package that the createrepo_c primary parser already handled. So we keep a list of pkgid's we've already written out in order to skip them once the iterative parser hits them a 2nd (or 3rd, etc.) time.
backports: #8944 https://pulp.plan.io/issues/8944
fixes #8962
(cherry picked from commit 8393a60695dd28a38d515cd7376396734626ae16)