models: Ensure UserProfile.user is configured

Django Admin seems to be doing something funky with how it's handling
the creation of a User's corresponding UserProfile instance when
modelled as an inline field. Re-setting the UserProfile.user attribute
seems to resolve the issue, so do just that.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #110

Add REST API validation using OpenAPI schema

Add validation using the rather excellent 'openapi_core' library. The
biggest issue we have to contend with is the fact that 'openapi_core'
expects us to be able to provide a templated URL string for each request
(e.g. '/api/patches/123/' would become '/api/patches/<id>/') and Django
doesn't provide a way to do this [*]. We work around this by
reverse-engineering some of the Django code to turn a URL to its
matching regex, which we can then easily convert into a template string.
It's kind of hacky and not at all portable but, crucially, it does work
and has highlighted some nice bugs in the API that have already merged.

Going forward, we can probably modify 'openapi_core' somewhat to remove
the need for the templated URL string. If and when this happens, most of
the funkier code here can happily go away.

[*] Django 2.0+ [1] does actually provide a way to do template
string-based URLs and in fact recommends them now, with regexes being
reserved for more advanced corner cases. However, we don't want to drop
support for the Django 1.11 yet as it is the most recent LTS release.

[1] https://docs.djangoproject.com/en/2.1/ref/urls/#django.urls.path

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Remove old Swagger 2.0 spec

This was incomplete and wrong in many places. It's since been replaced
by the OpenAPI 3.0.0 specs, so we have happily remove this now.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Store versioned OpenAPI schemas

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Make API document versioned

OpenAPI doesn't appear to support versioning natively, suggesting
instead that separate documents are kept. Rather than doing this
manually, let's use a templating tool - Jinja2, in this case - to
generate these document for us from a single master document.

Note that while we can now auto-generate these whenever we need them
(and we tend to avoid storing auto-generated assets in VCS), these
change so rarely that it's easier to just store them. This also means we
can reference the schemas themselves online. We do this in a following
change.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/events' resource

This is the final resource to document and also the most complicated, on
account of the polymorphism of the responses. However, with this done,
our first pass at an OpenAPI 3.0 schema is completed.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/patches/{patch_id}/checks' resource

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/patch/{id}/comments' resource

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/covers/{id}/comments' resource

Our first nested resource. Nothing too weird here though, save for the
raising of a HTTP 404 on a list resource (due to a missing patch).

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/series' resource

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/patches' resource

The big one (TM). Nothing too odd here though.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/covers' resource

Again, we're adding embedded serializers before the main resource but
that will come.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/bundles' resource

This one's a little unusual too, in that we provide the embedded
serializer for resources we haven't defined the end resource for. That's
necessary in general, due to recursive references in the API
(series-patch, patch-series etc.) so might as well embrace it early.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/projects' resource

This one's pretty straightforward.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/people' resource

This introduces our first use of embedded serializers, which are
separate models from the main ones. Other than that, this is pretty
standard.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Document the '/users' resource

This introduces our first use of parameters, both in the path and the
query. The latter are extracted out as they'll be used by later changes.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Start documenting API using OpenAPI

When the REST API was first added, we attempted to document it using
OpenAPI 2.0 (formerly Swagger). This was mostly never completed because
(a) it was really tedious and (b) no one was that bothered. However, as
we expand the range of clients for the REST API, having a well
documented API becomes more and more of an asset.

Start doing this by adding a brand new schema, this time using OpenAPI.
This will entirely replace the older schema and, as such, is namespaced
separately. We start by documenting '/' (i.e. the index) page and will
add additional resources as we go.

Signed-off-by: Stephen Finucane <stephen@that.guru>

REST: Handle JSON requests

This was raising an attribute error when switching tests to use JSON
bodies instead of form-data.

  AttributeError: 'dict' object has no attribute '_mutable'

The easy fix is to check if it's a dictionary and avoid the mutability
check if so.

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Use project.id in requests, not object

The object is not serializable.

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Use valid URLs

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Start setting Project.listemail

I'm not actually sure why this wasn't raising an error. Perhaps it's
because null validation for char fields happens in forms rather than at
the database level. In any case, this won't happen normally since we
only allow creation via the admin API so simply start setting this.

Signed-off-by: Stephen Finucane <stephen@that.guru>

REST: Drop embedded bundle serializer

This wasn't used anywhere and can therefore be removed.

Signed-off-by: Stephen Finucane <stephen@that.guru>

REST: Fix typo in embedded serializers

This was preventing the v1.1 fields from getting dropped as intended.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #237

REST: Handle unset series

This was introduced in the recent "convert series from N:M to 1:N"
series. We take the opportunity to make the 'create_patch' and
'create_cover' utility methods a little smarter, in that series will
automatically be created for the patch/cover letter unless told not to.
This requires some related changes to other modules.

Signed-off-by: Stephen Finucane <stephen@that.guru>

signals: Fix 'series-completed' event

I'm not sure how I ever intended this to work and there were no tests
verifying things. Fix it now and add tests to prevent it regressing.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Fixes: 76505e91 ("models: Convert Series-Patch relationship to 1:N")

tests: Add 'store_samples' decorator to 'test_event'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add tests for '/events' resource

This highlights an issue that will be resolved in a future change.

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_check'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_comment'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_series'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_cover'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_patch'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_user'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_people'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_project'

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator to 'test_bundle'

Add the decorator to the 'test_bundle' test class. This involves
splitting up the test cases so that each test case we care about makes
only a single request. We also add a missing test to ensure private
bundles cannot be shown by anyone but the owner.

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Add 'store_samples' decorator

We want to start including sample API requests and responses in our
documentation.  Given that these may get out of date over time, we
should really generate these things dynamically. Create a decorator that
will allow us to do just that.

Signed-off-by: Stephen Finucane <stephen@that.guru>

REST: Ensure submission exists for comment listing

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #225

REST: Ensure patch exists for check creation

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #226

REST: Show 'web_url' in embedded series responses

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #224
Fixes: 9c179bf4c ("REST: Add 'web_url' link to API responses")

REST: Add additional documentation

As noted in the Django REST Framework docs [1], views that support
multiple methods can and should split their documentation using
'method:' style delimiters. Do just this.

[1] https://www.django-rest-framework.org/topics/documenting-your-api/#documenting-your-views

Signed-off-by: Stephen Finucane <stephen@that.guru>

Don't passthrough 'Content-Type: multipart/signed' header

We don't GPG signatures, therefore this header is incorrect. Stop
passing it through.

Test for the other dropped header are also included.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Acked-by: Veronika Kabatova <vkabatov@redhat.com>
Closes: #221

Update psycopg2-binary from 2.7.5 to 2.7.6

Update psycopg2-binary from 2.7.5 to 2.7.6

Update django from 2.1.2 to 2.1.3

parser: Use 'objects.create' instead of 'save'

As noted in the Django documentation [1], this lets us do things in one
step.

[1] https://docs.djangoproject.com/en/dev/topics/db/queries/#creating-objects

Signed-off-by: Stephen Finucane <stephen@that.guru>
Acked-by: Daniel Axtens <dja@axtens.net>

README: Note latest version of requirements

These were missed previously

Signed-off-by: Stephen Finucane <stephen@that.guru>
Fixes: 8eb3719a ("Add support for django-filter 2.0")
Fixes: fab5571e ("Add support for Django REST Framework 3.9")

docs: Fix design doc

These items weren't really headers and shouldn't have been rendered as
such.

Signed-off-by: Stephen Finucane <stephen@that.guru>

Update python-dateutil from 2.7.4 to 2.7.5

travis: Use consistent quote style

It's the little things in life.

Signed-off-by: Stephen Finucane <stephen@that.guru>

travis: Stop testing PostgreSQL 11

Travis seems to be doing something really weird with PG11 since it was
released a few weeks ago. This is currently breaking our CI and can't
continue. It doesn't seem possible to mark this as an expected failure
so simply remove it. We can re-add this once Travis gains proper support
for this version via their addons.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Suggested-by: Daniel Black <daniel@linux.ibm.com>
Acked-by: Daniel Axtens <dja@axtens.net>

views: no longer import Series in __init__.py

Fix a pep8 error due to an unused import.

Fixes: 780b1a2d07ad ("views: Use select_related, not prefetch_related")
Signed-off-by: Daniel Axtens <dja@axtens.net>

views: Use select_related, not prefetch_related

Now that there's a 1:N rather than N:M relationship between series and
patches, we should be using select_related rather than prefetch_related.

Tested-by: Daniel Axtens <dja@axtens.net>
Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #223
Fixes: 76505e91 ("models: Convert Series-Patch relationship to 1:N")
Signed-off-by: Daniel Axtens <dja@axtens.net>

Update python-dateutil from 2.7.3 to 2.7.4

trivial: Use implicit string concatenation

While pycodestyle's W504 ("line break after binary operator") error was
recently disabled, it did highlight a number of areas where explicit
string concatenation was being used despite it not be necessary. Fix
these while we're aware of them.

[1] http://pycodestyle.pycqa.org/en/latest/intro.html#error-codes

Signed-off-by: Stephen Finucane <stephen@that.guru>

tox: Disable W504 ("line break after binary operator")

This was introduced in a recent version of 'pycodestyle'. The
documentation notes [1] that it is mutually exclusive with W503, which
we do enforce, suggesting that we disable one or the other. Avoid the
churn and stick to the older rule, which I personally prefer.

[1] http://pycodestyle.pycqa.org/en/latest/intro.html#error-codes

Signed-off-by: Stephen Finucane <stephen@that.guru>

README: Use pyup.io badge instead of requires.io

requires.io has an open issue whereby it doesn't consider Python version
in assessing the latest version of a package available. pyup.io also has
the same issue but we've already worked around this by ignoring the
offending lines. Seeing as we're already using that elsewhere, double
down on it.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Update links for django-rest-framework 3.7, 3.8

Don't you hate it when links break? These release notes haven't been
included in a release yet so we can just go ahead and update things.

Signed-off-by: Stephen Finucane <stephen@that.guru>

Add support for Django REST Framework 3.9

As with 3.7 and 3.8, there are no breaking changes we need to be
concerned with here.

Signed-off-by: Stephen Finucane <stephen@that.guru>

tests: Remove 'create_series_patch'

The 'SeriesPatch' object was recently removed, but the
'create_series_patch' was retained in order to minimize the changes
necessary. This can now be removed and the logic moved to the
'create_patch' and 'create_cover' functions instead.

Signed-off-by: Stephen Finucane <stephen@that.guru>

models: Convert Series-Patch relationship to 1:N

Late in the development of the series feature, it was decided that there
were advantages to allowing an N:M relationship between series and
patches. This would allow us to do things like create complete series
where a sole vN patch was sent to a list rather than the full series.
After some time using series in the wild, it's apparent that such
features are very difficult to implement correctly and will likely never
be implemented. As such, it's time to start cleaning up the mess, paving
the way for things like an improved tagging feature.

There are some significant changes to the model required:

- models.py, migrations/0027, migrations/0028, migrations/0029

  The migrations make the following changes:

  1. - Add 'Patch.series_alt' and 'Patch.number' fields.
  2. - Populate the 'Patch.series_alt' and 'Patch.number' fields from
       their 'SeriesPatch' equivalents.
  3. - Remove the 'SeriesPatch' model.
     - Rename 'Patch.series_alt' to 'Patch.series'.
     - Change 'Series.cover_letter' to a 'OneToOneField' since a cover
       letter can no longer be assigned to multiple series.

  Note that the migrations have to be split into multiple parts as the
  combined migration raises an OperationalError as below.

    (1072, "Key column 'series_alt_id' doesn't exist in table")

  This is due to Django's penchant for creating indexes for newly
  created fields, as noted here: https://stackoverflow.com/q/35158530/

Aside from the model changes, there are numerous other changes required:

- admin.py

  Reflect model changes for the 'PatchInline' inline used by
  'SeriesAdmin'

- api/cover.py, api/patch.py

  Update the 'series' field for the cover letter and patch resources to
  reflect the model changes. A 'to_representation' function is added in
  both cases to post-process this field and make it look like a list
  again. This is necessary to avoid breaking clients.

- parser.py

  Update to reflect the replacement of 'SeriesPatch' with 'Patch'.

- signals.py

  Update to filter on changes to 'Patch' instead of 'SeriesPatch'. This
  requires some reworking due to how we set these fields now, as we can
  no longer receive on 'post_save' signals for 'SeriesPatch' and must
  instead watch for 'pre_save' on 'Patch', which is what we do for
  delegate and state changes on same.

- templates/patchwork/*.html

  Remove logic that handled multiple series in favour of the (simpler)
  single series logic.

- tests/*

  Modify the 'create_series_patch' helper to reflect the removal of the
  'SeriesPatch' model. This entire helper will be removed in a future
  change. Improve some tests to cover edge cases that were highlighted
  during development

Unfortunately, all of the above changes must go in at the same time,
otherwise we end up with either (a) broken views, API etc. or (b) split
brain because we need to keep the new single-series fields alongside the
older multi-series fields and models while we rework the views. It's
unfortunate but there's not much to be done here.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Tested-by: Daniel Axtens <dja@axtens.net>

tests: Hardcode expected values

Compare against known values to ensure bugs introduced in the function
are caught.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Reviewed-by: Daniel Axtens <dja@axtens.net>

tests: Add more tests for series-ified mbox views

Cover some testing gaps identified during the migration from a M:N to a
1:N series-patch relationship, namely:

- Downloading a patch's mbox with dependencies using a numerical series
  ID
- Downloading a series' mbox

Signed-off-by: Stephen Finucane <stephen@that.guru>
Reviewed-by: Daniel Axtens <dja@axtens.net>

tox: Fix issues with 'bashate' target

We don't care about too long lines for bash so the error is disabled.

The version marker is removed as it's unnecessary.

Signed-off-by: Stephen Finucane <stephen@that.guru>

REST: Allow unsetting of delegate

While we recently fixed setting of this field via the API, we didn't
resolve unsetting. Fix this now.

Signed-off-by: Stephen Finucane <stephen@that.guru>

REST: Allow setting of values using embedded serializers

Unfortunately, the use of embedded serializers for some fields breaks
the ability to update these fields, either via the HTML interface (where
the widget is totally busted) or via a client like 'git-pw'. What we
actually want is to be able to update these fields like normal primary
key but show them using the embedded serializer. We do just this by
using a modified variant of the PrimaryKeyRelatedField and using the
serializers simply for displaying.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #216

REST: Validate patch delegate

At present, only users who are maintainers of projects can be delegated
a project. Validate this. This is currently broken due to #216 but that
will be fixed in a future change.

Signed-off-by: Stephen Finucane <stephen@that.guru>

REST: Don't allow settings of some project fields

These should only be configurable by superusers as invalid configuration
can break things.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #217

docs: Add release notes for recent bug fixes

We need to start adding these as part of the bug fix itself.

Signed-off-by: Stephen Finucane <stephen@that.guru>

templates: Don't show forced filters

No point in showing a filter that you can't change.

Signed-off-by: Stephen Finucane <stephen@that.guru>

filters: Rework code

Make extensive use of properties, remove dead code, add documentation
and generally clean this whole thing up.

Signed-off-by: Stephen Finucane <stephen@that.guru>

filters: Pre-populate delegate, submitter filters

This appears to have got lost in the transition to 'selectize.js'. In
brief, this will ensure that a previously selected delegate or submitter
is still enabled post-filtering. For more information, see [1].

[1] https://stackoverflow.com/a/45124779

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #78

filters: Return dictionaries

This makes it a little easier to figure out what filters are active and
will be used in a future patch.

Signed-off-by: Stephen Finucane <stephen@that.guru>

settings: Do configure logging for parsearchive (but differently)

In commit aa266a28, we removed the logging configuration for the
'parsearchive' tool because it was using the same configuration as
'parsemail' and could send an email to administrators for ERROR logs.
While we shouldn't be doing that, we should be configuring _some_ kind
of logging.

Signed-off-by: Stephen Finucane <stephen@that.guru>

requirements: Add sqlparse to test requirements

This was previously installed as part of 'django-debug-toolbar' [1] but
as this dependency is no longer installed for tests, the dependency is
now missing. Fix this by manually specifying it.

[1] https://github.com/jazzband/django-debug-toolbar/blob/1.10.1/setup.py#L20-L23

Signed-off-by: Stephen Finucane <stephen@that.guru>
Fixes: a7454712 ("Remove django-debug-toolbar from test requirements")

docs: Add documentation for django-dbbackup

This replaces the older manual backup steps. The section on
'django-debug-toolbar' is updated to match the new 'django-dbbackup'
section.

Signed-off-by: Stephen Finucane <stephen@that.guru>

doc: Cleanup development installation guide

Use literals where possible, don't leave a space between a term and
definition, and note that Python 3.4, not 3.3, is now the minimum
supported version of Python 3.

Signed-off-by: Stephen Finucane <stephen@that.guru>

Remove django-debug-toolbar from test requirements

This should speed up tests slightly and remove an unnecessary
dependency.

Signed-off-by: Stephen Finucane <stephen@that.guru>

Add support for 'django-dbbackup'

'parsemail' and 'parsearchive' are slow. When messing with models and
migrations, it can be very useful to backup the database in its current
state and restore it if/when you mess up. Currently, we've documented
how to do this via some commands run in the shell of the container, but
we can do things easier via an application designed for this purpose:
'django-dbshell'.

Signed-off-by: Stephen Finucane <stephen@that.guru>

docs: Add 'unreleased' to release notes index page

It's already on the main index page.

Signed-off-by: Stephen Finucane <stephen@that.guru>

Update sphinx_rtd_theme from 0.3.0 to 0.4.2

views: Populate bundles for 'todo' view

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #213

docs: Call out Python 3 dependency for Django 2.0

Just for those that don't read the Django release notes.

Signed-off-by: Stephen Finucane <stephen@that.guru>

Add support for Django 2.1

For once, this just works. Yay!

Signed-off-by: Stephen Finucane <stephen@that.guru>

Add support for django-filter 2.0

This is necessary for Django 2.1 support. We retain support for
django-filter 1.0 and 1.1 as 2.0 is Python 3-only. Thankfully there is
essentially zero cost in doing so for now.

Signed-off-by: Stephen Finucane <stephen@that.guru>

requirements: Bump version of django-debug-toolbar to 1.10.1

Use the latest and greatest.

Signed-off-by: Stephen Finucane <stephen@that.guru>

templates: Avoid recursive call

We had registered an event handler on a checkbox in table header which
would call a function, 'checkboxes', on all checkboxes within that
table. This function, in turn, causes does its work and then triggers
event handlers for all modified checkboxes which include the original
table header checkbox. This resulted in the original event calling
itself recursively.

Resolve this by only modifying the checkboxes in the table body.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Fixes: 44fe7bae ("js: Allow shift-select of checkboxes")

templates: Remove trailing newlines

These got added accidentally during merge.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Fixes: 45fa5c5c ("templates: Move additional email subjects to templates")

views: Add error handling for user registration

This was already present for registration confirmation but missing for
initial registration. Resolve this.

Signed-off-by: Stephen Finucane <stephen@that.guru>

templates: Remove 'email_sent' attribute

Further normalization to ensure all related code paths use similar code.

Signed-off-by: Stephen Finucane <stephen@that.guru>

templates: Move additional email subjects to templates

Use a uniform pattern for this stuff.

Signed-off-by: Stephen Finucane <stephen@that.guru>

templates: Keep only whole templates in the top-level

Again, this should make this a little more understandable as it ensures
a rough mapping exists between views and template names.

Signed-off-by: Stephen Finucane <stephen@that.guru>

templates: Rename additional templates

Make ALL the things consistent.

Signed-off-by: Stephen Finucane <stephen@that.guru>

templates: Move mails to separate directory

This makes things a little easier to parse. A couple of templates are
renamed and the 'register.mail' template, which appears to be unused
since commit f1e089f7, is removed.

Signed-off-by: Stephen Finucane <stephen@that.guru>

settings: Don't configure logging for parsearchive

A recent change added additional ERROR level logs to the 'parsearchive'
tool. This highlighted an issue, whereby we had configured all modules
in 'patchwork.management.command' to email administrators on ERROR logs.
We clearly shouldn't be doing this for the 'parsearchive' command or for
anything other than 'parsemail', so fix this.

Along the way, we remove a now-unnecessary 'extra' argument to one of
the logging calls in 'parsearchive' and resolve a pep8 issue.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Fixes: 133091da ("parsearchive: Fix logging")
Cc: Daniel Axtens <dja@axtens.net>

requirements: Start using fixed versions

Given that 'tox' doesn't actually read any of these, there's no reason
to use ranges of requirements. Instead, use the latest and greatest for
live instances and rely on tox to validate behavior with older versions.

The selenium dependency, which is no longer required since commit
bab2895f, is removed. The psycopg2 dependency is updated to use
psycopg2-binary, as this avoids the need for the libpg library and
removes a deprecation warning.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Signed-off-by: Daniel Axtens <dja@axtens.net>

parser: Handle IntegrityError for cover letters, comments

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 allows us to specifically identify these types of
exceptions, print a log and still skip them (which we want, as seen in
commit d2eb1f6d2).

While we're here, we change from separate create-save calls to a
combined create-save call for both created CoverLetter and Comment
objects. We were already doing this for patches.

Signed-off-by: Stephen Finucane <stephen@that.guru>

parsearchive: Fix logging

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.

In addition, the default logging level for the parser module is set to
'WARNING' to make it less chatty.

Signed-off-by: Stephen Finucane <stephen@that.guru>

admin: Configure 'list_select_related', 'get_queryset'

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 Finucane <stephen@that.guru>

REST: Only list checks for the given patch

This is either a regression or it never worked. In any case, fix it and
add a test to ensure it doesn't happen again.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Reviewed-by: Veronika Kabatova <vkabatov@redhat.com>
Closes: #203