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

	F18603342: D19055.diff
	Sat, Sep 13, 4:44 PM

	F18601109: D19055.diff
	Sat, Sep 13, 11:32 AM

	F18582353: D19055.id.diff
	Thu, Sep 11, 8:16 AM

	F18509290: D19055.id.diff
	Fri, Sep 5, 3:20 AM

	F18502122: D19055.diff
	Thu, Sep 4, 10:13 PM

	F18382080: D19055.id45683.diff
	Thu, Aug 28, 8:55 PM

	F18366379: D19055.diff
	Wed, Aug 27, 10:16 PM

	F18361865: D19055.id45688.diff
	Wed, Aug 27, 12:50 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...