<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 12, 2014 at 6:40 AM, 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 03/12/2014 09:01 AM, Flavio Percoco wrote:<br>
> On 11/03/14 16:25 -0700, Clint Byrum wrote:<br>
>> Hi. I asked in #openstack-glance a few times today but got no response,<br>
>> so sorry for the list spam.<br>
>><br>
>> <a href="https://review.openstack.org/#/c/79710/" target="_blank">https://review.openstack.org/#/c/79710/</a><br>
>><br>
>> This change introduces a backward incompatible change to defaults with<br>
>> Havana. If a user has chosen to configure swift, but did not add swift<br>
>> to the known_stores, then when that user upgrades Glance, Glance will<br>
>> fail to start because their swift configuration will be invalid.<br>
>><br>
>> This broke TripleO btw, which tries hard to use default configurations.<br>
>><br>
>> Also I am not really sure why this approach was taken. If a user has<br>
>> explicitly put swift configuration options in their config file, why<br>
>> not just load swift store? Oslo.config will help here in that you can<br>
>> just add all of the config options but not actually expect them to be<br>
>> set. It seems entirely backwards to just fail in this case.<br>
>><br>
><br>
> Here's an attempt to fix this issues without reverting the patch.<br>
> Feedback appreciated.<br>
><br>
> <a href="https://review.openstack.org/#/c/79935/" target="_blank">https://review.openstack.org/#/c/79935/</a><br>
<br>
</div></div>ACK. Looks pretty good. You might want to consider using one of the oslo<br>
deprecation functions to make it consistent on the deprecation side.<br>
<div class="HOEnZb"><div class="h5"><br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
Samsung Research America<br>
<a href="mailto:sean@dague.net">sean@dague.net</a> / <a href="mailto:sean.dague@samsung.com">sean.dague@samsung.com</a><br>
<a href="http://dague.net" target="_blank">http://dague.net</a><br>
<br>
</div></div><br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote><div><br></div><div>Sorry, I suppose I should have interrogated the backwards-incompatibility assumptions people were making about this change a bit more.</div><div><br></div><div>It looks like the latest patch is a great deprecation mechanism. Thanks for working out a solution, Flavio et al.</div>
</div><br></div></div>