Bug 1026646 - Better to disable "enter" button works for the fan status table of ipmi page
Summary: Better to disable "enter" button works for the fan status table of ipmi page
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-node
Version: unspecified
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: 3.4.1
Assignee: Ryan Barry
QA Contact: bugs@ovirt.org
URL:
Whiteboard: node
Depends On:
Blocks: 1026647
TreeView+ depends on / blocked
 
Reported: 2013-11-05 07:07 UTC by haiyang,dong
Modified: 2014-04-30 11:03 UTC (History)
13 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1026647 (view as bug list)
Environment:
Last Closed: 2014-04-30 11:03:07 UTC
oVirt Team: ---


Attachments (Terms of Use)
attached Screenshot for hang page (15.34 KB, image/png)
2013-11-05 07:07 UTC, haiyang,dong
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 21929 0 None None None Never

Description haiyang,dong 2013-11-05 07:07:06 UTC
Created attachment 819545 [details]
attached Screenshot for hang page

Description of problem:
Using "edit-node --repo=edit-node.repo --install=ovirt-node-plugin-ipmi-3.1.0-0.999.20131105052117git6b9cff5.el6.noarch.rpm ovirt-node-iso-3.1.0-0.999.1181.el6.iso
" to install ipmi plugin into rhev-h.
It will install ovirt-node-plugin-ipmi ipmitool OpenIPMI and OpenIPMI-libs package.
Then clean install this edited iso on DELL PowerEdge R510,login rhev-h.
Enter into IPMI page and check that the "IPMI Enabled" show "True" and Fan Status also was shown in IPMI page.

But the "enter" button works for the fan status table of ipmi page, if users selected one item of the fan status table
and click "enter" button, rhev-h will hang on(Seen hang page.png).
Also "enter" button works wasn't necessary for the fan status table of ipmi page,
so it's better to disable "enter" button works for the fan status table of ipmi page

Version-Release number of selected component (if applicable):
ovirt-node-iso-3.1.0-0.999.1181.el6.iso
ovirt-node-plugin-ipmi-3.1.0-0.999.20131105052117git6b9cff5.el6.noarch.rpm

How reproducible:
100%

Steps to Reproduce:

Actual results:


Expected results:

Additional info:

Comment 1 Ryan Barry 2013-12-02 18:48:07 UTC
I wasn't able to actually reproduce a hang on an R300, though that's definitely output from "ipmitool -E open chassis status", and it looks like it may have been passed through to stdout.

I was able to make the UI scroll off the screen by pressing enter (though the arrow keys still responded and I was able to change screens as normal).

I don't see a clean way to resolve disabling input on a Table by Table basis, and changing the codebase to add parameters for something which is only problematic here and readily resolved by catching the output seems intrusive.

That said, if this patch doesn't resolve or it's otherwise desirable to add the ability to disable input on TableEntries, I'll definitely investigate it.

Comment 2 Sandro Bonazzola 2014-03-04 09:18:59 UTC
This is an automated message.
Re-targeting all non-blocker bugs still open on 3.4.0 to 3.4.1.


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