From ddfa5cbdf67f056af13fbd6bfe60650cef7b00c4 Mon Sep 17 00:00:00 2001 From: James Rhodes Date: Wed, 27 Aug 2014 08:27:46 +1000 Subject: [PATCH] Remove setWorkingDirectory call on SFTP interface Summary: I derped on this; the SFTP interface doesn't have setWorkingDirectory because it implements DrydockFilesystemInterface and not DrydockCommandInterface. So when you use the Upload File build step, the daemon will crash due to an undefined method. Test Plan: Tested on my live server. Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: epriestley, Korvin Differential Revision: https://secure.phabricator.com/D10351 --- .../DrydockPreallocatedHostBlueprintImplementation.php | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/applications/drydock/blueprint/DrydockPreallocatedHostBlueprintImplementation.php b/src/applications/drydock/blueprint/DrydockPreallocatedHostBlueprintImplementation.php index d9b705fe10..9ee4830989 100644 --- a/src/applications/drydock/blueprint/DrydockPreallocatedHostBlueprintImplementation.php +++ b/src/applications/drydock/blueprint/DrydockPreallocatedHostBlueprintImplementation.php @@ -115,8 +115,7 @@ final class DrydockPreallocatedHostBlueprintImplementation ->setConfiguration(array( 'host' => $resource->getAttribute('host'), 'port' => $resource->getAttribute('port'), - 'credential' => $resource->getAttribute('credential'))) - ->setWorkingDirectory($lease->getAttribute('path')); + 'credential' => $resource->getAttribute('credential'))); } throw new Exception("No interface of type '{$type}'.");