Bug 773208 – Migration with non-existent xml does not report error

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 773208 - Migration with non-existent xml does not report error

Summary:	Migration with non-existent xml does not report error

Status:	CLOSED ERRATA

Aliases:	None

Product:	Red Hat Enterprise Linux 6
Classification:	Red Hat
Component:	libvirt (Show other bugs)
Sub Component:
Version:	6.3
Hardware:	x86_64 Linux

Priority	medium Severity medium
Target Milestone:	rc
Target Release:	---
Assigned To:	Alex Jia
QA Contact:	Virtualization Bugs
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2012-01-11 04:14 EST by weizhang
Modified:	2012-06-20 02:46 EDT (History)
CC List:	7 users (show)

See Also:
Fixed In Version:	libvirt-0.9.10-1.el6
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2012-06-20 02:46:18 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)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHSA-2012:0748	normal	SHIPPED_LIVE	Low: libvirt security, bug fix, and enhancement update	2012-06-19 15:31:38 EDT

Groups: None (edit)

Description weizhang 2012-01-11 04:14:56 EST

Description of problem:
Migration with non-existent xml does not report error messages although it returns non-zero value


Version-Release number of selected component (if applicable):
kernel-2.6.32-223.el6.x86_64
libvirt-0.9.9-1.el6.x86_64
qemu-kvm-0.12.1.2-2.213.el6.x86_64

How reproducible:
100%

Steps to Reproduce:
1. Prepare migration environment with shared nfs dir and start a guest

2. Do migration with
# virsh migrate kvm-rhel6u2-x86_64-new --live qemu+ssh://10.66.83.197/system --xml non-existent-xml
no error messages here and migration returns immediately 

3. Check return value
# echo $?
1
  
Actual results:
It does not report any error messages

Expected results:
It should report error messages

Additional info:

Comment 2 Alex Jia 2012-01-11 04:23:06 EST

Patch for upstream:
https://www.redhat.com/archives/libvir-list/2012-January/msg00388.html

Comment 3 Alex Jia 2012-01-11 09:53:33 EST

In POST:
commit c41a21de3abb6a080d87360ed7b246b070703b88
Author: Alex Jia <ajia@redhat.com>
Date:   Wed Jan 11 16:23:03 2012 +0800

    virsh: improve doMigrate function docs
    
    When running virsh migrate with --xml option and actual xml file doesn't
    exist, virsh hasn't output any error information, although return value
    is 1.
    
    * tools/virsh.c: Raising a appropriate error information when operation fails.
    
    * How to reproduce?
    % virsh migrate <domain> --live qemu+ssh://<target host>/system --xml non-existent.xml
    % echo $?
    
    * Fixed result:
    error: file 'non-existent.xml' doesn't exist
    
    Signed-off-by: Alex Jia <ajia@redhat.com>

Comment 5 weizhang 2012-02-14 06:11:12 EST

Verify pass on 
libvirt-0.9.10-1.el6.x86_64
qemu-kvm-0.12.1.2-2.225.el6.x86_64
kernel-2.6.32-230.el6.x86_64

# virsh migrate --live tt qemu+ssh://10.66.85.218/system --xml none-exist.xml
error: file 'none-exist.xml' doesn't exist

Comment 7 errata-xmlrpc 2012-06-20 02:46:18 EDT

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.

http://rhn.redhat.com/errata/RHSA-2012-0748.html

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