Merge lp:~adam-collard/ols-vms/lxd-3.0 into lp:ols-vms

Proposed by Adam Collard
Status: Merged
Approved by: Adam Collard
Approved revision: 306
Merge reported by: Otto Co-Pilot
Merged at revision: 303
Proposed branch: lp:~adam-collard/ols-vms/lxd-3.0
Merge into: lp:ols-vms
Diff against target: 76 lines (+20/-7)
5 files modified
NEWS.rst (+8/-2)
debian/changelog (+6/-0)
olsvms/__init__.py (+1/-1)
olsvms/tests/test_config.py (+3/-3)
olsvms/vms/lxd.py (+2/-1)
To merge this branch: bzr merge lp:~adam-collard/ols-vms/lxd-3.0
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+352954@code.launchpad.net

Commit message

Fix error message matcher for LXD 3.0

Fix bit-rotted test

Release 1.2.5

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'NEWS.rst'
--- NEWS.rst 2017-03-16 17:56:50 +0000
+++ NEWS.rst 2018-08-13 08:28:43 +0000
@@ -4,8 +4,14 @@
44
5Overview of changes to ols-vms in reverse chronological order.5Overview of changes to ols-vms in reverse chronological order.
66
7dev71.2.5
8===8=====
9
10* Fix bug with LXD 3.0 in 'vms.lxd.lxd_info'.
11
12
131.2.4
14=====
915
10* Fix setup failure when running on a host where bzr is not installed.16* Fix setup failure when running on a host where bzr is not installed.
1117
1218
=== modified file 'debian/changelog'
--- debian/changelog 2017-03-16 17:57:55 +0000
+++ debian/changelog 2018-08-13 08:28:43 +0000
@@ -1,3 +1,9 @@
1ols-vms (1.2.5) unstable; urgency=medium
2
3 * Fix bug with LXD 3.0 in vms.lxd.lxd_info.
4
5 -- Adam Collard <adam.collard@canonical.com> Mon, 13 Aug 2018 09:01:37 +0100
6
1ols-vms (1.2.4) unstable; urgency=medium7ols-vms (1.2.4) unstable; urgency=medium
28
3 * Fix setup failure when running on a host where bzr is not installed.9 * Fix setup failure when running on a host where bzr is not installed.
410
=== modified file 'olsvms/__init__.py'
--- olsvms/__init__.py 2017-03-16 17:57:55 +0000
+++ olsvms/__init__.py 2018-08-13 08:28:43 +0000
@@ -19,4 +19,4 @@
19# the release level is 'dev' or 'final'. The19# the release level is 'dev' or 'final'. The
20# version_info value corresponding to the olsvms version 2.0 is (2, 0, 0,20# version_info value corresponding to the olsvms version 2.0 is (2, 0, 0,
21# 'final', 0).21# 'final', 0).
22__version__ = (1, 2, 4, 'final', 0)22__version__ = (1, 2, 5, 'final', 0)
2323
=== modified file 'olsvms/tests/test_config.py'
--- olsvms/tests/test_config.py 2017-03-15 16:25:51 +0000
+++ olsvms/tests/test_config.py 2018-08-13 08:28:43 +0000
@@ -139,10 +139,10 @@
139139
140 def test_lxd_invalid_nesting(self):140 def test_lxd_invalid_nesting(self):
141 self.conf.set('lxd.nesting', 'true')141 self.conf.set('lxd.nesting', 'true')
142 with self.assertRaises(errors.OptionValueError) as cm:142 with self.assertRaisesRegexp(
143 errors.OptionValueError,
144 'lxd.nesting: Value "true" is not valid.'):
143 self.conf.get('lxd.nesting')145 self.conf.get('lxd.nesting')
144 self.assertEqual('lxd.nesting: Value "true" is not valid.',
145 str(cm.exception))
146146
147147
148class TestPathOption(unittest.TestCase):148class TestPathOption(unittest.TestCase):
149149
=== modified file 'olsvms/vms/lxd.py'
--- olsvms/vms/lxd.py 2017-02-13 16:43:25 +0000
+++ olsvms/vms/lxd.py 2018-08-13 08:28:43 +0000
@@ -87,7 +87,8 @@
87 try:87 try:
88 _, source, err = subprocesses.run(info_command)88 _, source, err = subprocesses.run(info_command)
89 except errors.CommandError as e:89 except errors.CommandError as e:
90 if e.err == 'error: not found\n':90 if e.err in ('error: not found\n',
91 'Error: not found\n'):
91 return info92 return info
92 else:93 else:
93 raise94 raise

Subscribers

People subscribed via source and target branches