It looks like these changes have gone into Quantum and Keystone. That leaves Glance, Nova, Cinder, and Oslo.<div><br></div><div>The gating tests for Cinder are failing because of the version conflicts between some of the projects now. I expect the same issue to come up for Nova, when that patch is approved. Is there a procedure for bypassing those tests or otherwise getting past the fact that one project can't merge the change until they all do?</div>
<div><br></div><div>Doug<br><br><div class="gmail_quote">On Mon, Jan 7, 2013 at 8:22 AM, Doug Hellmann <span dir="ltr"><<a href="mailto:doug.hellmann@dreamhost.com" target="_blank">doug.hellmann@dreamhost.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div class="im">On Mon, Jan 7, 2013 at 3:13 AM, Sascha Peilicke <span dir="ltr"><<a href="mailto:saschpe@suse.de" target="_blank">saschpe@suse.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Am 3. Januar 2013 19:34:46 schrieb Doug Hellmann <<a href="mailto:doug.hellmann@dreamhost.com" target="_blank">doug.hellmann@dreamhost.com</a>>:<div><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Sure; I'm not sure why my grep didn't find that use.<br>
</blockquote>
<br></div>
Hi Doug,<br>
<br>
I did most of those and I'll have a look at the other OpenStack components soon. We run all testsuites against openSUSE:Factory packages, which are always the latest and greatest. However, no issues with a newer WebOb popped up so far. So either the coverage isnt't sufficient or its working already ;-)</blockquote>
<div><br></div></div><div>Thanks, Sascha! Let me know if you run into any issues, please.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Doug</div></font></span><div><div class="h5"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Doug<br>
<br>
On Thu, Jan 3, 2013 at 12:30 PM, Brian Waldon <<a href="mailto:bcwaldon@gmail.com" target="_blank">bcwaldon@gmail.com</a>> wrote:<br>
<br>
> Webob is being used in Glance as well. There have been some patches to<br>
> prepare the code for webob 1.2.1, but we're still depending on 1.0.8. Would<br>
> you mind taking a look?<br>
><br>
><br>
> On Jan 3, 2013, at 8:32 AM, Doug Hellmann wrote:<br>
><br>
> The ceilometer team is using Pecan [1] for our next-generation API<br>
> framework. Pecan depends on a newer WebOb than is used by the other<br>
> OpenStack components, so I spent this morning reviewing the differences in<br>
> WebOb between 1.0.8 and 1.2.3 (the latest stable release) [2] and what<br>
> impact those changes might have on nova, cinder, keystone, and oslo (the 4<br>
> projects where I found WebOb being used -- please let me know if I missed<br>
> one).<br>
><br>
> Most of the changes appear to be bug fixes and changes that either have a<br>
> backwards-compatible alias, are deprecations that were reversed later, or<br>
> should not have an impact on us because we were not using the changed area<br>
> of the code directly. The only thing I found to be questionable is the<br>
> change to Request.path_info and Request.script_name. Those properties are<br>
> now unicode objects instead of str objects, as WebOb is trying to provide<br>
> better Python 3 support. I found a few uses of those variables in<br>
> comparisons and concatenation with str objects, but the casting should work<br>
> automatically as long as the path can be encoded in the default encoding,<br>
> which should be the case as the paths to our API use only ASCII characters.<br>
> All of the unit tests pass when I run them locally.<br>
><br>
> I have submitted patches against the 4 projects to update the version<br>
> number in tools/pip-requires, without any other code changes:<br>
><br>
> Oslo: <a href="https://review.openstack.org/#/c/18885/" target="_blank">https://review.openstack.org/#<u></u>/c/18885/</a><br>
> Nova: <a href="https://review.openstack.org/#/c/18886/" target="_blank">https://review.openstack.org/#<u></u>/c/18886/</a><br>
> Keystone: <a href="https://review.openstack.org/#/c/18887/" target="_blank">https://review.openstack.org/#<u></u>/c/18887/</a><br>
> Cinder: <a href="https://review.openstack.org/#/c/18888/" target="_blank">https://review.openstack.org/#<u></u>/c/18888/</a><br>
><br>
> If anyone else has done similar analysis in the past and come to a<br>
> different conclusion about whether it is safe to upgrade, please let me<br>
> know.<br>
><br>
> Doug<br>
><br>
> [1] Pecan: <a href="http://pecan.readthedocs.org/en/latest/" target="_blank">http://pecan.readthedocs.org/<u></u>en/latest/</a><br>
> [2] The full release notes for WebOb:<br>
> <a href="http://docs.webob.org/en/latest/news.html" target="_blank">http://docs.webob.org/en/<u></u>latest/news.html</a><br>
><br>
> ______________________________<u></u>_________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
><br>
><br>
><br>
> ______________________________<u></u>_________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
><br>
><br>
</blockquote></div></div>
--<br>
With kind regards,<br>
Sascha Peilicke<br>
SUSE Linux GmbH, Maxfeldstr. 5, D-90409 Nuernberg, Germany<br>
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer HRB 16746 (AG Nürnberg)<br>
<br>
<br>
</blockquote></div></div></div><br>
</blockquote></div><br></div>