Merge ~hloeung/layer-snap:master into ~stub/layer-snap:master

Proposed by Haw Loeung
Status: Merged
Merged at revision: 9c69a33ea0586ac68a4e47d6b55b3a0374b96b26
Proposed branch: ~hloeung/layer-snap:master
Merge into: ~stub/layer-snap:master
Diff against target: 19 lines (+7/-1)
1 file modified
lib/charms/layer/snap.py (+7/-1)
Reviewer Review Type Date Requested Status
Stuart Bishop Pending
Review via email: mp+381417@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/lib/charms/layer/snap.py b/lib/charms/layer/snap.py
index f760d69..28af4e4 100644
--- a/lib/charms/layer/snap.py
+++ b/lib/charms/layer/snap.py
@@ -382,7 +382,13 @@ def _resource_get(snapname):
382def get_available_refreshes():382def get_available_refreshes():
383 '''Return a list of snaps which have refreshes available.383 '''Return a list of snaps which have refreshes available.
384 '''384 '''
385 out = subprocess.check_output(['snap', 'refresh', '--list']).decode('utf8')385 try:
386 out = subprocess.check_output(['snap', 'refresh', '--list']).decode('utf8')
387 except subprocess.CalledProcessError:
388 # If snap refresh fails for whatever reason, we should just return no
389 # refreshes available - LP:1869630.
390 return []
391
386 if out == 'All snaps up to date.':392 if out == 'All snaps up to date.':
387 return []393 return []
388 else:394 else:

Subscribers

People subscribed via source and target branches

to all changes: