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

	F19089774: D19055.diff
	Dec 3 2025, 9:03 AM

	F18865563: D19055.id45688.diff
	Nov 3 2025, 12:54 PM

	F18856146: D19055.id.diff
	Nov 1 2025, 8:15 AM

	F18852448: D19055.diff
	Oct 31 2025, 9:08 AM

	F18807643: D19055.id45688.diff
	Oct 19 2025, 2:40 AM

	F18737083: D19055.id.diff
	Oct 1 2025, 8:53 AM

	F18727815: D19055.diff
	Sep 30 2025, 8:49 AM

	F18670204: D19055.id45683.diff
	Sep 24 2025, 11:25 PM

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...