Page MenuHomePhabricator

Use Drydock authorizations when acquiring leases
ClosedPublic

Authored by epriestley on Oct 12 2015, 1:26 PM.
Tags
None
Referenced Files
F15459132: D14254.id34416.diff
Mon, Mar 31, 10:04 AM
F15435238: D14254.id.diff
Tue, Mar 25, 7:28 AM
F15430134: D14254.diff
Mon, Mar 24, 6:04 AM
F15422460: D14254.id34416.diff
Sat, Mar 22, 8:31 AM
F15416530: D14254.id34427.diff
Thu, Mar 20, 11:49 AM
F15381745: D14254.id34427.diff
Fri, Mar 14, 9:12 AM
F15335988: D14254.diff
Sat, Mar 8, 5:59 PM
F15298859: D14254.id34416.diff
Mar 5 2025, 1:52 PM
Subscribers
None

Details

Summary

Ref T9519. When acquiring leases on resources:

  • Only consider resources created by authorized blueprints.
  • Only consider authorized blueprints when creating new resources.
  • Fail with a tailored error if no blueprints are allowed.
  • Fail with a tailored error if missing authorizations are causing acquisition failure.

One somewhat-substantial issue with this is that it's pretty hard to figure out from the Harbormaster side. Specifically, the Build step UI does not show field value anywhere, so the presence of unapproved blueprints is not communicated. This is much more clear in Drydock. I'll plan to address this in future changes to Harbormaster, since there are other related/similar issues anyway.

Test Plan

Screen Shot 2015-10-12 at 5.43.47 AM.png (272×1 px, 51 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Use Drydock authorizations when acquiring leases.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, hach-que.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 12 2015, 2:08 PM
This revision was automatically updated to reflect the committed changes.