Page MenuHomePhabricator

Give Buildables a status, populate it, and return it over Conduit
ClosedPublic

Authored by epriestley on Apr 17 2014, 3:33 PM.
Tags
None
Referenced Files
F18739064: D8799.id.diff
Wed, Oct 1, 7:26 PM
F18727733: D8799.diff
Tue, Sep 30, 8:44 AM
F18629489: D8799.diff
Tue, Sep 16, 9:02 AM
F18557462: D8799.id.diff
Mon, Sep 8, 7:58 PM
F18545298: D8799.diff
Mon, Sep 8, 1:45 AM
F18513779: D8799.diff
Fri, Sep 5, 9:45 AM
F18441394: D8799.id.diff
Aug 31 2025, 4:26 PM
F18413918: D8799.diff
Aug 30 2025, 8:03 AM
Subscribers

Details

Summary

Ref T4809. Currently, buildables have a status field but nothing populates it. Populate it:

  • When builds change state, update the Buildable state.
  • Use the new Buildable state on the web UI.
  • Return the new Buildable state from Conduit.

To make it easier to debug/test this:

  • Provide bin/harbormaster update Bxxx ... to force foreground update of a Buildable.
Test Plan
  • Used bin/harbormaster update Bxxx --force --trace to update buildables.
  • Looked at buidlable list, saw statuses reported properly.
  • Used Conduit to read statuses.

Diff Detail

Repository
rP Phabricator
Branch
hm4
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

epriestley retitled this revision from to Give Buildables a status, populate it, and return it over Conduit.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
src/applications/harbormaster/conduit/ConduitAPI_harbormaster_sendmessage_Method.php
48–52

This basically does the same thing, but does it in the task queue so if we end up having to wait for the lock for a while it doesn't block the caller. I'd just forgotten that we already have this.

src/applications/harbormaster/query/HarbormasterBuildableSearchEngine.php
56

We no longer need to load the builds because we can read the status directly from the Buildable.

src/applications/harbormaster/storage/HarbormasterBuildable.php
19

goodbye venerable "whatever" status

btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 17 2014, 6:31 PM
epriestley updated this revision to Diff 20896.

Closed by commit rP0ef599e90676 (authored by @epriestley).