<div dir="ltr">I've tried to do it some time ago - <a href="https://review.openstack.org/#/c/266425/">https://review.openstack.org/#/c/266425/</a><div>But I tried to remove more than s3_image<br><div>But there was no consensus...</div><div><br></div><div>Regards,</div><div>Andrey.</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 22, 2016 at 3:01 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"><span class="">On 09/22/2016 02:53 AM, Clint Byrum wrote:<br>
> Excerpts from Matt Riedemann's message of 2016-09-21 18:43:06 -0500:<br>
>> The s3 image configuration options were deprecated for removal in newton<br>
>> [1].<br>
>><br>
>> Clint has a patch up to remove the boto dependency from nova [2] which<br>
>> is only used in the nova.image.s3 module.<br>
>><br>
>> Rather than remove the boto dependency, I think we should just remove<br>
>> this code. Is there any reason to not do that now that Ocata is open and<br>
>> it's been over the requisite 3 month window of deprecation for CD shops?<br>
>><br>
>> [1] <a href="https://review.openstack.org/#/c/314697/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/314697/</a><br>
>> [2] <a href="https://review.openstack.org/#/c/374433/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/374433/</a><br>
>><br>
><br>
> I went ahead and updated this review to remove the code. This is<br>
> definitely cleaner, and users of it can certainly resurrect it out of<br>
> tree if they are in need of it.<br>
<br>
</span>+1 on the drop, but we should make sure to pull the related code in the<br>
process. I left notes in the review for other things that can go.<br>
<br>
The one open question to me, we've got a dedicated db table for<br>
s3_images. Which, I have no idea when it was last used for anything.<br>
What kind of warning do we need to do on a full table drop? Do we need<br>
to handle the independently?<br>
<span class="HOEnZb"><font color="#888888"><br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
<a href="http://dague.net" rel="noreferrer" target="_blank">http://dague.net</a><br>
</font></span><div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Kind regards,<div>Andrey Pavlov.</div></div></div>
</div>