Project

Profile

Help

Issue #8430

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

Added by daviddavis 4 months ago. Updated 4 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.

Associated revisions

Revision bd75e125 View on GitHub
Added by daviddavis 4 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 4 months ago

  • Status changed from NEW to POST

#2 Updated by daviddavis 4 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 4 months ago

  • Status changed from POST to MODIFIED

#4 Updated by mdellweg 4 months ago

  • Sprint/Milestone set to 3.12.0

#5 Updated by pulpbot 4 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF