Page MenuHomePhabricator

Make Harbormaster input and output artifacts more explicit
ClosedPublic

Authored by epriestley on Mar 23 2014, 5:26 PM.
Tags
None
Referenced Files
F14060120: D8599.diff
Sun, Nov 17, 11:17 PM
F14035234: D8599.diff
Sun, Nov 10, 5:06 AM
F13997119: D8599.id20429.diff
Thu, Oct 24, 2:20 AM
F13993034: D8599.diff
Tue, Oct 22, 8:16 PM
F13993031: D8599.diff
Tue, Oct 22, 8:13 PM
F13993010: D8599.id.diff
Tue, Oct 22, 8:04 PM
F13993005: D8599.id20388.diff
Tue, Oct 22, 8:03 PM
F13992995: D8599.id20429.diff
Tue, Oct 22, 8:01 PM
Subscribers

Details

Summary

Ref T1049. In Harbormaster, build steps may have various inputs (like a host they should run on) and outputs (like a reference to an uploaded file).

  • Currently, inputs aren't defined anywhere (except implicitly at runtime).
    • Instead, define inputs explicitly.
  • Currently, outputs are defined in a way that loses information when misconfigured (the keys will collide).
    • Instead, define inputs and outputs so they work whether a step is configured correctly or not.
  • Currently, there's no simple way to see a step's inputs and outputs.
    • Add some UI for this.
  • Currently, reordering steps has some surprising side effects.
    • Instead of invalidating steps after reordering them, validate them at display time and warn the user.
Test Plan

Screen_Shot_2014-03-23_at_10.15.16_AM.png (1×1 px, 172 KB)

Screen_Shot_2014-03-23_at_10.15.21_AM.png (1×1 px, 163 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Make Harbormaster input and output artifacts more explicit.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

One thing this affects is that we no longer validate other fields on display. The next diff will fix this in a general way.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 25 2014, 8:17 PM
epriestley updated this revision to Diff 20429.

Closed by commit rP72337dedaff9 (authored by @epriestley).