Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1139896 - synced RPMs do not have their checksums verified
synced RPMs do not have their checksums verified
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Content Management (Show other bugs)
6.0.4
Unspecified Unspecified
unspecified Severity high (vote)
: Unspecified
: Unused
Assigned To: David Davis
Tazim Kolhar
http://projects.theforeman.org/issues...
: Triaged
: 1153649 (view as bug list)
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-09 18:43 EDT by Mike McCune
Modified: 2017-02-23 15:58 EST (History)
11 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-08-12 01:16:15 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
TESTPLAN Screenshot - verify you see this in 6.1 (71.81 KB, image/png)
2015-04-27 19:33 EDT, Mike McCune
no flags Details
broken package (63.37 KB, image/png)
2015-04-30 07:12 EDT, Tazim Kolhar
no flags Details


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:1592 normal SHIPPED_LIVE Important: Red Hat Satellite 6.1.1 on RHEL 6 2015-08-12 05:04:35 EDT

  None (edit)
Description Mike McCune 2014-09-09 18:43:13 EDT
During testing we ran into a situation where RPMs had been synchronized from the CDN, copied to disk and then published in a Content View with an RPM that was truncated and not fully complete.  

Extraction of the RPM shows an incomplete file:

# rpm2cpio selinux-policy-targeted-3.7.19-231.el6.noarch.rpm | cpio -idmv
....
./usr/share/selinux/targeted/piranha.pp.bz2
cpio: ./usr/share/selinux/targeted/pkcsslotd.pp.bz2 not created: newer or same age version exists
./usr/share/selinux/targeted/pkcsslotd.pp.bz2
cpio: ./usr/share/selinux/targeted/plymouthd.pp.bz2 not created: newer or same age version exists
cpio: premature end of file

The metadata containing filesizes and checksums for obvious reasons, does not match the above file since it is corrupt for whatever reason.
Comment 1 Mike McCune 2014-09-09 18:49:39 EDT
Hit submit too soon, meant to say:

We need to investigate turning on checksum validation during download which has incorrect documentation as stated here:

https://bugzilla.redhat.com/show_bug.cgi?id=1139888

we also need to determine and document a method for resolving this for customers who experience this issue.
Comment 3 Ivan Necas 2014-09-23 05:39:01 EDT
I wonder, if this is similar to what was reported here https://www.redhat.com/archives/pulp-list/2013-December/msg00059.html

The issue I've seen is having a package synchronized and then re-signed upstream, the subsequential sync of the repository seemed to have this issue.
Comment 4 David Davis 2014-10-14 17:39:46 EDT
Without validation, it took 1 hr 12 min. With validation 1 hr 40 min.
Comment 5 Partha Aji 2014-10-14 17:46:19 EDT
Created redmine issue http://projects.theforeman.org/issues/7947 from this bug
Comment 6 David Davis 2014-10-14 17:50:58 EDT
Need to add documentation on how to enable checksum validation.
Comment 7 Bryan Kearney 2014-10-16 09:28:00 EDT
*** Bug 1153649 has been marked as a duplicate of this bug. ***
Comment 8 Bryan Kearney 2014-10-27 10:04:23 EDT
Moving to POST since upstream bug http://projects.theforeman.org/issues/7947 has been closed
-------------
David Davis
Applied in changeset commit:katello|cdb1f1e7b726ed1023dc34fa9c73627703277015.
Comment 17 Tazim Kolhar 2015-04-02 05:51:26 EDT
please provide verification steps
thanks
Comment 18 David Davis 2015-04-08 10:44:44 EDT
Mike can you provide verification steps?
Comment 19 Mike McCune 2015-04-27 19:33:24 EDT
TESTPLAN:

sync a repository you know has broken packages (size does not match the metadata).

I created a repo like this to assist with verification:

https://mmccune.fedorapeople.org/repos/broken-checksum/

Steps:

1) Create a custom repository with the URL https://mmccune.fedorapeople.org/repos/broken-checksum/

2) sync the repository

3) ensure that the task states that 1 package failed to download, see attached screenshot.

Even thou the task is still 'Success' this bug is just about ensuring that it notices the corrupted package which we did *not* notice in 6.0.

For the future bug about properly recording errors as a Warning see:

https://bugzilla.redhat.com/show_bug.cgi?id=1215838
Comment 20 Mike McCune 2015-04-27 19:33:56 EDT
Created attachment 1019462 [details]
TESTPLAN Screenshot - verify you see this in 6.1
Comment 21 Tazim Kolhar 2015-04-30 07:11:11 EDT
VERIFIED:

# rpm -qa | grep foreman
puppet-foreman_scap_client-0.3.3-8.el7sat.noarch
ruby193-rubygem-foreman_docker-1.2.0.10-1.el7sat.noarch
ruby193-rubygem-foreman-tasks-0.6.12.4-1.el7sat.noarch
rubygem-hammer_cli_foreman_discovery-0.0.1.8-1.el7sat.noarch
foreman-libvirt-1.7.2.18-1.el7sat.noarch
ruby193-rubygem-foreman_bootdisk-4.0.2.12-1.el7sat.noarch
foreman-compute-1.7.2.18-1.el7sat.noarch
foreman-ovirt-1.7.2.18-1.el7sat.noarch
ruby193-rubygem-foreman_discovery-2.0.0.12-1.el7sat.noarch
rubygem-hammer_cli_foreman_tasks-0.0.3.4-1.el7sat.noarch
foreman-debug-1.7.2.18-1.el7sat.noarch
foreman-postgresql-1.7.2.18-1.el7sat.noarch
qe-sat6-rhel71.usersys.redhat.com-foreman-client-1.0-1.noarch
qe-sat6-rhel71.usersys.redhat.com-foreman-proxy-1.0-1.noarch
ruby193-rubygem-foreman_gutterball-0.0.1.9-1.el7sat.noarch
foreman-1.7.2.18-1.el7sat.noarch
foreman-gce-1.7.2.18-1.el7sat.noarch
ruby193-rubygem-foreman_hooks-0.3.7-2.el7sat.noarch
rubygem-hammer_cli_foreman-0.1.4.10-1.el7sat.noarch
foreman-selinux-1.7.2.13-1.el7sat.noarch
foreman-vmware-1.7.2.18-1.el7sat.noarch
rubygem-hammer_cli_foreman_bootdisk-0.1.2.6-1.el7sat.noarch
foreman-proxy-1.7.2.4-1.el7sat.noarch
qe-sat6-rhel71.usersys.redhat.com-foreman-proxy-client-1.0-1.noarch
ruby193-rubygem-foreman-redhat_access-0.1.0-1.el7sat.noarch

1) Create a custom repository with the URL https://mmccune.fedorapeople.org/repos/broken-checksum/

2) sync the repository

3) ensure that the task states that 1 package failed to download, 

screenshot attached
Comment 22 Tazim Kolhar 2015-04-30 07:12:11 EDT
Created attachment 1020555 [details]
broken package
Comment 23 Bryan Kearney 2015-08-11 09:23:34 EDT
This bug is slated to be released with Satellite 6.1.
Comment 24 errata-xmlrpc 2015-08-12 01:16:15 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2015:1592

Note You need to log in before you can comment on or make changes to this bug.