patchwork:renovate/configure

Last commit made on 2020-04-18
Get this branch:
git clone -b renovate/configure https://git.launchpad.net/patchwork

Branch merges

Branch information

Name:
renovate/configure
Repository:
lp:patchwork

Recent commits

0acb60f... by Stephen Finucane <email address hidden>

renovate: Configure requirement file locations

Signed-off-by: Stephen Finucane <email address hidden>

1f285cd... by Renovate Bot <email address hidden>

chore(deps): add renovate.json

d89c20b... by Stephen Finucane <email address hidden>

travis: Resolve warnings, info messages from Travis

The following were reported by Travis' build config validation:

- root: deprecated key 'sudo' (The key `sudo` has no effect anymore.)
- env: key 'matrix' is an alias for 'jobs', using 'jobs'
- root: key 'matrix' is an alias for 'jobs', using 'jobs'
- root: missing 'os', using the default 'linux'

Resolve all of the above.

Signed-off-by: Stephen Finucane <email address hidden>

55fb26b... by Jeremy Kerr

parser: don't trigger database IntegrityErrors on duplicate coverletters

As we've done for the Patch and Comment models, this change prevents
database errors from duplicate CoverLetters.

Signed-off-by: Jeremy Kerr <email address hidden>
Signed-off-by: Stephen Finucane <email address hidden>
[stephenfin: Add release note]

55aa9cd... by Jeremy Kerr

parser: don't trigger database IntegrityErrors on duplicate comments

As we've done for the Patch model, this change prevents database errors
from duplicate Comments.

Signed-off-by: Jeremy Kerr <email address hidden>
Reviewed-by: Stephen Finucane <email address hidden>

947c6aa... by Jeremy Kerr

parser: prevent IntegrityErrors

Currently, the parser relies on causing (and catching) IntegrityErrors
on patch insert to catch duplicate (msgid,project) mails.

This change performs an atomic select -> insert instead.

Signed-off-by: Jeremy Kerr <email address hidden>
Signed-off-by: Stephen Finucane <email address hidden>
[stephenfin: Remove 'expectedFailure' marker again]

a60e75e... by Jeremy Kerr

tests: ensure we don't see database errors during duplicate insert

Currently, the parser causes IntegrityErrors while inserting duplicate
patches; these tend to pollute database logs.

This change adds a check, which currently fails, to ensure we do not
cause errors during a duplicate patch parse.

Signed-off-by: Jeremy Kerr <email address hidden>
Signed-off-by: Stephen Finucane <email address hidden>
[stephenfin: Add 'expectedFailure' marker to keep all tests green]

d4b5fbd... by Jeremy Kerr

tests: Add duplicate mail test

Test that we get the correct DuplicateMailError from parsing the same
mail twice.

Signed-off-by: Jeremy Kerr <email address hidden>
Reviewed-by: Stephen Finucane <email address hidden>

a971756... by Andrew Donnellan <email address hidden>

parser: Don't crash when From: is list email but has weird mangle format

get_original_sender() tries to demangle DMARC-mangled From headers, in
the case where the email's From address is the list address. It knows how
to handle Google Groups and Mailman style mangling, where the original
submitter's name will be turned into e.g. "Andrew Donnellan via
linuxppc-dev".

If an email has the From header set to the list address but has a name that
doesn't include " via ", we'll throw an exception because stripped_name
hasn't been set. Sometimes this is because the list name is seemingly
empty, resulting in a mangled name like "Andrew Donnellan via"
without the space after "via" that we detect. Handle this as well as we can
instead, and add a test.

Fixes: 8279a84238c10 ("parser: Unmangle From: headers that have been mangled for DMARC purposes")
Reported-by: Jeremy Kerr <email address hidden>
Signed-off-by: Andrew Donnellan <email address hidden>
Reviewed-by: Stephen Finucane <email address hidden>

162b4ad... by Stephen Finucane <email address hidden>

docs: Update sphinxcontrib-openapi

No changes necessary, thankfully, though there is a feature gap here
that we will need 0.7.0 to close [1] :(

[1] https://github.com/sphinx-contrib/openapi/pull/87

Signed-off-by: Stephen Finucane <email address hidden>