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

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

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

	Unknown Object (File)
	Sat, Feb 8, 12:12 PM

	Unknown Object (File)
	Sat, Feb 8, 12:12 PM

	Unknown Object (File)
	Sat, Feb 8, 12:11 PM

	Unknown Object (File)
	Feb 1 2025, 8:38 PM

	Unknown Object (File)
	Feb 1 2025, 3:00 AM

	Unknown Object (File)
	Jan 31 2025, 6:51 AM

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

Branch

drop-windows-manual-escaping

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 6844
Build 6866: [Placeholder Plan] Wait for 30 Seconds

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 32279

src/applications/drydock/blueprint/DrydockPreallocatedHostBlueprintImplementation.php

Loading...