Bug 621056

Summary: v7 failed after reboot test
Product: [Retired] Red Hat Hardware Certification Program Reporter: Qian Cai <qcai>
Component: Test Suite (tests)Assignee: Greg Nichols <gnichols>
Status: CLOSED ERRATA QA Contact: Qian Cai <qcai>
Severity: high Docs Contact:
Priority: high    
Version: 1.2CC: qcai, rlandry, ykun
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
After issuing the "v7 certify -m auto" command, v7 no longer fails after a reboot test.
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-09-20 12:13:27 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
reboot patch adding pause before shutdown -r
none
test.py patch moving waitForLull and assoc. functions to Test base class
none
cpuscaling test patch for moving waitForLull to Test base class
none
reboot patch adding pause before shutdown -r and before panic none

Description Qian Cai 2010-08-04 06:31:32 UTC
Description of problem:
After the "v7 certify -m auto", v7 could not continue after the reboot test.

# v7 continue -m auto
Traceback (most recent call last):
  File "/usr/bin/v7", line 45, in <module>
    success = v7.do(args)
  File "/usr/share/v7/lib/v7/hardwaretest.py", line 179, in do
    self.commands[self.command]()
  File "/usr/share/v7/lib/v7/hardwaretest.py", line 586, in doContinue
    self.load()
  File "/usr/share/v7/lib/v7/hardwaretest.py", line 210, in load
    self.certification.load(self.environment.getResultsPath())     
  File "/usr/share/v7/lib/v7/certificationtest.py", line 52, in load
    DocumentBase.load(self, filename)
  File "/usr/share/v7/lib/v7/documentbase.py", line 230, in load
    self.document = parse(file)
  File "/usr/lib64/python2.6/site-packages/_xmlplus/dom/minidom.py", line 1915, in parse
    return expatbuilder.parse(file)
  File "/usr/lib64/python2.6/site-packages/_xmlplus/dom/expatbuilder.py", line 930, in parse
    result = builder.parseFile(file)
  File "/usr/lib64/python2.6/site-packages/_xmlplus/dom/expatbuilder.py", line 211, in parseFile
    parser.Parse("", True)
xml.parsers.expat.ExpatError: no element found: line 1, column 0

Version-Release number of selected component (if applicable):
v7-1.2-14.el6

How reproducible:
often

Comment 1 Qian Cai 2010-08-04 06:38:26 UTC
"v7 print" also failed.

Comment 2 Greg Nichols 2010-08-04 14:37:46 UTC
Created attachment 436560 [details]
reboot patch adding pause before shutdown -r

This patch uses code written for the cpuscaling test (waitForLull) that calls sync, and check for low load before calling shutdown.  This should allow v7 to complete writing its results.xml before shutdown is called.

Comment 3 Greg Nichols 2010-08-04 14:39:07 UTC
Created attachment 436564 [details]
test.py patch moving waitForLull and assoc. functions to Test base class

Comment 4 Greg Nichols 2010-08-04 14:40:20 UTC
Created attachment 436566 [details]
cpuscaling test patch for moving waitForLull to Test base class

Comment 5 Greg Nichols 2010-08-05 15:28:19 UTC
Created attachment 436884 [details]
reboot patch adding pause before shutdown -r and before panic

Comment 8 qcui 2010-08-25 06:57:47 UTC
Verified it on el5.5 and el6 with R20.

Comment 11 errata-xmlrpc 2010-09-20 12:13:27 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2010-0702.html

Comment 12 Jaromir Hradilek 2010-09-21 09:24:52 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
After issuing the "v7 certify -m auto" command, v7 no longer fails after a reboot test.