Bug 657187 - Output message is a little confused when starting xendomains with invalid image
Summary: Output message is a little confused when starting xendomains with invalid image
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: xen
Version: 5.6
Hardware: Unspecified
OS: Linux
low
low
Target Milestone: rc
: ---
Assignee: Michal Novotny
QA Contact: Virtualization Bugs
URL:
Whiteboard:
Depends On:
Blocks: 514498
TreeView+ depends on / blocked
 
Reported: 2010-11-25 06:40 UTC by Binbin Yu
Modified: 2014-02-02 22:38 UTC (History)
9 users (show)

Fixed In Version: xen-3.0.3-126.el5
Doc Type: Bug Fix
Doc Text:
When starting or stopping the xendomains service with an invalid image and a regular saved domain, the formatting of the output messages was confusing and hard to read. With this update, the output has been clarified.
Clone Of:
Environment:
Last Closed: 2011-07-21 09:17:35 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Improve xendomains output (2.50 KB, patch)
2011-03-09 08:05 UTC, Michal Novotny
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2011:1070 0 normal SHIPPED_LIVE xen bug fix and enhancement update 2011-07-21 09:12:56 UTC

Description Binbin Yu 2010-11-25 06:40:50 UTC
Description of problem:
The output message is a little confused when starting xendomains with invalid image and regular saved domain.


Version-Release number of selected component (if applicable):
host: x86_64, kernel-xen-2.6.18-233.el5, xen-3.0.3-119.el5
guest: rhel5.5-32bit-pv


How reproducible:
always

Steps to Reproduce:
1.Set the XENDOMAINS_SAVE option to /var/lib/xen/save
2.Set the XENDOMAINS_RESTORE option to true
3.Set the XENDOMAINS_RESTOREFAILTYPE to 'rename'
4.Create a regular domain
5.Create the invalid image under $XENDOMAINS_SAVE
#dd if=/dev/zero of=/var/lib/xen/save/invalid bs=1M count=50
6.Stop xendomains service
7.Start xendomains service


Actual results:
After step 7, the output message is a little confused. The "!" stands alone. Whether the "[failed]" shows the regular domain rhel5u5-32-pv restore failure or this "start xendomains service" operation failure?

......
[host]# /etc/init.d/xendomains start
Restoring Xen domains: invalid
FAILED: Invalid restore image /var/lib/xen/save/invalid
Restore image has been renamed to /var/lib/xen/save/.invalid

An error occured while restoring domain invalid:
Error: Restore failed
!
 rhel5u5-32-pv.
[failed]  
                                                 [FAILED]


Expected results:
Devel can design and make the output more clear.


Additional info:
1.The output message is also not neat when stoping xendomains to shutdown auto-created domain, because "[ OK ]" mixed with ".....[done]"
[host]# /etc/init.d/xendomains stop
Shutting down Xen domains: rhel5u5-64-pv(shut).............[  OK  ]...[done]

Comment 1 Binbin Yu 2010-11-25 06:41:51 UTC
 If devel think this is not a problem, just feel free to close it.

Comment 2 Michal Novotny 2011-03-02 10:34:40 UTC
(In reply to comment #0)
> Description of problem:
> The output message is a little confused when starting xendomains with invalid
> image and regular saved domain.
> 
> 
> Version-Release number of selected component (if applicable):
> host: x86_64, kernel-xen-2.6.18-233.el5, xen-3.0.3-119.el5
> guest: rhel5.5-32bit-pv
> 
> 
> How reproducible:
> always
> 
> Steps to Reproduce:
> 1.Set the XENDOMAINS_SAVE option to /var/lib/xen/save
> 2.Set the XENDOMAINS_RESTORE option to true
> 3.Set the XENDOMAINS_RESTOREFAILTYPE to 'rename'
> 4.Create a regular domain
> 5.Create the invalid image under $XENDOMAINS_SAVE
> #dd if=/dev/zero of=/var/lib/xen/save/invalid bs=1M count=50
> 6.Stop xendomains service
> 7.Start xendomains service
> 
> 
> Actual results:
> After step 7, the output message is a little confused. The "!" stands alone.
> Whether the "[failed]" shows the regular domain rhel5u5-32-pv restore failure
> or this "start xendomains service" operation failure?
> 
> ......
> [host]# /etc/init.d/xendomains start
> Restoring Xen domains: invalid
> FAILED: Invalid restore image /var/lib/xen/save/invalid
> Restore image has been renamed to /var/lib/xen/save/.invalid
> 
> An error occured while restoring domain invalid:
> Error: Restore failed
> !
>  rhel5u5-32-pv.
> [failed]  
>                                                  [FAILED]
> 
> 
> Expected results:
> Devel can design and make the output more clear.
> 
> 
> Additional info:
> 1.The output message is also not neat when stoping xendomains to shutdown
> auto-created domain, because "[ OK ]" mixed with ".....[done]"
> [host]# /etc/init.d/xendomains stop
> Shutting down Xen domains: rhel5u5-64-pv(shut).............[  OK  ]...[done]


Hi Binbin,
I've tried to design and implement some better output and here are the results but I'm not sure it's good enough from the user's perspective.

This example output is done for 2 guests, rhel5-32pv and rhel5-64fv, that are both running on the host and also we need to create the invalid file. Touch command is good enough to create the invalid save file so here's the output including the commands used:

# touch /var/lib/xen/save/invalid; /etc/init.d/xendomains start
Restoring Xen domains:  invalid
FAILED: Invalid restore image /var/lib/xen/save/invalid
Restore image has been renamed to /var/lib/xen/save/.invalid

An error occured while restoring domain invalid:
Error: Restore failed !

Going on... Restoring domain rhel5-32pv
Domain rhel5-32pv restored successfully
 rhel5-64fv
Domain rhel5-64fv restored successfully
.
# 

Is that good? What do you think?

Thanks,
Michal

Comment 3 Qixiang Wan 2011-03-09 07:47:00 UTC
Hi Michal,
I think the improved output show more clear information and enough for the enhancement.

Comment 5 Michal Novotny 2011-03-09 08:05:34 UTC
Created attachment 483122 [details]
Improve xendomains output

Hi,
this is the patch for to implement a better xendomains script output.

It's been tested on RHEL-5 x86_64 dom0 with the patch applied and the
formatting seems to be a little better and not that much confused now.

Also, the output has been reviewed by Qixiang Wan <qwan> and
according to Qixiang the output is improved and the information output
are more clear now.

Michal

Comment 8 Miroslav Rezanina 2011-03-17 09:55:09 UTC
Fix built into xen-3.0.3-126.el5

Comment 10 Qixiang Wan 2011-04-22 07:37:03 UTC
verified the output has been improved with xen-3.0.3-129.el5.

$ /etc/init.d/xendomains start
Restoring Xen domains: invalid
FAILED: Invalid restore image /var/lib/xen/save/invalid
Restore image has been renamed to /var/lib/xen/save/.invalid

An error occured while restoring domain invalid:
Error: Restore failed!

[failed]                                                   [FAILED]

Comment 11 Tomas Capek 2011-07-13 13:27:59 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:
When starting or stopping the xendomains service with an invalid image and a regular saved domain, the formatting of the output messages was confusing and hard to read. With this update, the output has been clarified.

Comment 12 errata-xmlrpc 2011-07-21 09:17:35 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-2011-1070.html

Comment 13 errata-xmlrpc 2011-07-21 11:59:37 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-2011-1070.html


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