Project

Profile

Help

Story #1878

Support for choosing the checksum type in updateinfo

Added by dgregor@redhat.com almost 5 years ago. Updated almost 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
2.10.0
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Pulp 2
Sprint:
Sprint 4
Quarter:

Description

Given errata where the package list has checksums of multiple types, pulp can publish only one of those types, and filter out any others.

When pulp generates updateinfo.xml as part of a yum publish, it has logic for choosing the checksum type(s) based on the checksum(s) in the errata metadata and the checksum type of the repo. For example, if the repo is configured for sha256 and the errata metadata includes both md5 and sha256, pulp will use sha256. For backward-compatibility reasons, it is sometimes needed to use a checksum type in updateinfo that is different than the checksum type of the repo itself. Please have a way to optionally configure a checksum type for updateinfo. When it's not set it can fall back to the checksum type of the repo.


Related issues

Related to RPM Support - Issue #1618: --checksum-type is brokenCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Blocked by RPM Support - Issue #2020: Cannot provide multiple checksums when uploading an erratumCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 1b7f1dc8 View on GitHub
Added by ttereshc over 4 years ago

Add support for choosing the checksum type in updateinfo.xml

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

History

#1 Updated by dgregor@redhat.com almost 5 years ago

  • Tracker changed from Issue to Story
  • Groomed set to No
  • Sprint Candidate set to No

#2 Updated by bmbouter almost 5 years ago

  • Parent task set to #1683

#3 Updated by bmbouter almost 5 years ago

  • Related to Issue #1618: --checksum-type is broken added

#4 Updated by bmbouter almost 5 years ago

  • Sprint Candidate changed from No to Yes

This makes sense to me but needs to be groomed by another Pulp developer and marked as Groomed=True.

#5 Updated by mhrivnak almost 5 years ago

I assume the checksums being referred to are the rpm checksums in the package list?

Dennis, are you asking for:

  • Given errata where the package list has checksums of multiple types, pulp can publish only one of those types, and filter out any others.

or are you asking for:

  • Pulp can publish an erratum using any supported checksum type. If publish is requested with a checksum type where those values are not already in the package list for any given erratum, pulp will go find the relevant RPMs in the database (assuming they are in the repo), and retrieve the checksums.

The former is easy. The latter is complex. An Errata unit in pulp does not have a direct association with RPMs, so retrieving a checksum value would require a unit search by nevra within the repo for each RPM. That could obviously add substantial time if there are many errata.

And if you want the latter, is md5 a checksum type we would need to support retrieval for? We have only been planning to store sha1 and sha256 checksums on RPM units.

#6 Updated by mhrivnak almost 5 years ago

  • Sprint Candidate changed from Yes to No

#7 Updated by dgregor@redhat.com almost 5 years ago

Just asking for the former: "Given errata where the package list has checksums of multiple types, pulp can publish only one of those types, and filter out any others."

#8 Updated by bmbouter over 4 years ago

  • Sprint Candidate changed from No to Yes

#9 Updated by mhrivnak over 4 years ago

  • Description updated (diff)
  • Groomed changed from No to Yes

#10 Updated by mhrivnak over 4 years ago

  • Sprint/Milestone set to 22

#11 Updated by ttereshc over 4 years ago

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

#12 Updated by ttereshc over 4 years ago

  • Blocked by Issue #2020: Cannot provide multiple checksums when uploading an erratum added

#13 Updated by ttereshc over 4 years ago

  • Status changed from ASSIGNED to POST

#14 Updated by ttereshc over 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#15 Updated by ttereshc over 4 years ago

  • Platform Release set to 2.10.0

#16 Updated by semyers over 4 years ago

  • Status changed from MODIFIED to 5

#17 Updated by semyers over 4 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#18 Updated by bmbouter almost 3 years ago

  • Sprint set to Sprint 4

#19 Updated by bmbouter almost 3 years ago

  • Sprint/Milestone deleted (22)

#20 Updated by bmbouter almost 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF