Page MenuHomePhabricator

When an Owners package accepts a revision, count that as an "involved owner" for the purposes of audit
ClosedPublic

Authored by epriestley on Feb 7 2019, 9:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 5, 2:15 AM
Unknown Object (File)
Wed, Apr 3, 2:51 PM
Unknown Object (File)
Sun, Mar 31, 7:56 PM
Unknown Object (File)
Wed, Mar 27, 4:39 PM
Unknown Object (File)
Tue, Mar 26, 1:30 PM
Unknown Object (File)
Tue, Mar 26, 1:30 PM
Unknown Object (File)
Tue, Mar 26, 1:29 PM
Unknown Object (File)
Tue, Mar 26, 1:29 PM
Subscribers
None

Details

Summary

Depends on D20129. Ref T13244. See PHI1058. When a revision has an "Accept" from a package, count the owners as "involved" in the change whether or not any actual human owners are actually accepting reviewers.

If a user owns "/" and uses "force accept" to cause "/src/javascript" to accept, or a user who legitimately owns "/src/javascript" accepts on behalf of the package but not on behalf of themselves (for whatever reason), it generally makes practical sense that these changes have owners involved in them (i.e., that's what a normal user would expect in both cases) and don't need to trigger audits under "no involvement" rules.

Test Plan

Used bin/repository reparse --force --owners <commit> to trigger audit logic. Saw a commit owned by O1 with a revision counted as "involved" when O1 had accepted the revision, even though no actual human owner had accepted it.

Diff Detail

Repository
rP Phabricator
Branch
audit5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21889
Build 29885: Run Core Tests
Build 29884: arc lint + arc unit