Bug 1317598 - CDN repo metadata checksum does not match ISO repo metadata checksum, resulting in redownload of content
Summary: CDN repo metadata checksum does not match ISO repo metadata checksum, resulti...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: releng
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
high vote
Target Milestone: Unspecified
Assignee: Lisa S
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-03-14 15:58 UTC by Kedar Bidarkar
Modified: 2018-08-02 20:58 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-08-02 20:58:18 UTC
Target Upstream Version:


Attachments (Terms of Use)
disconnected mode sync status (58.98 KB, image/png)
2016-03-14 16:58 UTC, Kedar Bidarkar
no flags Details
connected mode sync status (58.18 KB, image/png)
2016-03-14 17:05 UTC, Kedar Bidarkar
no flags Details

Description Kedar Bidarkar 2016-03-14 15:58:11 UTC
Description of problem:
A Satellite6 setup in disconnected mode when again connected  to cdn , end up resyncing whole content again.

Syncing content in disconnected mode:

Space occupied: 31 GB
Repos enabled: RHEL7_optional, RHEL7_x86_64, RHEL7_Sat6_Tools.
Disconnected URL : http://abc.redhat.com/pub/sat-import

After Syncing Content with cdn URL:

Space Occupied: 56 GB
Repos enabled: RHEL7_optional_x86_64, RHEL7_OS_x86_64, RHEL7_Sat6_Tools.

Version-Release number of selected component (if applicable):
Sat6.2-SNAP3-compose1

How reproducible:


Steps to Reproduce:
1. 
2.
3.

Actual results:
Disconnected to Connected setup resync all the rpm/content.


Expected results:
Should be syncing only the non-existent rpm/content.

Additional info:

Comment 1 Kedar Bidarkar 2016-03-14 16:58:48 UTC
Created attachment 1136212 [details]
disconnected mode sync status

The screenshot of sync page with repos: RHEL7_OS_X86_64 , RHEL7_OPTIONAL_X86_64 and RHEL7_SATTOOLS

Comment 2 Kedar Bidarkar 2016-03-14 17:05:36 UTC
Created attachment 1136226 [details]
connected mode sync status

The screenshot of sync page with repos: RHEL7_OS_X86_64 , RHEL7_OPTIONAL_X86_64 and RHEL7_SATTOOLS

Comment 3 Kedar Bidarkar 2016-03-14 17:12:51 UTC
In comment1 screenshot please note, the "new package" count: 73 for sattools. 
In comment2 screenshot please note, the "new package" count: 10 for sattools.


But this is not the case with other repos which were synced with it, RHEL7_{optional and OS}, they seem to be syncing all the content once again.

Comment 6 Chris Duryee 2016-03-15 17:41:44 UTC
Were all repos set as "immediate"?

Comment 7 Michael Hrivnak 2016-03-15 19:33:06 UTC
The RHEL7 repo on the CDN appears to have metadata generated with sha1, vs the iso whose repo metadata is sha256. Unfortunately, pulp sees that the rpms from the CDN have different checksums than those on the ISO and is not able to determine that they are the same, so pulp re-downloads each rpm.

This has previously resulted in double download and storage of rpms in the case where one rhel7 repo used sha1, but another similar repo (with mostly the same RPMs) used sha256.

The only quick solution is to ensure that the metadata in each repo on the CDN is using the same checksum type as the corresponding repo on the ISO.

Long-term we'll make pulp more flexible regarding checksum algorithms, but that is a way off and will require substantial change to pulp's data model.

Comment 10 Kedar Bidarkar 2016-03-16 09:23:19 UTC
Yes, all repos were set as "immediate", before syncing content in disconnected mode.

This was done by changing the download_policy to "immediate" via settings --> katello --> "default_download_policy" : "immediate"

Comment 19 Michael Hrivnak 2017-08-15 18:42:27 UTC
Pulp's export distributor now has a checksum_type option. It's documented here all the way at the bottom, called "checksum_type". Let us know if there are any questions about its use.

http://docs.pulpproject.org/plugins/pulp_rpm/tech-reference/export-distributor.html#configuration-parameters

It was introduced upstream in Pulp 2.9.0.

"Checksum type to use for metadata generation. For any units where the checksum of this type is not already known, it will be computed on-the-fly and saved for future use."

Comment 21 Bryan Kearney 2018-08-02 20:58:18 UTC
Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in the product in the forseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.


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