From: Anthony Liguori <aliguori@us.ibm.com>
To: Ewan Mellor <ewan@xensource.com>
Cc: Ian Pratt <m+Ian.Pratt@cl.cam.ac.uk>,
xen-devel <xen-devel@lists.xensource.com>
Subject: Re: XML-RPC for Xend status
Date: Tue, 21 Mar 2006 10:05:11 -0600 [thread overview]
Message-ID: <44202437.5060800@us.ibm.com> (raw)
In-Reply-To: <20060321154715.GC6745@leeni.uk.xensource.com>
Ewan Mellor wrote:
> On Mon, Mar 20, 2006 at 10:39:13AM -0600, Anthony Liguori wrote:
>
> Could you send your patches through anyway? I'll start some tests at this end
> too. If you don't want to send them through the list until you've got a
> clean regression test result, that's fine, but I can get a regression test
> running here too, either way.
>
Yeah, I'll submit it this afternoon once I head into the office. Once I
saw 2.6.16, I realized we're out of time :-) I hope we can fix the
xm-test regressions though before 3.0.2...
> Did you give any more thought to error handling? The version you sent out
> didn't do any rerouting of exceptions, so if VmError or XendError was raised,
> that turned into ProtocolError, which gives a revolting splat at the client.
> I think that the best thing to do is to catch those exceptions at the server
> and turn them into generic Faults, and then after 3.0.2 is out we can shuffle
> through turning the VmError or XendErrors into more specific Faults with
> proper error codes etc.
>
Yeah, I just allocated a faultcode for VmError and XendError and handled
those specifically. In general, we could probably an exception to
faultcode mapping and pick up the common exceptions (OSError etc.) but
for now I just special case it.
Regards,
Anthony Liguori
> Cheers,
>
> Ewan.
>
next prev parent reply other threads:[~2006-03-21 16:05 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-20 16:39 XML-RPC for Xend status Anthony Liguori
2006-03-21 15:47 ` Ewan Mellor
2006-03-21 16:05 ` Anthony Liguori [this message]
2006-03-21 23:40 ` [PATCH] Xend XML-RPC Support Anthony Liguori
2006-03-21 23:53 ` Ewan Mellor
2006-03-22 0:08 ` Anthony Liguori
2006-03-23 13:59 ` Ewan Mellor
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=44202437.5060800@us.ibm.com \
--to=aliguori@us.ibm.com \
--cc=ewan@xensource.com \
--cc=m+Ian.Pratt@cl.cam.ac.uk \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.