Bug 665128 - Candlepin should just store the product name in the pool.
Summary: Candlepin should just store the product name in the pool.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Candlepin
Classification: Community
Component: candlepin
Version: 0.5
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
: ---
Assignee: Devan Goodwin
QA Contact: John Sefler
URL:
Whiteboard:
Depends On:
Blocks: Entitlement-Beta
TreeView+ depends on / blocked
 
Reported: 2010-12-22 19:51 UTC by Bryan Kearney
Modified: 2015-05-14 15:22 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-02-23 21:51:27 UTC
Embargoed:


Attachments (Terms of Use)

Description Bryan Kearney 2010-12-22 19:51:05 UTC
Candlepin today attempts to calculate the prodcut name on the fly. It should, instead, just store it in the DB all the time. To do this, the following changes should be made:

1) When creating the pool, set the product name to the result of calling getProductNamesByProductId
2) Ensure this is persistted.
3) remove the calls to addProductName in the curator.

Comment 1 Devan Goodwin 2011-01-14 15:56:41 UTC
Fixed in candlepin.git:

master: f4609c8c4480c1ed8326b89612ed05d7a08ceae8
BETA: f9385c6f56b860a84892726c06eb56d1c59d95da

Comment 2 John Sefler 2011-01-24 15:24:00 UTC
Devan,Bryan,
Can you verify this bugzilla since it appears to be a change in code implementation with results that are transparent to the end user.

Comment 3 Devan Goodwin 2011-01-26 19:06:32 UTC
We've killed off the code that set these on the fly, added code to set them when pools are created. This code has been deployed in IT for awhile, product names are appearing healthy, unit tests passing, going to call it verified.


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