Paths

Table of Contentst

Differential D19055

When a build is aborted, fail the buildable
ClosedPublic
Actions

Authored by epriestley on Feb 10 2018, 7:38 PM.

Tags

None

Referenced Files

	F18361865: D19055.id45688.diff
	Wed, Aug 27, 12:50 PM

	F18361796: D19055.id45683.diff
	Wed, Aug 27, 12:42 PM

	F18349979: D19055.diff
	Tue, Aug 26, 7:02 PM

	F18263137: D19055.id.diff
	Fri, Aug 22, 8:47 AM

	F18245184: D19055.diff
	Thu, Aug 21, 9:10 AM

	F18107233: D19055.diff
	Sun, Aug 10, 11:47 PM

	F18089022: D19055.id45683.diff
	Wed, Aug 6, 10:15 AM

	F17868705: D19055.diff
	Jul 28 2025, 8:52 AM

Subscribers

Restricted Owners Package

Tokens

"Manufacturing Defect?" token, awarded by yelirekim.

Details

Reviewers: None
Maniphest Tasks: T10746: Aborted build in Harbormaster still shows as building in Diffusion
T11154: Build results don't always get conferred to their buildable
T13054: Plans: Harbormaster Interactions with Differential / Chaos Reduction
Commits: rPa2d02aed2211: When a build is aborted, fail the buildable

Summary

Ref T13054. Fixes T10746. Fixes T11154. This is really a one-line fix (include ABORTED in BuildEngine->updateBuildable()) but try to structure the code a little more clearly too and reduce (at least slightly) the number of random lists of status attributes spread throughout the codebase.

Also add a header tag for buildable status.

Test Plan

Aborted a build, saw buildable fail properly.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 10 2018, 7:38 PM

Owners added a subscriber: Restricted Owners Package.Feb 10 2018, 7:38 PM

Harbormaster completed remote builds in B19485: Diff 45683.Feb 10 2018, 7:39 PM

epriestley requested review of this revision.Feb 10 2018, 7:39 PM

🐶 🐶 🐶

This revision was not accepted when it landed; it landed in state Needs Review.Feb 11 2018, 12:08 AM

Closed by commit rPa2d02aed2211: When a build is aborted, fail the buildable (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

harbormaster/

constants/

HarbormasterBuildStatus.php

115 lines

controller/

HarbormasterBuildableViewController.php

4 lines

engine/

HarbormasterBuildEngine.php

8 lines

storage/

HarbormasterBuildable.php

12 lines

build/

HarbormasterBuild.php

19 lines

Diff 45688

src/applications/harbormaster/constants/HarbormasterBuildStatus.php

Loading...

src/applications/harbormaster/controller/HarbormasterBuildableViewController.php

Loading...

src/applications/harbormaster/engine/HarbormasterBuildEngine.php

Loading...

src/applications/harbormaster/storage/HarbormasterBuildable.php

Loading...

src/applications/harbormaster/storage/build/HarbormasterBuild.php

Loading...