Remove localhost Drydock allocator
ClosedPublic
Actions

Authored by hach-que on Aug 18 2014, 9:35 AM.

Tags

None

Referenced Files

	F15458392: D10287.id24799.diff
	Sun, Mar 30, 11:33 PM

	F15458390: D10287.id24775.diff
	Sun, Mar 30, 11:33 PM

	F15457255: D10287.diff
	Sun, Mar 30, 3:16 PM

	F15425505: D10287.id24775.diff
	Sun, Mar 23, 4:50 AM

	F15419162: D10287.diff
	Fri, Mar 21, 4:08 AM

	F15407348: D10287.id24799.diff
	Tue, Mar 18, 5:04 PM

	F15386918: D10287.diff
	Sat, Mar 15, 1:31 AM

	F15384280: D10287.id.diff
	Fri, Mar 14, 7:22 PM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rPdf7fb098451c: Remove localhost Drydock allocator

Summary

This has never been enabled by default, and isn't safe. Remove it since people can use preallocated or EC2 hosts.

Test Plan

Removed it; didn't see it appear on the "Create Blueprint" page.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

hach-que updated this revision to Diff 24775.Aug 18 2014, 9:35 AM

hach-que retitled this revision from to Remove localhost Drydock allocator.

hach-que updated this object.

hach-que edited the test plan for this revision. (Show Details)

hach-que added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 18 2014, 9:35 AM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B2263: Diff 24775.Aug 18 2014, 9:36 AM

epriestley accepted this revision.Aug 19 2014, 9:49 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Aug 19 2014, 9:49 PM

Closed by commit rPdf7fb098451c (authored by @hach-que).

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

drydock/

blueprint/

DrydockLocalHostBlueprintImplementation.php

Diff 24799

View Options

src/__phutil_library_map__.php