[PATCH v3 0/3] Store the 'actor' responsible for events

Johan Herland johan at herland.net
Thu Oct 17 09:44:39 AEDT 2019


The V4L project (https://patchwork.linuxtv.org) uses patch states and
delegates extensively to track progress. We want an audit log to keep
track of the changes made to these patch fields. The Event model already
records this information, but leaves out one crucial detail: which
maintainer/user actually updated the patch state/delegate. The need for
this enhancement is also documented in Issue #73.

This patch series adds an 'actor' field to the Event model, and - for
applicable events - stores the user responsible for that event (i.e. the
current/active user, if any) into this field.

This applies to the following events:
 - patch-created
 - patch-completed
 - patch-state-changed
 - patch-delegated
 - series-completed
 - check-created

For the other events (cover-created and series-created), I could not
easily determine if there is a responsible user at all, as these events
are typically generated in response to incoming emails. In these cases
(and any other scenario where we cannot find the active/current user)
the Event.actor field is left as null/None.

Finally, the new Event.actor field is exposed in the events view of the
REST API (as of API version 1.2).

Changes since v2:
 - Update docs/usage/overview.rst
 - Acked-by: Daniel


Have fun!
...Johan


Johan Herland (3):
  models.Event: Add the user responsible for the event
  Include the responsible actor in applicable events
  /api/events: Add 'actor' field to generated JSON

 docs/api/schemas/latest/patchwork.yaml   |  6 ++++++
 docs/api/schemas/patchwork.j2            |  8 ++++++++
 docs/api/schemas/v1.2/patchwork.yaml     |  6 ++++++
 docs/usage/overview.rst                  |  3 +++
 patchwork/api/event.py                   | 10 +++++++---
 patchwork/migrations/0037_event_actor.py | 21 +++++++++++++++++++++
 patchwork/models.py                      | 10 +++++++++-
 patchwork/signals.py                     | 10 ++++++++--
 patchwork/tests/api/test_event.py        | 24 ++++++++++++++++++++++++
 patchwork/tests/test_events.py           |  7 +++++++
 10 files changed, 99 insertions(+), 6 deletions(-)
 create mode 100644 patchwork/migrations/0037_event_actor.py

-- 
2.19.2



More information about the Patchwork mailing list