<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Nov 1, 2013 at 8:55 PM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 11/01/2013 04:58 AM, Steven Hardy wrote:<br></div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb">
<div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
going into testing is finding some real bugs! :) )<br>
<br>
I notice there's a review associated with that bug, but I can't view it -<br>
if it's draft can you please add me to the reviewers list?<br>
<br>
Looking forward to seeing the patch, as you said the unit test examples<br>
should help me complete my Tempest patch.<br>
<br>
Also note I raised:<br>
<br>
<a href="https://bugs.launchpad.net/keystone/+bug/1246831" target="_blank">https://bugs.launchpad.net/<u></u>keystone/+bug/1246831</a><br>
<br>
Which seems somewhat related to this (we get a 500 with the XML encoded<br>
expires_at=None, which results in a zero-length string, so the trust<br>
controller treats it as a valid timestamp instead of ignoring it)<br>
<br>
I was planning to send a patch for the latter, but seems like it may<br>
overlap with your XML fixes, so I'll hold off for now.<br>
</blockquote>
<br></div></div>
My experience with the nova API and adding XML testing is that any service's XML API is broken by default (because the underlying logic is pretty JSON skewed, and the python clients talk JSON). So this isn't very surprising. Thanks again for diving into it!<br>

<br>
Honestly, one of these days we should have another serious conversation about dropping XML entirely again (across all projects). A single data payload that works is way better than additional payloads that don't.<span class="HOEnZb"><font color="#888888"><br>

<br></font></span></blockquote><div><br>+1 !<br></div></div></div></div>