Page MenuHomePhabricator

Simply how Differential drafts ignore Harbormaster autobuilds
ClosedPublic

Authored by epriestley on Oct 20 2017, 6:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 7:29 AM
Unknown Object (File)
Tue, Dec 10, 6:04 PM
Unknown Object (File)
Sat, Dec 7, 5:41 PM
Unknown Object (File)
Thu, Dec 5, 11:08 PM
Unknown Object (File)
Wed, Dec 4, 2:48 PM
Unknown Object (File)
Wed, Nov 27, 3:33 PM
Unknown Object (File)
Nov 22 2024, 7:06 PM
Unknown Object (File)
Nov 18 2024, 9:28 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T2543. When a revision is created, we check if any builds are waiting/failed, and submit it for review immediately if we aren't waiting for anything.

In doing this, we ignore builds with only autotargets, since these are client-side and failures from local arc lint / arc unit should not count (the user has already chosen to ignore/skip them).

The way we do this has some issues:

  • Herald may have started builds, but they may still be PENDING and not have any targets yet. In this case, we'll see "no non-autotargets" and ignore the build, which is wrong.
  • We have to load targets but don't really care about them, which is more work than we really need to do.
  • And it's kind of complex, too.

Instead, just let BuildQuery filter out "autobuilds" (builds generated from autoplans) with a JOIN.

Test Plan

Ran arc diff with builds configured, got a clean "Draft" state instead of an incorrect promotion directly to "Needs Review".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Oct 20 2017, 6:59 PM
This revision is now accepted and ready to land.Oct 21 2017, 4:17 PM
This revision was automatically updated to reflect the committed changes.