Re: [PATCH] admin: Configure 'list_select_related', 'get_queryset'

2018-09-19 Thread Stephen Finucane
On Wed, 2018-09-12 at 15:57 -0600, Stephen Finucane wrote: > This has a significant improvement for the patch and series views. > > /patchwork/patch > FROM: ~114 queries > TO: ~14 queries > > /patchwork/series > FROM: ~210 queries > TO: ~10 queries > > Signed-off-by: Stephen

Re: [PATCH RESEND 1/2] Rework tagging infrastructure

2018-09-19 Thread Stephen Finucane
On Wed, 2018-09-12 at 04:57 -0400, Veronika Kabatova wrote: > > - Original Message - > > From: "Stephen Finucane" > > To: vkaba...@redhat.com, patchwork@lists.ozlabs.org > > Sent: Tuesday, September 11, 2018 7:55:46 PM > > Subject: Re: [PATCH RESEND 1/2] Rework tagging infrastructure > >

[PATCH 1/2] parsearchive: Fix logging

2018-09-19 Thread Stephen Finucane
We should use a counter normally, avoid using the counter and emit logs when more detailed output is requested, and emit nothing when no output is requested. Signed-off-by: Stephen Finucane --- patchwork/management/commands/parsearchive.py | 22 --- patchwork/settings/base.py

[PATCH 2/2] parser: Handle IntegrityError for cover letters, comments

2018-09-19 Thread Stephen Finucane
This was already done for patches but cover letters and comments were not handled correctly, resulting in errors while parsing archives. While we're here, we slightly modify how these exceptions are handle. Rather than simply ignoring them, as we were doing, we raise a custom exception. This

Re: [PATCH] REST: Only list checks for the given patch

2018-09-19 Thread Veronika Kabatova
- Original Message - > From: "Stephen Finucane" > To: patchwork@lists.ozlabs.org > Cc: "Michael Ellerman" > Sent: Wednesday, September 12, 2018 5:46:56 PM > Subject: [PATCH] REST: Only list checks for the given patch > > This is either a regression or it never worked. In any case,

Re: [PATCH] Remove '__future__.absolute_import' imports

2018-09-19 Thread Stephen Finucane
On Mon, 2018-09-17 at 14:10 +0100, Stephen Finucane wrote: > These were added as part of the Python 3 support series but are not > required and can be safely removed. > > Signed-off-by: Stephen Finucane Caused no issues with unit tests or a running instance so applied. Stephen

Re: [PATCH] tox: Specify doctree directory

2018-09-19 Thread Stephen Finucane
On Tue, 2018-09-18 at 16:49 +0100, Stephen Finucane wrote: > Sphinx 1.8 has a change in where it places 'doctree' directories. Rather > than ignore this directory via gitignore, specify where this directory > should go. This will ensure future changes in Sphinx's behavior won't > affect us. > >

Re: [PATCH v3 2/2] docs: Note new requirement to include a SPDX line

2018-09-19 Thread Stephen Finucane
On Tue, 2018-09-18 at 12:11 -0400, Veronika Kabatova wrote: > > - Original Message - > > From: "Stephen Finucane" > > To: patchwork@lists.ozlabs.org > > Cc: "Stephen Finucane" , "Daniel Axtens" > > , "Veronika Kabatova" > > > > Sent: Tuesday, September 18, 2018 5:46:16 PM > > Subject: