Page MenuHomePhabricator

In "bin/drydock lease", take a JSON "--attributes" so we can accept complex values
ClosedPublic

Authored by epriestley on Oct 24 2018, 2:22 PM.
Tags
None
Referenced Files
F19700958: D19751.diff
Tue, Feb 10, 9:56 PM
F19668774: D19751.id.diff
Sat, Feb 7, 7:50 PM
F19647685: D19751.id.diff
Fri, Feb 6, 3:58 AM
F19555908: D19751.diff
Fri, Jan 30, 7:07 PM
F19532000: D19751.diff
Tue, Jan 20, 8:18 AM
F19528321: D19751.diff
Jan 19 2026, 8:19 AM
F19122298: D19751.diff
Dec 9 2025, 11:21 PM
F19060038: D19751.id47183.diff
Nov 29 2025, 7:43 AM
Subscribers
Restricted Owners Package

Details

Summary

Depends on D19750. See T13210. The bin/drydock lease command makes it easier to request ad-hoc leases, but currently takes lease attributes in the form --attributes x=y,a=b.

This was okay for all leases at the time, but doesn't really work for modern WorkingCopy resources since they take a repositories.map which has a dictionary as a value. You can't specify that with repositories.map=....

Instead, point --attributes at a JSON file or use --attributes - to read from stdin.

Test Plan

Used --attributes with a file and stdin to allocate working copy leases with repositories.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable