From a1d1cf77a7884a108b393bbf7a2c0e313139ce80 Mon Sep 17 00:00:00 2001 From: Chad Little Date: Tue, 18 Aug 2015 17:52:55 -0700 Subject: [PATCH] Remove call to loadDrydockLease Summary: Fixes T9219... I think. Test Plan: swagging a guess for epriestley Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Maniphest Tasks: T9219 Differential Revision: https://secure.phabricator.com/D13932 --- .../HarbormasterUploadArtifactBuildStepImplementation.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/applications/harbormaster/step/HarbormasterUploadArtifactBuildStepImplementation.php b/src/applications/harbormaster/step/HarbormasterUploadArtifactBuildStepImplementation.php index 88651826e7..f8bdb7fdfd 100644 --- a/src/applications/harbormaster/step/HarbormasterUploadArtifactBuildStepImplementation.php +++ b/src/applications/harbormaster/step/HarbormasterUploadArtifactBuildStepImplementation.php @@ -25,6 +25,7 @@ final class HarbormasterUploadArtifactBuildStepImplementation public function execute( HarbormasterBuild $build, HarbormasterBuildTarget $build_target) { + $viewer = PhabricatorUser::getOmnipotentUser(); $settings = $this->getSettings(); $variables = $build_target->getVariables(); @@ -35,7 +36,8 @@ final class HarbormasterUploadArtifactBuildStepImplementation $variables); $artifact = $build_target->loadArtifact($settings['hostartifact']); - $lease = $artifact->loadDrydockLease(); + $impl = $artifact->getArtifactImplementation(); + $lease = $impl->loadArtifactLease($viewer); $interface = $lease->getInterface('filesystem'); @@ -44,7 +46,7 @@ final class HarbormasterUploadArtifactBuildStepImplementation // Insert the artifact record. $artifact = $build_target->createArtifact( - PhabricatorUser::getOmnipotentUser(), + $viewer, $settings['name'], HarbormasterFileArtifact::ARTIFACTCONST, array(