Project

Profile

Help

Issue #8430

closed

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

Added by daviddavis almost 4 years ago. Updated over 3 years 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 - DUPLICATEggaineyActions
Actions #1

Updated by pulpbot almost 4 years ago

  • Status changed from NEW to POST
Actions #2

Updated by daviddavis almost 4 years 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"

Added by daviddavis over 3 years ago

Revision bd75e125 | View on GitHub

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

Actions #3

Updated by daviddavis over 3 years ago

  • Status changed from POST to MODIFIED
Actions #4

Updated by mdellweg over 3 years ago

  • Sprint/Milestone set to 3.12.0
Actions #5

Updated by pulpbot over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #6

Updated by dalley over 3 years ago

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

Also available in: Atom PDF