Page MenuHomePhabricator
Differential D16566

Manage Drydock leases from arcanist
Needs ReviewPublic
Actions

Authored by yelirekim on Sep 17 2016, 1:57 AM.
Tags
None
Referenced Files
F14050481: D16566.id39875.diff
Thu, Nov 14, 6:52 PM
F14048048: D16566.id39876.diff
Thu, Nov 14, 6:36 AM
F14047927: D16566.id.diff
Thu, Nov 14, 5:50 AM
F14047825: D16566.id39875.diff
Thu, Nov 14, 5:26 AM
F14047810: D16566.id39876.diff
Thu, Nov 14, 5:22 AM
F14047796: D16566.id.diff
Thu, Nov 14, 5:19 AM
F14047619: D16566.diff
Thu, Nov 14, 4:39 AM
F13959141: D16566.id39876.diff
Oct 14 2024, 6:02 PM
View All Files
Subscribers
chad
epriestley
Tokens
"Love" token, awarded by hach-que.

Details

Reviewers
None
Group Reviewers
Blessed Reviewers
Summary

Depends on D16565. Glues together all of the methods therein so that arcanist can acquire and release leases.

Test Plan

leased a working copy using arc lease, released it using arc release

Diff Detail

Repository
rARC Arcanist
Branch
lease_workflow (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13753
Build 17761: Run Core Tests
Build 17760: arc lint + arc unit

Event Timeline

yelirekim updated this revision to Diff 39875.Sep 17 2016, 1:57 AM
yelirekim retitled this revision from to Manage Drydock leases from arcanist.
yelirekim updated this object.
yelirekim edited the test plan for this revision. (Show Details)
yelirekim added a parent revision: D16565: Expose Drydock lease management from conduit.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 17 2016, 1:57 AM
Herald added a subscriber: epriestley. · View Herald Transcript
yelirekim updated this revision to Diff 39876.Sep 17 2016, 1:58 AM
yelirekim edited edge metadata.

fix lint problem

chad added a subscriber: chad.Sep 17 2016, 2:14 AM
chad added inline comments.
src/workflow/ArcanistReleaseWorkflow.php
92–94

do these need pht()?

hach-que awarded a token.Sep 18 2016, 3:33 AM
yelirekim added inline comments.Sep 19 2016, 5:22 PM
src/workflow/ArcanistReleaseWorkflow.php
92–94

I don't see why they wouldn't, but also I am really lazy today. I will do this tomorrow / or whenever I'm pretty sure you guys don't want me to actually just delete all of this code.

epriestley added a comment.Sep 21 2016, 4:50 PM

The shape of this looks generally reasonable to me, but I don't want to bring some of the endpoints it relies on upstream yet.

(For anyone finding their way here, D16565 has more discussion.)

yelirekim mentioned this in T11693: Make drydock command interfaces accessible via SSH workflows.Sep 24 2016, 4:12 AM
• liugev6 added a commit: rP88a966993be1: Ajax commenting on lightbox images.Nov 22 2016, 8:51 AM
epriestley removed a commit: rP88a966993be1: Ajax commenting on lightbox images.Nov 22 2016, 12:09 PM
• TweikerManager added a commit: rP2604c5af55f6: (stable) Promote 2017 Week 4.Jan 29 2017, 10:50 PM
epriestley removed a commit: rP2604c5af55f6: (stable) Promote 2017 Week 4.Jan 29 2017, 10:52 PM

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
src/
4 lines--
workflow/
214 lines0%
Loading...
108 lines0%
Loading...

Diff 39876

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/workflow/ArcanistLeaseWorkflow.php

Loading...
View Options

src/workflow/ArcanistReleaseWorkflow.php

Loading...
Phabricator · Built by Phacility