Page MenuHomePhabricator

Fail in a more comprehensible way when a WorkingCopy lease omits or mangles "repositories.map"
ClosedPublic

Authored by epriestley on May 3 2022, 6:36 PM.
Tags
None
Referenced Files
F13208025: D21796.diff
Thu, May 16, 7:52 AM
F13201239: D21796.diff
Tue, May 14, 9:44 AM
F13198271: D21796.diff
Mon, May 13, 5:32 AM
F13193998: D21796.diff
Sun, May 12, 8:33 PM
F13183313: D21796.diff
Fri, May 10, 8:17 AM
F13177107: D21796.id51952.diff
Wed, May 8, 7:19 PM
Unknown Object (File)
Tue, May 7, 9:57 AM
Unknown Object (File)
Sun, May 5, 8:42 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13676. When the required "repositories.map" attribute is omitted, bin/drydock lease currently fatals in an unhelpful way when trying to lease a working copy.

Test Plan

Ran bin/drydock lease --type working-copy with no attributes, after following steps in T13676.

<Allocation Failed> One or more blueprints promised a new resource, but failed when allocating: [PhutilAggregateException] All blueprints failed to allocate a suitable new resource when trying to allocate lease ("PHID-DRYL-orbtwtlinksm3xqpyhmw").
    - Exception: Working copy lease is missing required attribute "repositories.map".
      
      Attribute "repositories.map" should be a map of repository specifications.

Diff Detail

Repository
rP Phabricator
Branch
drydock2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25690
Build 35518: arc lint + arc unit