Project

Profile

Help

Send by e-mail

Issue #4006

Multi-Artifact Content Units receive Duplicate Key Error with DeclarativeVersion

Added by bmbouter about 1 year ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
High
Assignee:
bmbouter
Category:
-
Sprint/Milestone:
3.0
Start date:
Due date:
Severity:
3. High
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

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/

Associated revisions

Revision c683ec82 View on GitHub
Added by bmbouter about 1 year ago

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

Revision c683ec82 View on GitHub
Added by bmbouter about 1 year ago

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

Revision c683ec82 View on GitHub
Added by bmbouter about 1 year ago

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

History

#2 Updated by bmbouter about 1 year ago

  • Status changed from ASSIGNED to MODIFIED

#3 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#4 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Send by e-mail

Also available in: Atom PDF