Project

Profile

Help

Issue #8683

re-uploading an existing advisory makes new advisory upload fails

Added by guialemas 3 months ago. Updated 2 months ago.

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

Description

Starting with a clean Pulp:

  1. upload an advisory
  2. upload the same advisory again (it will fail because it already exists)
  3. upload any new advisory and they will all fail

Versions:

python3-pulpcore-3.9.1-2.el8.noarch python3-pulp-rpm-3.10.0-1.el8.noarch


Related issues

Related to Pulp - Issue #8708: Tasking: Uploading many advisories in a row can have intermittent failuresCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Related to RPM Support - Issue #8716: Advisory-digest-calculation can result in the 'same' advisory having different digestsCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision e0570897 View on GitHub
Added by ggainey 2 months ago

Handle temporary-blank-digest advisory in create() error path(s).

Prior to this fix, a failure in create() would result in being unable to create() advisories again, ever, until the advisory with a blank digest is removed "by hand".

fixes #8683

History

#1 Updated by guialemas 3 months ago

from ggainey on IRC: "select count(*) from rpm_updaterecord where digest = ''; shows one entry and further uploads fail

#2 Updated by ggainey 3 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ggainey

#3 Updated by ggainey 3 months ago

Here's where the problem comes from, and a Giant Hammer for fixing it:

https://github.com/ggainey/pulp_rpm/commit/805aa8f2eb737a2d888011dffc4dc886aa95f275

Not ready to submit as a PR yet.

#4 Updated by pulpbot 3 months ago

  • Status changed from ASSIGNED to POST

#5 Updated by ggainey 3 months ago

  • Related to Issue #8708: Tasking: Uploading many advisories in a row can have intermittent failures added

#6 Updated by ggainey 3 months ago

  • Related to Issue #8716: Advisory-digest-calculation can result in the 'same' advisory having different digests added

#7 Updated by dalley 3 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 96

#8 Updated by rchan 3 months ago

  • Sprint changed from Sprint 96 to Sprint 97

#9 Updated by ggainey 2 months ago

  • Status changed from POST to MODIFIED

#10 Updated by ttereshc 2 months ago

  • Sprint/Milestone set to 3.12.0

#11 Updated by pulpbot 2 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF