Issue #4006
closed
Multi-Artifact Content Units receive Duplicate Key Error with DeclarativeVersion
Description
Problem¶
The pulp_gem plugin code here [0] is written correctly, and the fixture data they use is also correct [1].
I've been able to reproduce the issue described to me on IRC.
Root Cause¶
There is a bug in the ArtifactDownloader stage whereby a unit with multiple Artifacts incorrectly reuses the first downloader's information (incorrectly) for subsequent Artifacts. This causes the Artifact to have the same sha256 value which raises the duplicate key exception.
Solution¶
Fix the bug by removing the extra, incorrect loop that is present.
[0]: https://github.com/ATIX-AG/pulp_gem/blob/update_api/pulp_gem/app/tasks/synchronizing.py#L69-L95
[1]: https://repos.fedorapeople.org/pulp/pulp/demo_repos/gems/repo/
Added by bmbouter over 6 years ago
Added by bmbouter over 6 years ago
Revision c683ec82 | View on GitHub
Multi-Artifact Content units work now
The artifact downloader was not instantiating the Artifacts correctly which caused duplicate key errors.
Updated by bmbouter over 6 years ago
PR available at: https://github.com/pulp/pulp/pull/3628
Updated by bmbouter over 6 years ago
- Status changed from ASSIGNED to MODIFIED
Applied in changeset pulp|c683ec82c4b43dc1316b7f3923641389ae4e8e60.
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Multi-Artifact Content units work now
The artifact downloader was not instantiating the Artifacts correctly which caused duplicate key errors.
https://pulp.plan.io/issues/4006 closes #4006