Merge lp:~cjwatson/launchpad-buildd/snap-proxy-allow-build into lp:launchpad-buildd

Proposed by Colin Watson
Status: Merged
Merged at revision: 205
Proposed branch: lp:~cjwatson/launchpad-buildd/snap-proxy-allow-build
Merge into: lp:launchpad-buildd
Diff against target: 32 lines (+12/-1)
2 files modified
buildsnap (+5/-1)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~cjwatson/launchpad-buildd/snap-proxy-allow-build
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+310050@code.launchpad.net

Commit message

buildsnap: Grant access to the proxy during the build phase as well as during the pull phase.

Description of the change

As discussed at the devices sprint.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'buildsnap'
--- buildsnap 2016-09-16 10:43:09 +0000
+++ buildsnap 2016-11-04 10:53:13 +0000
@@ -136,8 +136,12 @@
136 def build(self):136 def build(self):
137 """Run all build, stage and snap phases."""137 """Run all build, stage and snap phases."""
138 print("Running build phase...")138 print("Running build phase...")
139 env = {}
140 if self.options.proxy_url:
141 env["http_proxy"] = self.options.proxy_url
142 env["https_proxy"] = self.options.proxy_url
139 self.run_build_command(143 self.run_build_command(
140 ["snapcraft"], path=os.path.join("/build", self.name))144 ["snapcraft"], path=os.path.join("/build", self.name), env=env)
141145
142 def revoke_token(self):146 def revoke_token(self):
143 """Revoke builder proxy token."""147 """Revoke builder proxy token."""
144148
=== modified file 'debian/changelog'
--- debian/changelog 2016-09-16 16:40:39 +0000
+++ debian/changelog 2016-11-04 10:53:13 +0000
@@ -1,3 +1,10 @@
1launchpad-buildd (141) UNRELEASED; urgency=medium
2
3 * buildsnap: Grant access to the proxy during the build phase as well as
4 during the pull phase.
5
6 -- Colin Watson <cjwatson@ubuntu.com> Fri, 04 Nov 2016 10:47:41 +0000
7
1launchpad-buildd (140) trusty; urgency=medium8launchpad-buildd (140) trusty; urgency=medium
29
3 * buildsnap: Catch urllib2.URLError as well as urllib2.HTTPError when10 * buildsnap: Catch urllib2.URLError as well as urllib2.HTTPError when

Subscribers

People subscribed via source and target branches

to all changes: