"xmlrpc: Don't load all Patches into memory" breaking Django 1.6 and 1.7

Stephen Finucane stephenfinucane at hotmail.com
Fri Sep 2 06:34:48 AEST 2016


On 25 Aug 11:19, Daniel Axtens wrote:
> Hi!
> 
> I've pulled the latest master, and I'm now getting the following failure
> on Django 1.6 and 1.7.
> 
> FAIL: test_list_negative_max_count (patchwork.tests.test_xmlrpc.XMLRPCTest)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File "/home/patchwork/patchwork/patchwork/tests/test_xmlrpc.py", line 65, in test_list_negative_max_count
>     self.assertEqual(patches[0]['id'], patch_objs[-1].id)
> AssertionError: 1 != 5L
> 
> 
> It was introduced by 5b70ecf7d318 - reverting it makes the failure go
> away. I haven't investigated in any more detail yet.
> 

I don't know why I didn't pick it up earlier, tbh, but this actually
looks like an issue with these Django versions (or, more likely, we're
doing something dumb). I'm replaced these with a slightly slower, but
just as effective solution here:

    https://patchwork.ozlabs.org/patch/664998/

Would appreciate the review.

Stephen


More information about the Patchwork mailing list