Page MenuHomePhabricator

Allow Harbormaster builds to publish to a different object
ClosedPublic

Authored by epriestley on Jan 4 2017, 9:24 PM.
Tags
None
Referenced Files
F14056217: D17139.diff
Sat, Nov 16, 6:57 PM
F14022405: D17139.id.diff
Wed, Nov 6, 5:02 PM
F13992007: D17139.id41218.diff
Tue, Oct 22, 1:51 PM
F13988131: D17139.id41219.diff
Mon, Oct 21, 1:23 PM
F13979872: D17139.id41217.diff
Oct 19 2024, 6:59 AM
Unknown Object (File)
Oct 9 2024, 6:45 AM
Unknown Object (File)
Oct 9 2024, 6:45 AM
Unknown Object (File)
Oct 9 2024, 6:44 AM
Subscribers
Restricted Owners Package

Details

Summary

Fixes T9276. Fixes T8650. The story so far:

  • We once published build updates to Revisions.
  • An unrelated fix (D10911) sent them to the Diffs instead of Revisions, which isn't useful, since you can't see a diff's timeline anywhere.
  • This also caused a race condition, where the RevisionEditor and DiffEditor would update the diff simultaneously (T8650).
  • The diff update was just disabled to avoid the race (part of D13441).
  • Instead, allow the updates to go somewhere else. In this case, we send commit updates to the commit but send diff updates to the revision so you can see 'em.
  • Since everything will be using the revision editor now, we should either get proper lock behavior for free or it should be easy to add if something whack is still happening.
  • Overall, this should pretty much put us back in working order like we were before D10911.

This behavior is undoubtedly refinable, but this should let us move forward.

Test Plan

Saw a build failure in timeline:

Screen Shot 2017-01-04 at 1.18.31 PM.png (432×1 px, 70 KB)

Diff Detail

Repository
rP Phabricator
Branch
harbor1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15103
Build 19827: Run Core Tests
Build 19826: arc lint + arc unit