Home GnuPG

Make implicit audits by the Owners tool use modern code
2e9cc5e8e8a3Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Make implicit audits by the Owners tool use modern code

Summary:
Ref T10978. This updates audits triggered by Owners to use a modern transaction. Minor changes:

  • After D17264, we no longer need the "AUDIT_NOT_REQUIRED" fake-audits to record package membership. This no longer creates them.
  • This previously saved English-language, untranslatable text strings about audit details onto the audit relationship. I've removed them, per discussion in D17263.

The "Audit Reasons" here are potentially a little more useful than the Herald/Explicit-By-Owner ones were, since the rules are a little more complex, but I'd still like to see evidence that we need them.

In particular, the transaction record now says "Owners added auditors: ...", just like Differential, so the source of the auditors should be clear:

{F2549087}

T11118 (roughly "add several Owners audit modes", despite the title at time of writing) might impact this too. Basically, this is simple and maybe good enough; if it's not quite good enough we can refine it.

Test Plan: Ran bin/repository reparse --owners <commit> saw appropriate owners audits trigger.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10978

Differential Revision: https://secure.phabricator.com/D17266

Details

Provenance
epriestley <git@epriestley.com>Authored on Jan 30 2017, 7:14 PM
Parents
rPHAB5e7a0917372a: Write an explicit edge for commit membership in packages
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB2e9cc5e8e8a3: Make implicit audits by the Owners tool use modern code (authored by epriestley <git@epriestley.com>).Jan 31 2017, 12:23 AM