Bug 754483

Summary: Review Request: subscription-manager-migration-data - data for migration tooling
Product: Red Hat Enterprise Linux 5 Reporter: Chris Duryee <cduryee>
Component: Package ReviewAssignee: Alex Wood <awood>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8CC: awood, jmolet, jsefler, mkhusid, notting, pm-rhel
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 757829 (view as bug list) Environment:
Last Closed: 2012-05-29 19:05:28 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 188273, 754492, 757829, 773030    

Description Chris Duryee 2011-11-16 16:24:49 UTC 
Spec URL: http://axiom.rdu.redhat.com/git/gitweb.cgi?p=subscription-manager-migration-data;a=blob_plain;f=subscription-manager-migration-data.spec;hb=HEAD
SRPM URL: http://download.devel.redhat.com/brewroot/work/tasks/4027/3824027/subscription-manager-migration-data-1.1-1.el5.src.rpm
Description: This package provides certificates for migrating a system from RHN Classic to RHSM.

This package is to be installed alongside subscription-manager-migration (a package that is generated from the existing subscription-manager src rpm). It contains only pem certificates and a mapping file.

The rationale behind breaking this out into a new package is to allow us to keep our existing codebase on fedorahosted.org without having to put product data in our code repository. Additionally, it gives us the option to update the product mappings without requiring an update to subscription-manager.
Comment 1 Chris Duryee 2011-11-16 19:09:14 UTC 
The maintainer for this package is awood (cc'd)
Comment 2 Chris Duryee 2011-11-17 20:27:24 UTC 
For package review, here are the only parts that stood out:

* the license in the spec is currently set to "Copyright 2011 Red Hat, Inc.  All rights reserved.". We believe this is ok, but mkhusid is following up w/ legal to make sure.

* rpmlint warned on invalid source tarball URL. This is ok since the rpm is built with tito, and the source tarball is generated when the srpm is built.
Comment 9 John Sefler 2011-12-14 20:07:10 UTC 
I am not an rpm expert, so I am mostly relying on the developers review in comment 2 to move this bug to VERIFIED.

Other verification support:
- license bug 757829 has been VERIFIED
- errata https://errata.devel.redhat.com/tps/errata_results/15350 is passing TPS

moving to VERIFIED