Paths

Table of Contentst

Differential D19066

Update buildable containerPHIDs in a proper way via BuildWorker rather than via sneaky uncoordinated write
ClosedPublic
Actions

Authored by epriestley on Feb 12 2018, 8:11 PM.

Tags

None

Referenced Files

	F13368663: D19066.diff
	Thu, Jun 27, 7:29 AM

	F13367560: D19066.id45707.diff
	Thu, Jun 27, 1:54 AM

	F13360382: D19066.diff
	Tue, Jun 25, 11:17 AM

	F13356809: D19066.diff
	Mon, Jun 24, 9:06 PM

	F13345398: D19066.diff
	Fri, Jun 21, 10:25 AM

	F13342008: D19066.id.diff
	Thu, Jun 20, 9:52 AM

	F13312931: D19066.diff
	Tue, Jun 11, 7:19 AM

	F13299493: D19066.diff
	Fri, Jun 7, 8:04 AM

View All 95 Files

Subscribers

Restricted Owners Package

Details

Reviewers: None
Maniphest Tasks: T13054: Plans: Harbormaster Interactions with Differential / Chaos Reduction
Commits: rP6f508a225869: Update buildable containerPHIDs in a proper way via BuildWorker rather than via…

Summary

Depends on D19065. Ref T13054. Instead of just updating containerPHID and hoping for the best, queue a proper BuildWorker to process a "your container has changed, update it" message.

We also need to remove a (superfluous) withContainerPHIDs() when loading active diffs for a revision.

Test Plan

Without daemons, created a revision and saw builds stick in "preparing" with no container PHID, but also stay in draft mode.
With daemons, saw builds actually build and get the right container PHID.

Diff Detail

Repository

Branch

hrace5

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 19505
Build 26392: Run Core Tests
Build 26391: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 12 2018, 8:11 PM

Owners added a subscriber: Restricted Owners Package.Feb 12 2018, 8:11 PM

Harbormaster completed remote builds in B19505: Diff 45702.Feb 12 2018, 8:12 PM

epriestley requested review of this revision.Feb 12 2018, 8:12 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 12 2018, 8:18 PM

Closed by commit rP6f508a225869: Update buildable containerPHIDs in a proper way via BuildWorker rather than via… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

editor/

DifferentialTransactionEditor.php

31 lines

storage/

DifferentialRevision.php

3 lines

harbormaster/

engine/

HarbormasterBuildEngine.php

19 lines

HarbormasterMessageType.php

1 line

Diff 45702

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...

src/applications/differential/storage/DifferentialRevision.php

Loading...

src/applications/harbormaster/engine/HarbormasterBuildEngine.php

Loading...

src/applications/harbormaster/engine/HarbormasterMessageType.php

Loading...