Bug 1393349

Summary: appliance_console crash when running Logfile Configuration without setting up database first
Product: Red Hat CloudForms Management Engine Reporter: luke couzens <lcouzens>
Component: ApplianceAssignee: Joe Vlcek <jvlcek>
Status: CLOSED CURRENTRELEASE QA Contact: luke couzens <lcouzens>
Severity: medium Docs Contact:
Priority: high    
Version: 5.7.0CC: abellott, cpelland, dajohnso, jhardy, jvlcek, obarenbo
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: black
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1411878 (view as bug list) Environment:
Last Closed: 2017-06-12 16:25:16 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:
Bug Depends On:    
Bug Blocks: 1411878    

Description luke couzens 2016-11-09 11:33:08 UTC
Description of problem:appliance console crashes when configuring new logfile disk if there is no database setup on the appliance


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


How reproducible:100%


Steps to Reproduce:
1.provision appliance with additional disk
2.run appliance_console
3.select option 11 'Logfile Configuration'
4.follow through configuration of new disk

Actual results: Appliance console seems to configure disk but crashes out


Expected results: Disk is configured correctly without crash


Additional info:http://pastebin.test.redhat.com/428329

Comment 3 CFME Bot 2017-01-09 21:32:43 UTC
New commit detected on ManageIQ/manageiq-gems-pending/master:
https://github.com/ManageIQ/manageiq-gems-pending/commit/ae955d2d8265c26dbea33e76625d957d03813291

commit ae955d2d8265c26dbea33e76625d957d03813291
Author:     Joe VLcek <jvlcek>
AuthorDate: Thu Jan 5 10:56:39 2017 -0500
Commit:     Joe VLcek <jvlcek>
CommitDate: Thu Jan 5 10:56:39 2017 -0500

    Do not stop and start evmserverd if it was not running.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1393349

 .../appliance_console/logfile_configuration.rb     |  5 ++-
 .../logfile_configuration_spec.rb                  | 52 +++++++++++++++-------
 2 files changed, 41 insertions(+), 16 deletions(-)

Comment 5 CFME Bot 2017-01-13 18:25:59 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/5020dd8299cbf5037ef7b3ba648e0ab7b939c3b5

commit 5020dd8299cbf5037ef7b3ba648e0ab7b939c3b5
Author:     Joe VLcek <jvlcek>
AuthorDate: Thu Jan 12 14:36:00 2017 -0500
Commit:     Joe VLcek <jvlcek>
CommitDate: Thu Jan 12 17:11:35 2017 -0500

    [EUWE] Do not stop and start evmserverd if it was not running
    
    Manually pulled commit ManageIQ/manageiq-gems-pending/commit/5bf80c831ea436c855e4ba1beccea3883a9aa174
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1393349

 .../appliance_console/logfile_configuration.rb       | 11 ++++++++---
 .../appliance_console/logfile_configuration_spec.rb  | 20 ++++++++++++++++++--
 2 files changed, 26 insertions(+), 5 deletions(-)

Comment 6 luke couzens 2017-02-28 12:27:26 UTC
Verified in 5.8.0.2