From 254b394f2c2b299e29f8a16f93aa80488028afe7 Mon Sep 17 00:00:00 2001 From: June Rhodes Date: Mon, 24 Aug 2015 17:20:59 +1000 Subject: [PATCH] Associate Harbormaster build target with leases Summary: Ref T1049. This ensures the Harbormaster build target is associated with leases, so in the future we can query things and find out whether builds are still running with associated leases. Test Plan: Leased a host, checked the DB and saw the field populated. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: joshuaspence, Korvin, epriestley Maniphest Tasks: T1049 Differential Revision: https://secure.phabricator.com/D10870 --- .../step/HarbormasterLeaseHostBuildStepImplementation.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/harbormaster/step/HarbormasterLeaseHostBuildStepImplementation.php b/src/applications/harbormaster/step/HarbormasterLeaseHostBuildStepImplementation.php index 127025fd26..b14d0975f7 100644 --- a/src/applications/harbormaster/step/HarbormasterLeaseHostBuildStepImplementation.php +++ b/src/applications/harbormaster/step/HarbormasterLeaseHostBuildStepImplementation.php @@ -24,6 +24,7 @@ final class HarbormasterLeaseHostBuildStepImplementation // Create the lease. $lease = id(new DrydockLease()) ->setResourceType('host') + ->setOwnerPHID($build_target->getPHID()) ->setAttributes( array( 'platform' => $settings['platform'],