Checkbox

Merge lp:~roadmr/checkbox/1306763-virt-exceptions into lp:checkbox

  1. 1306763-virt-exceptions
  2. Merge into trunk
Proposed by Daniel Manrique
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 2919
Merged at revision: 2919
Proposed branch: lp:~roadmr/checkbox/1306763-virt-exceptions
Merge into: lp:checkbox
Diff against target: 14 lines (+4/-1)
1 file modified
providers/plainbox-provider-checkbox/bin/virtualization (+4/-1)
To merge this branch: bzr merge lp:~roadmr/checkbox/1306763-virt-exceptions
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+215502@code.launchpad.net

Commit message

providers:checkbox: catch URLError in virtualization script.

Description of the change

providers:checkbox: catch URLError in virtualization script.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-checkbox/bin/virtualization'
2--- providers/plainbox-provider-checkbox/bin/virtualization 2014-03-31 19:44:35 +0000
3+++ providers/plainbox-provider-checkbox/bin/virtualization 2014-04-11 20:42:20 +0000
4@@ -82,7 +82,10 @@
5 # Attempt download
6 try:
7 resp = urllib.request.urlretrieve(image_url, cloud_iso)
8- except (IOError, OSError, urllib.error.HTTPError) as exception:
9+ except (IOError,
10+ OSError,
11+ urllib.error.HTTPError,
12+ urllib.error.URLError) as exception:
13 logging.error("Failed download of image from %s: %s", image_url, exception)
14 return False
15

Subscribers

People subscribed via source and target branches