Bug 754483 - Review Request: subscription-manager-migration-data - data for migration tooling
Summary: Review Request: subscription-manager-migration-data - data for migration tooling
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: Package Review
Version: 5.8
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: Alex Wood
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 188273 754492 757829 773030
TreeView+ depends on / blocked
 
Reported: 2011-11-16 16:24 UTC by Chris Duryee
Modified: 2012-05-29 19:05 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 757829 (view as bug list)
Environment:
Last Closed: 2012-05-29 19:05:28 UTC
Target Upstream Version:


Attachments (Terms of Use)

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


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