Project

Profile

Help

Issue #8683

closed

re-uploading an existing advisory makes new advisory upload fails

Added by guialemas almost 3 years ago. Updated almost 3 years 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 - CURRENTRELEASEmdellwegActions
Related to RPM Support - Issue #8716: Advisory-digest-calculation can result in the 'same' advisory having different digestsCLOSED - CURRENTRELEASEggaineyActions
Actions #1

Updated by guialemas almost 3 years ago

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

Actions #2

Updated by ggainey almost 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ggainey
Actions #3

Updated by ggainey almost 3 years 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.

Actions #4

Updated by pulpbot almost 3 years ago

  • Status changed from ASSIGNED to POST
Actions #5

Updated by ggainey almost 3 years ago

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

Updated by ggainey almost 3 years ago

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

Updated by dalley almost 3 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 96
Actions #8

Updated by rchan almost 3 years ago

  • Sprint changed from Sprint 96 to Sprint 97

Added by ggainey almost 3 years ago

Revision e0570897 | View on GitHub

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

Actions #9

Updated by ggainey almost 3 years ago

  • Status changed from POST to MODIFIED
Actions #10

Updated by ttereshc almost 3 years ago

  • Sprint/Milestone set to 3.12.0
Actions #11

Updated by pulpbot almost 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF