<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 24, 2015 at 12:36 AM, Victor Stinner <span dir="ltr"><<a href="mailto:vstinner@redhat.com" target="_blank">vstinner@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I wrote my spec to Port Nova to Python 3:<br>
<a href="https://review.openstack.org/#/c/176868/" target="_blank">https://review.openstack.org/#/c/176868/</a><br>
<span class=""><br>
>> I squashed all my commits into a single commit of my draft port and I pushed it at:<br>
>> <a href="https://github.com/haypo/nova/commit/bad54bc2b278c7c7cb7fa6cc73d03c70138bd89d" target="_blank">https://github.com/haypo/nova/commit/bad54bc2b278c7c7cb7fa6cc73d03c70138bd89d</a><br>
><br>
> I like how the sha1 starts with 'bad'<br>
<br>
</span>Ah ah, I didn't notice that. I would prefer "python" prefix, but it's not possible.<br>
<span class=""><br>
> Overall that is a pretty small patch.<br>
<br>
</span>Cool.<br>
<span class=""><br>
> My general concern, is having to manually review new code for python3 compliance.<br>
<br>
</span>Do you prefer a single very large patch (as the one I posted above) or multiple small patches fixing similar issues?</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> If this will take more then a week or two to make a big project python3 compat (during a virtual sprint), it would be good to have some tooling in place to make sure we don't add a lot more burden on reviewers to make sure new patches are python3 compatible by hand.<br>
<br>
</span>I tried to write a detailed plan in my spec. Until "tox -e py34" pass, I would prefer to not touch nova gates nor any add Python 3 checks.<br></blockquote><div><br></div><div>Great, I'll follow up on that spec.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
Victor<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
</div></div></blockquote></div><br></div></div>