<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 8 August 2016 at 17:50, Jay Pipes <span dir="ltr"><<a href="mailto:jaypipes@gmail.com" target="_blank">jaypipes@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Tempest devs,<br>
<br>
Let me please draw your attention to a LP bug that may not seem particularly high priority, but I believe could be resolved easily with a patch already proposed.<br>
<br>
LP bug 1536251 [1] accurately states that Tempest is actively verifying that an OpenStack API call violates RFC 7230.<br>
<br>
When a 204 No Content is received, the Content-Length header MUST NOT be present.<br>
<br>
However, Swift returns a Content-Length header and also an HTTP response code of 204 for a request to list containers of a new user (that has no containers).<br>
<br>
Tempest has been validating this behaviour even though it is a violation of RFC 7230:<br>
<br>
<a href="https://github.com/openstack/tempest/blob/master/tempest/api/object_storage/test_account_services.py#L81" rel="noreferrer" target="_blank">https://github.com/openstack/t<wbr>empest/blob/master/tempest/api<wbr>/object_storage/test_account_<wbr>services.py#L81</a><br>
<br>
RadosGW provides a proxy API that attempts to match the OpenStack Object Storage API, backed by Ceph object storage. In order for RadosGW to pass RefStack's burden of compatibility, it must pass the Tempest OpenStack Object Storage API tests. It currently cannot do so because RadosGW does not violate RFC 7230.<br>
<br>
The RadosGW developer community does not wish to argue about whether or not to make Swift's API comply with RFC 7230. At the same time, they do not want to add a configuration option to RadosGW to force the proxy service to violate RFC 7230 just to satisfy the RefStack/Tempest API tests.<br>
<br>
Therefore, Radoslaw (cc'd) has proposed a patch to Tempest that would allow RadosGW's proxy API to meet the RefStack compatibility tests while also not violating RFC 7230 and not requiring any change of Swift:<br>
<br>
<a href="https://review.openstack.org/#/c/272062" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/272062</a><br>
<br>
I ask Tempest devs to re-review the above patch and consider merging it for the sake of collaboration between the OpenStack and Ceph developer communities.<br>
<br>
Thanks very much!<br>
-jay<br>
<br>
[1] <a href="https://bugs.launchpad.net/tempest/+bug/1536251" rel="noreferrer" target="_blank">https://bugs.launchpad.net/tem<wbr>pest/+bug/1536251</a></blockquote><div><br></div><div><br></div><div>These sorts of issues aren't just theoretical and following policy for the sake of it..  Glance had 3 x areas where 200 responses that also included a Location header (against RFC-2616 §14.30) which totally broke glance when deployed behind apache+fcgid+flup (the presence of Location, that stack rewrote it to a 302).</div><div><br></div><div>Fun bug btw:</div><div><a href="https://bugs.launchpad.net/glance/+bug/1299095">https://bugs.launchpad.net/glance/+bug/1299095</a><br></div><div><br></div><div>--</div><div>Kind Regards,</div><div>Dave Walker</div></div></div></div>