Bug 985938

Summary: [RFE] store checksum type on ISOs
Product: [Retired] Pulp Reporter: Dennis Gregorovic <dgregor>
Component: iso-supportAssignee: pulp-bugs
Status: CLOSED UPSTREAM QA Contact: Preethi Thomas <pthomas>
Severity: low Docs Contact:
Priority: medium    
Version: 2.2 BetaCC: dgregor, mhrivnak, rbarlow, sskracic
Target Milestone: ---Keywords: FutureFeature, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-02-19 01:11:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Dennis Gregorovic 2013-07-18 14:33:36 UTC
Units of type ISO have a checksum field.  Please add a checksum_type field as well.

Comment 1 Michael Hrivnak 2013-07-18 15:00:22 UTC
It looks like the manifest files on the CDN also don't include a checksum type, so we aren't able to determine what the type is. 

http://download.devel.redhat.com/cds/prod/content/dist/rhel/server/6/6Server/x86_64/iso/PULP_MANIFEST

I understand that you basically own this file's format, so perhaps we should start by adding it there, and then it'll be easy for pulp to grab it and save it.

Of course we'll have to be careful to make sure current pulp doesn't fail to parse the file once the checksum type is added, so we should collaborate on the plan and make sure pulp is able to handle it.

Comment 3 Jay Dobies 2013-11-01 19:03:17 UTC
Is this still needed?

Comment 4 Dennis Gregorovic 2013-11-01 19:12:22 UTC
Yes.  It's not urgent, but I would like this added at some point.

Comment 6 Brian Bouterse 2015-02-19 01:11:34 UTC
Moved to https://pulp.plan.io/issues/201