Differential D13341

Drop Windows-specific escaping in preallocated host
ClosedPublic
Actions

Authored by hach-que on Jun 18 2015, 3:04 PM.

Tags

Referenced Files

	F15468009: D13341.id.diff
	Thu, Apr 3, 7:19 PM

	F15466252: D13341.id32279.diff
	Thu, Apr 3, 3:28 AM

	F15464296: D13341.diff
	Wed, Apr 2, 8:16 AM

	F15459647: D13341.id32279.diff
	Mon, Mar 31, 2:53 PM

	F15439102: D13341.id.diff
	Wed, Mar 26, 5:45 AM

	F15336462: D13341.diff
	Sat, Mar 8, 6:58 PM

	Unknown Object (File)
	Feb 25 2025, 10:38 AM

	Unknown Object (File)
	Feb 9 2025, 9:36 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP70a82017b3b9: Drop Windows-specific escaping in preallocated host

Summary

This drops the Windows-specific escaping code for the creation of directories when acquiring a lease. This is basically the change from D10378 without the other, no longer necessary changes.

Test Plan

This code hasn't been run in a production environment for a while (any instances of Phabricator using Drydock / Harbormaster with Windows have had this code removed by the D10378 patch for a while).

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

hach-que updated this revision to Diff 32279.Jun 18 2015, 3:04 PM

hach-que retitled this revision from to Drop Windows-specific escaping in preallocated host.

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 TranscriptJun 18 2015, 3:04 PM

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

hach-que mentioned this in D10378: Spawn a new Powershell session when running commands on Windows.Jun 18 2015, 3:04 PM

hach-que mentioned this in D10495: [drydock/core] Use WinRM instead of SSH for Windows hosts.Jun 18 2015, 3:21 PM

hach-que edited edge metadata.Jun 18 2015, 4:11 PM

hach-que added a project: Drydock.

epriestley accepted this revision.Jun 19 2015, 12:17 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 19 2015, 12:17 AM

Closed by commit rP70a82017b3b9: Drop Windows-specific escaping in preallocated host (authored by hach-que, committed by hach-que). · Explain WhyJun 19 2015, 5:07 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

drydock/

blueprint/

DrydockPreallocatedHostBlueprintImplementation.php

15 lines

Diff 32313

src/applications/drydock/blueprint/DrydockPreallocatedHostBlueprintImplementation.php

Loading...