Page MenuHomePhabricator
Differential D19077

Don't let `bin/drydock lease --attributes` overwrite blueprints
ClosedPublic
Actions

Authored by epriestley on Feb 13 2018, 5:35 PM.
Tags
None
Referenced Files
F18750260: D19077.id45721.diff
Oct 4 2025, 7:09 AM
F18735567: D19077.diff
Oct 1 2025, 2:15 AM
F18649706: D19077.diff
Sep 20 2025, 10:03 PM
F18509314: D19077.id.diff
Sep 5 2025, 3:20 AM
F18502154: D19077.diff
Sep 4 2025, 10:14 PM
F18369555: D19077.id45733.diff
Aug 28 2025, 2:22 AM
F18358793: D19077.diff
Aug 27 2025, 8:45 AM
F18111717: D19077.id.diff
Aug 12 2025, 11:32 AM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13073: Plans: Drydock for normal software use cases where builds take more than 45 seconds
Commits
rP2994753d2333: Don't let `bin/drydock lease --attributes` overwrite blueprints
Summary

Depends on D19076. Ref T13073. Blueprints are stored as an attribute and setAttributes() overwrites all attributes.

This is sorta junk but make it less obviously broken, at least.

Test Plan

Ran bin/drydock lease --type working-copy --attributes x=y without instantly getting a fatal about "no blueprint PHIDs".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 13 2018, 5:35 PM
Owners added a subscriber: Restricted Owners Package.Feb 13 2018, 5:35 PM
Harbormaster completed remote builds in B19524: Diff 45721.Feb 13 2018, 5:36 PM
epriestley requested review of this revision.Feb 13 2018, 5:36 PM
epriestley added a child revision: D19078: Reduce collision rate for concurrency-limiting slot locks.Feb 13 2018, 5:53 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 13 2018, 9:20 PM
Closed by commit rP2994753d2333: Don't let `bin/drydock lease --attributes` overwrite blueprints (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
drydock/
management/
8 lines

Diff 45733

View Options

src/applications/drydock/management/DrydockManagementLeaseWorkflow.php

Loading...
Phabricator · Built by Phacility