[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] Xend domain_lookup XendInvalidDomain error reporting patch


  • To: xen-devel@xxxxxxxxxxxxxxxxxxx
  • From: "Yung Giang" <ygiang@xxxxxxxxx>
  • Date: Wed, 28 Feb 2007 16:47:30 -0500
  • Delivery-date: Thu, 01 Mar 2007 01:29:17 -0800
  • 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=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.