launchpad-reviewers team mailing list archive

[Jürgen Gmach <mp+474077@xxxxxxxxxxxxxxxxxx>]

Jürgen Gmach has proposed merging ~jugmac00/launchpad:end-proxy-session-explicitly into launchpad:master.

Commit message:
End fetch service proxy session explicitly

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~jugmac00/launchpad/+git/launchpad/+merge/474077

Similar to https://git.launchpad.net/launchpad/commit/?id=dd90403bd5235d9f35a9844db9ce857871eed418
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~jugmac00/launchpad:end-proxy-session-explicitly into launchpad:master.

diff --git a/lib/lp/rocks/model/rockrecipebuildbehaviour.py b/lib/lp/rocks/model/rockrecipebuildbehaviour.py
index 65301ba..58ec5bb 100644
--- a/lib/lp/rocks/model/rockrecipebuildbehaviour.py
+++ b/lib/lp/rocks/model/rockrecipebuildbehaviour.py
@@ -124,3 +124,7 @@ class RockRecipeBuildBehaviour(BuilderProxyMixin, BuildFarmJobBehaviourBase):
         # that check does not make sense.  We do, however, refuse to build
         # for obsolete series.
         assert self.build.distro_series.status != SeriesStatus.OBSOLETE
+
+    @defer.inlineCallbacks
+    def _saveBuildSpecificFiles(self, upload_path):
+        yield self.endProxySession(upload_path)