Bug 1391929

Summary:	appliance_console crash when running Logfile Configuration
Product:	Red Hat CloudForms Management Engine	Reporter:	luke couzens <lcouzens>
Component:	Appliance	Assignee:	Joe Vlcek <jvlcek>
Status:	CLOSED CURRENTRELEASE	QA Contact:	luke couzens <lcouzens>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.7.0	CC:	abellott, cpelland, dajohnso, jhardy, jvlcek, obarenbo, simaishi
Target Milestone:	GA	Keywords:	TestOnly
Target Release:	5.7.0	Flags:	jvlcek: needinfo? (dajohnso)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	black
Fixed In Version:	5.7.0.13	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-01-11 20:18:46 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:

Description luke couzens 2016-11-04 12:21:53 UTC

Description of problem: appliance_console crashes if you setup Logfile configuration without adding a new partition


Version-Release number of selected component (if applicable):5.7.0.9


How reproducible:100%


Steps to Reproduce:
1.provision 5.7 appliance
2.select option 11 ' Logfile Configuration'
3.say yes to 'configure a new logfile disk volume?'
4.say yes to 'Are you sure you don't want to partition the logfile disk?' 

Actual results:appliance console crash


Expected results:return 'No partition found for logfile disk. You probably want to add an unpartitioned disk and try again.' and exit back to appliance console


Additional info:
http://pastebin.test.redhat.com/427053
If you say no to 'Are you sure you don't want to partition the logfile disk?' it returns to appliance console.

Comment 2 Dave Johnson 2016-11-15 18:16:49 UTC

We could potentially address bug 1393349 while chasing this one as its so closely related.

Comment 3 Joe Vlcek 2016-11-21 14:22:40 UTC

Please confirm if this is really a blocker?

I agreed this is a nuisance but the issue only happens when the user tries
to configure a bigger disk to store log files on, when they failed to add
an extra disk to the appliance.

Comment 4 CFME Bot 2016-11-23 18:26:39 UTC

https://github.com/ManageIQ/manageiq/pull/12823

Comment 5 CFME Bot 2016-11-28 14:56:01 UTC

New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/2b8fef980c8977cc15e30195bb8efd3e8e344db3

commit 2b8fef980c8977cc15e30195bb8efd3e8e344db3
Author:     Joe VLcek <jvlcek>
AuthorDate: Wed Nov 23 13:03:25 2016 -0500
Commit:     Joe VLcek <jvlcek>
CommitDate: Wed Nov 23 13:03:25 2016 -0500

    When no disks are found gracefully return
    
    This change is only needed on the euwe branch.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1391929

 gems/pending/appliance_console/logfile_configuration.rb           | 1 +
 gems/pending/spec/appliance_console/logfile_configuration_spec.rb | 8 ++++++++
 2 files changed, 9 insertions(+)

Comment 7 luke couzens 2016-12-01 10:37:09 UTC

Verified in 5.7.0.13