Bug 1519875

Summary:	Classification validation errors in seeding keep server from starting
Product:	Red Hat CloudForms Management Engine	Reporter:	Satoe Imaishi <simaishi>
Component:	Appliance	Assignee:	Chris Arcand <carcand>
Status:	CLOSED CURRENTRELEASE	QA Contact:	luke couzens <lcouzens>
Severity:	urgent	Docs Contact:
Priority:	high
Version:	5.8.0	CC:	abellott, carcand, gmccullo, gtanzill, jhardy, lcouzens, obarenbo, rspagnol, simaishi, smallamp
Target Milestone:	GA
Target Release:	5.9.0
Hardware:	All
OS:	All
Whiteboard:	upgrade:black
Fixed In Version:	5.9.0.12	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1507240	Environment:
Last Closed:	2018-03-06 15:12:49 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:
Embargoed:
Bug Depends On:	1507240
Bug Blocks:

Comment 2 CFME Bot 2017-12-01 16:27:03 UTC

New commit detected on ManageIQ/manageiq/gaprindashvili:
https://github.com/ManageIQ/manageiq/commit/7eecd58992e02055d8b799aad83e80b94c6e83e0

commit 7eecd58992e02055d8b799aad83e80b94c6e83e0
Author:     Gregg Tanzillo <gtanzill>
AuthorDate: Thu Nov 30 12:07:46 2017 -0500
Commit:     Satoe Imaishi <simaishi>
CommitDate: Fri Dec 1 11:01:46 2017 -0500

    Merge pull request #16568 from chrisarcand/fix-classification-seeding
    
    Skip seeding of categories if their creation is invalid
    (cherry picked from commit 48846ec50d8fee8be7d894148903b56a9285d855)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1519875

 app/models/classification.rb       | 17 ++++++++++-------
 spec/models/classification_spec.rb | 12 ++++++++++++
 2 files changed, 22 insertions(+), 7 deletions(-)

Comment 3 luke couzens 2017-12-14 17:16:41 UTC

Hey Chris, whats the best way to reproduce this one for verification?

Comment 4 Chris Arcand 2017-12-14 19:42:52 UTC

Change the name/description of a seeded Classification record in the Rails console and attempt to start the server.

Comment 5 luke couzens 2017-12-20 11:14:31 UTC

Verified in 5.9.0.13