Bug 800145 - Manifest import needs to be smarter about product attribute copying
Summary: Manifest import needs to be smarter about product attribute copying
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Subscription Asset Manager
Classification: Retired
Component: candlepin
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: Bryan Kearney
QA Contact: Tazim Kolhar
URL:
Whiteboard:
Depends On:
Blocks: sam12-tracker
TreeView+ depends on / blocked
 
Reported: 2012-03-05 20:19 UTC by James Bowes
Modified: 2016-04-26 00:54 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-02-21 19:15:27 UTC
Embargoed:
tkolhar: needinfo+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2013:0544 0 normal SHIPPED_LIVE Important: Subscription Asset Manager 1.2 update 2013-02-26 04:08:04 UTC

Description James Bowes 2012-03-05 20:19:16 UTC
When you do a manifest import that updates existing products, if the product has modified attributes, those attributes should be copied onto all pools of all orgs that reference that product as their toplevel product (and then any related entitlements need to be regenerated)

Comment 2 James Bowes 2012-10-10 14:32:49 UTC
Fixed in candlepin master, 0ceb1369
Will be in candlepin 0.7.13

Comment 4 Tazim Kolhar 2012-11-27 05:01:40 UTC
Hi all,
Please provide the verification steps to complete the testing. thanks

Comment 5 Devan Goodwin 2012-12-03 14:55:54 UTC
I think it's safe to assume this cannot be QE tested. The steps to reproduce would involve creating custom products and subscriptions in two orgs each using the same custom product, generating manifests for each org, importing each on same candlepin downstream, going back upstream and then changing the attributes on the custom product, generating a new manifest in either org, importing it again downstream, and verifying that the changed attributes are applied to both orgs downstream, not just the one that did the import.

James' actual code went in commit 18f3dd2dd5b62c97bded834efe7dcd8c1f8db965, and looks to contain very good test coverage.

Comment 6 Tazim Kolhar 2012-12-03 15:39:37 UTC
Verified by developers.

Comment 10 errata-xmlrpc 2013-02-21 19:15:27 UTC
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.

http://rhn.redhat.com/errata/RHSA-2013-0544.html


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