Project

Profile

Help

Issue #8430

Pulp raises misleading message "Package matching query does not exist"

Added by daviddavis 7 months ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

A user on pulp-list ran into this problem. Consider the case where you have a unique constraint for a Content unit that isn't part of the natural key. First, you hit this IntegrityError and Pulp assumes that there was a natural key conflict so it tries to look up the existing Content by natural key. However, since the unique constraint isn't part of the natural key, it hits a DoesNotExist and then logs "X matching query does not exist" which isn't helpful.


Related issues

Related to Pulp - Issue #7828: syncing two repos with the same content at the same time results in an errorCLOSED - DUPLICATE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision bd75e125 View on GitHub
Added by daviddavis 7 months ago

Fix error for unique constraint violation

In the case where there's a unique constraint that isn't part of the natural key, using q() won't find the record. This erroneously raises a DoesNotExist exception.

fixes #8430

History

#1 Updated by pulpbot 7 months ago

  • Status changed from NEW to POST

#2 Updated by daviddavis 7 months ago

  • Subject changed from Pulp raises unhelpful message "Package matching query does not exist" to Pulp raises misleading message "Package matching query does not exist"

#3 Updated by daviddavis 7 months ago

  • Status changed from POST to MODIFIED

#4 Updated by mdellweg 6 months ago

  • Sprint/Milestone set to 3.12.0

#5 Updated by pulpbot 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

#6 Updated by dalley 3 months ago

  • Related to Issue #7828: syncing two repos with the same content at the same time results in an error added

Please register to edit this issue

Also available in: Atom PDF