Merge lp:~harlowja/cloud-init/encode-resolve-fix into lp:~cloud-init-dev/cloud-init/trunk

Proposed by Joshua Harlow
Status: Merged
Merged at revision: 1201
Proposed branch: lp:~harlowja/cloud-init/encode-resolve-fix
Merge into: lp:~cloud-init-dev/cloud-init/trunk
Diff against target: 33 lines (+7/-1)
2 files modified
cloudinit/distros/rhel_util.py (+1/-1)
tests/unittests/test_distros/test_resolv.py (+6/-0)
To merge this branch: bzr merge lp:~harlowja/cloud-init/encode-resolve-fix
Reviewer Review Type Date Requested Status
Scott Moser Approve
Review via email: mp+266490@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'cloudinit/distros/rhel_util.py'
--- cloudinit/distros/rhel_util.py 2015-01-27 19:24:22 +0000
+++ cloudinit/distros/rhel_util.py 2015-07-30 22:20:35 +0000
@@ -86,4 +86,4 @@
86 r_conf.add_search_domain(s)86 r_conf.add_search_domain(s)
87 except ValueError:87 except ValueError:
88 util.logexc(LOG, "Failed at adding search domain %s", s)88 util.logexc(LOG, "Failed at adding search domain %s", s)
89 util.write_file(fn, r_conf, 0o644)89 util.write_file(fn, str(r_conf), 0o644)
9090
=== modified file 'tests/unittests/test_distros/test_resolv.py'
--- tests/unittests/test_distros/test_resolv.py 2015-01-27 01:02:31 +0000
+++ tests/unittests/test_distros/test_resolv.py 2015-07-30 22:20:35 +0000
@@ -1,6 +1,8 @@
1from cloudinit.distros.parsers import resolv_conf1from cloudinit.distros.parsers import resolv_conf
2from cloudinit.distros import rhel_util
23
3import re4import re
5import tempfile
4from ..helpers import TestCase6from ..helpers import TestCase
57
68
@@ -19,6 +21,10 @@
19 rp_r = str(rp).strip()21 rp_r = str(rp).strip()
20 self.assertEquals(BASE_RESOLVE, rp_r)22 self.assertEquals(BASE_RESOLVE, rp_r)
2123
24 def test_write_works(self):
25 with tempfile.NamedTemporaryFile() as fh:
26 rhel_util.update_resolve_conf_file(fh.name, [], [])
27
22 def test_local_domain(self):28 def test_local_domain(self):
23 rp = resolv_conf.ResolvConf(BASE_RESOLVE)29 rp = resolv_conf.ResolvConf(BASE_RESOLVE)
24 self.assertEquals(None, rp.local_domain)30 self.assertEquals(None, rp.local_domain)