Page MenuHomePhabricator

Improve audit behavior for "uninteresting" auditors
ClosedPublic

Authored by epriestley on May 17 2016, 7:01 PM.
Tags
None
Referenced Files
F14360276: D15940.diff
Fri, Dec 20, 10:08 AM
Unknown Object (File)
Sun, Dec 1, 11:12 AM
Unknown Object (File)
Thu, Nov 28, 12:10 AM
Unknown Object (File)
Wed, Nov 27, 6:48 PM
Unknown Object (File)
Nov 18 2024, 9:54 AM
Unknown Object (File)
Nov 12 2024, 4:27 PM
Unknown Object (File)
Nov 12 2024, 12:55 PM
Unknown Object (File)
Nov 5 2024, 5:31 PM
Subscribers
None

Details

Summary

Ref T10939. Fixes T10174. We can currently trigger "uninteresting" auditors in two ways:

  • Packages with auditing disabled ("NONE" audits).
  • Packages with auditing enabled, but they don't need an audit (e.g., author is a pacakge owner; "NOT REQUIRED" audits).

These audits aren't interesting (we only write them so we can list "commits in this package" from other UIs) but right now they take up the audit slot. In particular:

  • They show in the UI, but are generally useless/confusing nowadays. The actual table of contents does a better job of just showing "which packages do these paths belong to" now, and shows all packages for each path.
  • They block Herald from adding real auditors.

Change this:

  • Don't show uninteresting auditors.
  • Let Herald upgrade uninteresting auditors into real auditors.
Test Plan
  • Ran bin/repository reparse --owners <commit> --force, and --herald to trigger Owners and Herald rules.
  • With a package with auditing disabled, triggered a "None" audit and saw it no longer appear in the UI with the patch applied.
  • With a package with auditing disabled, added a Herald rule to trigger an audit. With the patch, saw it go through and upgrade the audit to "Audit Required".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Improve audit behavior for "uninteresting" auditors.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.May 17 2016, 8:15 PM
This revision was automatically updated to reflect the committed changes.