<br><br><div class="gmail_quote">On Mon, Jul 9, 2012 at 9:16 PM, Dan Prince <span dir="ltr"><<a href="mailto:dprince@redhat.com" target="_blank">dprince@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
<br>
----- Original Message -----<br>
> From: "Dave Walker" <<a href="mailto:DaveWalker@ubuntu.com">DaveWalker@ubuntu.com</a>><br>
> To: "Monty Taylor" <<a href="mailto:mordred@inaugust.com">mordred@inaugust.com</a>><br>
> Cc: <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a>, "John Garbutt" <<a href="mailto:John.Garbutt@citrix.com">John.Garbutt@citrix.com</a>><br>
> Sent: Monday, July 9, 2012 6:01:19 PM<br>
> Subject: Re: [Openstack] PEP8 checks<br>
><br>
</div><div><div class="h5">> On Mon, Jul 02, 2012 at 08:28:04AM -0400, Monty Taylor wrote:<br>
> ><br>
> ><br>
> > On 07/02/2012 06:46 AM, John Garbutt wrote:<br>
> > > Hi,<br>
> > ><br>
> > > I noticed I can now run the pep8 tests like this (taken from<br>
> > > Jenkins job):<br>
> > >   tox -v -epep8<br>
> > >   ...<br>
> > >   pep8: commands succeeded<br>
> > >   congratulations :)<br>
> > ><br>
> > > But the old way to run tests seems to fail:<br>
> > >   ./run-tests.sh -p<br>
> > >   ...<br>
> > >   File<br>
> > >   "/home/johngar/openstack/nova/.venv/local/lib/python2.7/site-packages/pep8.py",<br>
> > >   line 1220, in check_logical<br>
> > >       for result in self.run_check(check, argument_names):<br>
> > >   TypeError: 'NoneType' object is not iterable<br>
> > ><br>
> > > Is this expected?<br>
> > > Did I just miss an email about this change?<br>
> ><br>
> > I cannot reproduce this on my system. Can you run<br>
> > "bash -x run_tests.sh -p" and pastebin the output? Also,<br>
> > tools/with_venv.sh pep8 --version just to be sure.<br>
> ><br>
><br>
> Hi,<br>
><br>
> The issue is that as of a recent change to upstream pep8 [1], the<br>
> additional pep8 rules in tools/hacking.py need to be changed from<br>
> returns to yields.. :(<br>
<br>
</div></div>This brings up a good point. Why are we following the latest pep8 release so closely in Nova? The latest release is hardly a month old and we are already using it? We aren't necessarily doing the same for all the other OpenStack projects... (nor am I suggesting that we do). So why Nova?<br>

<br>
I'm not convinced the latest pep8 "features" really provide us enough benefit that we need to bump our pep8 baseline every month or two.  in fact they may be hurting us in terms of churn, extra work, back-portability of upstream patches, etc. Ultimately is tracking the latest pep8 really worth it?<br>
</blockquote><div><br></div><div>+1</div><div><br></div><div>Some of the "features" are requiring useless whitespace changes to working code just to get through the gating tests. In ceilometer we've pegged our pep8 tests to 1.1.</div>
<div><br></div><div>Doug</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im HOEnZb"><br>
<br>
><br>
> [1]<br>
> <a href="https://github.com/jcrocholl/pep8/commit/b9f72b16011aac981ce9e3a47fd0ffb1d3d2e085" target="_blank">https://github.com/jcrocholl/pep8/commit/b9f72b16011aac981ce9e3a47fd0ffb1d3d2e085</a><br>
><br>
> Kind Regards,<br>
><br>
> Dave Walker <<a href="mailto:Dave.Walker@canonical.com">Dave.Walker@canonical.com</a>><br>
> Engineering Manager,<br>
> Ubuntu Server Infrastructure<br>
><br>
</div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
> Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
><br>
<br>
_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
</div></div></blockquote></div><br>