WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Xend domain_lookup XendInvalidDomain error reporting patch

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] Xend domain_lookup XendInvalidDomain error reporting patch
From: "Yung Giang" <ygiang@xxxxxxxxx>
Date: Wed, 28 Feb 2007 16:47:30 -0500
Delivery-date: Thu, 01 Mar 2007 01:29:17 -0800
Dkim-signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:mime-version:content-type; b=fDLY/Iv5PlCrOrq+bGF392ykfzEbYtW3JxKVIWaNvDTm+qrG6H0a3xX77Q+puAp0GU4wDtIF105VDEIUkr6F6KrLi/KeZ0g+nIeuTFrOMOl25mD5wzO5xSKCePUgU4ke1wUga08iMeawF34CnFXou05h3t/ZMXBrDBsoDq3CQic=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:mime-version:content-type; b=LYcjzjrHd03WwQ+1pcgSZDQlVaXoAqM9tmpuaYgVG4BrrE3yGEDa1edaXyxqxGMa8YZ3ApTUXqiQjWPW67rfdaJvOv1sCcSQYzjhmrKBacRJN5nr3rq/MqqGzXUpokeAJ0SXm7S8TqJMawS7jr8uaq6eI+6jwFvrq2xe7JMUHSA=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Hi,



The error reporting when a domain is invalid is currently XendError (raise XendError("No domain named '%s'." % str(domid))), this should really be XendInvaildDomain (raise XendInvalidDomain ("No domain named '%s'." % str(domid)) for the correct Xend error reporting. And since Xend is already raising XendInvaildDomain exception in the domain_lookup function we don't need check the same exception again in functions that use domain_lookup.

Here is the patch that should fix this problem, please take a look. Thanks! 

-Yung Giang
ygiang@xxxxxxxxx

Attachment: xend_domain_lookup_error_report.patch
Description: Binary data

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-devel] Xend domain_lookup XendInvalidDomain error reporting patch, Yung Giang <=