Bug 1321109 - Hang on invalid static ip
Summary: Hang on invalid static ip
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Appliance
Version: 5.4.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.6.0
Assignee: Šimon Lukašík
QA Contact: luke couzens
URL:
Whiteboard: appliance:black
Depends On:
Blocks: 1334934
TreeView+ depends on / blocked
 
Reported: 2016-03-24 16:54 UTC by luke couzens
Modified: 2016-06-29 15:44 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1334934 (view as bug list)
Environment:
Last Closed: 2016-06-29 15:44:41 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:1348 0 normal SHIPPED_LIVE CFME 5.6.0 bug fixes and enhancement update 2016-06-29 18:50:04 UTC

Description luke couzens 2016-03-24 16:54:06 UTC
Description of problem:setting invalid ip cause an infinite hang when applying


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


How reproducible:100%


Steps to Reproduce:
1.ssh into appliance
2.start appliance_console
3.setup static ip with invalid ip address


Actual results:Applying static network configuration (never finishes)


Expected results:invalid ip address 


Additional info:

Static Network Configuration

        IP Address:      01.01.01.01
        Netmask:         255.255.252.0
        Gateway:         10.16.7.254
        Primary DNS:     10.10.160.1
        Secondary DNS:   
        Search Order:    rhq.lab.eng.bos.redhat.com

Apply static network configuration? (Y/N)
y

Applying static network configuration...

Comment 3 luke couzens 2016-04-01 18:11:13 UTC
Bug also in 5.5.3.2

Comment 4 Šimon Lukašík 2016-04-12 08:52:10 UTC
Luke, I cannot reproduce this in 5.5.3.2. The 5.5.3.2 seems to have nice
error message 

    01.01.01.01 is not a valid IPv4 or IPv6 address> (NoMethodError)

instead of hanging on. This has been fixed by Nick, who has rewrote
static network handling in

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

I am flipping this ON_QA. Please let me know if this is not appropriate
state in this case.

Comment 5 luke couzens 2016-04-12 09:40:35 UTC
Hi Simon,

Just checked this and you are correct, however this error still crashes out of appliance console. I would expect it to just request a correct IP not require the user to launch appliance_console again.

Hope this helps.

Comment 6 Šimon Lukašík 2016-04-12 15:40:15 UTC
Yeah, right. It crashes. I have fixed the crash in https://github.com/ManageIQ/manageiq/pull/7823

Can you confirm that the traceback contains word 'massage'?

Comment 7 luke couzens 2016-04-26 18:18:40 UTC
Verified in 5.6.0.4-beta2.3

Comment 8 luke couzens 2016-05-04 15:11:33 UTC
Appliance_console also crashes in 5.5.4.0

Comment 11 errata-xmlrpc 2016-06-29 15:44:41 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.

https://access.redhat.com/errata/RHBA-2016:1348


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