Bug 598544 - Control-C interrupt of v7 often results in broken results.xml
Control-C interrupt of v7 often results in broken results.xml
Status: CLOSED ERRATA
Product: Red Hat Hardware Certification Program
Classification: Red Hat
Component: Test Suite (harness) (Show other bugs)
1.1
All Linux
low Severity medium
: ---
: ---
Assigned To: Greg Nichols
CAI Qian
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-01 11:31 EDT by Greg Nichols
Modified: 2010-09-21 08:54 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Interrupting the v7 process using the Control-C keystroke now does not result in corrupted test results.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-09-20 08:14:21 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
documentbase.py patch with exception handling (1.44 KB, patch)
2010-06-01 11:38 EDT, Greg Nichols
no flags Details | Diff
documentbase.py patch with exception handling (933 bytes, patch)
2010-06-01 19:19 EDT, Greg Nichols
no flags Details | Diff

  None (edit)
Description Greg Nichols 2010-06-01 11:31:30 EDT
Description of problem:

When v7 is interrupted by the user via Control-C, a broken results.xml file is often produced.

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

v7 1.1, 1.2

How reproducible:

very

Steps to Reproduce:

When a test is running, or at a test prompt, hit control-c twice - once to kill the test, then again for the v7 harness.

Then try "v7 print" and look for a traceback as v7 tries to parse the results.xml file.
Comment 1 Greg Nichols 2010-06-01 11:38:20 EDT
Created attachment 418689 [details]
documentbase.py patch with exception handling

This patch handles the KeyboardInterupt exception generated via control-c during saving of v7's results.xml file.  The handler prints a message asking the user to wait for the file to be saved, then raises the exception.   

This allows v7 to be killed via control-c, but minimizes the chance of it breaking the results.xml.
Comment 2 Greg Nichols 2010-06-01 19:19:23 EDT
Created attachment 418855 [details]
documentbase.py patch with exception handling

Removed spurious change found in previous patch.
Comment 5 Caspar Zhang 2010-09-09 04:42:16 EDT
(In reply to comment #1)
> Created attachment 418689 [details]
> documentbase.py patch with exception handling
> 
> This patch handles the KeyboardInterupt exception generated via control-c
> during saving of v7's results.xml file.  The handler prints a message asking
> the user to wait for the file to be saved, then raises the exception.   
> 
> This allows v7 to be killed via control-c, but minimizes the chance of it
> breaking the results.xml.

>=v7-1.2-R21 does print a message asking the user to wait. So we can move the bug state to 'VERIFIED'.
Comment 8 errata-xmlrpc 2010-09-20 08:14:21 EDT
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 9 Jaromir Hradilek 2010-09-21 08:54:19 EDT
    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:
Interrupting the v7 process using the Control-C keystroke now does not result in corrupted test results.

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