Project

Profile

Help

Issue #5924

closed

Advisory conflict resolution affects mirror sync case

Added by ttereshc over 4 years ago. Updated about 4 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 66
Quarter:

Description

Currently advisory conflict resolution assumes that advisories from a base/previous version are always present in the new one. It might not always be the case (e.g. mirror case). Current behaviour leads to wrong results - either more advisories are present than needed, or the error is raised due to inability to resolve advisory conflicts.

resolve_advisories should check whether any conflicts are present in the current version and only then, look into previous version for such advisories to figure out which ones were present before and which ones are being added.

Actions #1

Updated by ttereshc over 4 years ago

Actions #2

Updated by ttereshc over 4 years ago

  • Triaged changed from No to Yes
Actions #3

Updated by ppicka about 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ppicka
  • Sprint set to Sprint 64
Actions #4

Updated by ppicka about 4 years ago

  • Status changed from ASSIGNED to POST
Actions #5

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 64 to Sprint 65
Actions #6

Updated by rchan about 4 years ago

  • Sprint changed from Sprint 65 to Sprint 66

Added by ppicka about 4 years ago

Revision 33761b79 | View on GitHub

Advisory conflict resolution

Advisory conflict resolution now check current version first if advisory resolution is needed.

Required PR: https://github.com/pulp/pulp_rpm/pull/1568

closes: #5924 https://pulp.plan.io/issues/5924

Actions #7

Updated by ppicka about 4 years ago

  • Status changed from POST to MODIFIED
Actions #8

Updated by ttereshc about 4 years ago

  • Sprint/Milestone set to Pulp RPM 3.2.0
Actions #9

Updated by ttereshc about 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF