Page MenuHomePhabricator

Write an explicit edge for commit membership in packages
ClosedPublic

Authored by epriestley on Jan 30 2017, 7:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 11, 3:54 AM
Unknown Object (File)
Mon, Dec 9, 3:25 PM
Unknown Object (File)
Fri, Dec 6, 9:34 PM
Unknown Object (File)
Thu, Dec 5, 12:11 PM
Unknown Object (File)
Wed, Nov 27, 12:10 AM
Unknown Object (File)
Mon, Nov 25, 9:00 PM
Unknown Object (File)
Sat, Nov 23, 12:23 AM
Unknown Object (File)
Tue, Nov 19, 6:38 AM
Subscribers
None

Details

Summary

Ref T10978. Currently, during commit import, we write an "Audit Not Required" auditor for commits which don't require an audit.

This auditor is used to power the "Commits in this package" query in Owners.

This conflates audits and commit/package membership. I think it might even predate edges. Code needs to dance around this mess and we get the wrong result in some cases, since auditors are now editable.

Instead, write an explicit edge which just says "this commit is part of such-and-such packages". Then use that to run the query. Logical!

I'll issue guidance on this but I'm not migrating it, since it fixes itself going forward and only really affects the UI in Owners.

Test Plan
  • Ran bin/audit update-owners with various arguments.
  • Viewed packages in web UI, saw them load the proper commits.
  • Queried by packages in Diffusion explicitly.
  • Clicked the "View All" link in Owners and got to the right search UI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable