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

	F13905407: D19055.id45688.diff
	Mon, Oct 7, 8:40 PM

	F13905406: D19055.id45683.diff
	Mon, Oct 7, 8:40 PM

	F13905404: D19055.id.diff
	Mon, Oct 7, 8:40 PM

	Unknown Object (File)
	Sat, Oct 5, 9:48 PM

	Unknown Object (File)
	Sep 6 2024, 2:17 AM

	Unknown Object (File)
	Sep 6 2024, 2:17 AM

	Unknown Object (File)
	Sep 6 2024, 2:16 AM

	Unknown Object (File)
	Sep 2 2024, 6:51 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...