Merge lp:~whitmo/charms/trusty/ntp/test-fix into lp:~brad-marshall/charms/trusty/ntp/add-auto-peers

Proposed by Whit Morriss
Status: Merged
Merged at revision: 37
Proposed branch: lp:~whitmo/charms/trusty/ntp/test-fix
Merge into: lp:~brad-marshall/charms/trusty/ntp/add-auto-peers
Diff against target: 17 lines (+2/-2)
1 file modified
tests/10-deploy-test.py (+2/-2)
To merge this branch: bzr merge lp:~whitmo/charms/trusty/ntp/test-fix
Reviewer Review Type Date Requested Status
Brad Marshall Pending
Review via email: mp+254802@code.launchpad.net

Description of the change

Fixes functional test failure caused by old variable name.

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
=== modified file 'tests/10-deploy-test.py'
--- tests/10-deploy-test.py 2015-03-27 06:24:40 +0000
+++ tests/10-deploy-test.py 2015-03-31 16:55:25 +0000
@@ -77,11 +77,11 @@
77 print(message)77 print(message)
78 amulet.raise_status(amulet.FAIL, msg=message)78 amulet.raise_status(amulet.FAIL, msg=message)
7979
80# The ubuntu cloud image does not have ntp installed by default, 80# The ubuntu cloud image does not have ntp installed by default,
81# and therefore does not have the /etc/ntp.conf file.81# and therefore does not have the /etc/ntp.conf file.
8282
83# Read in the ntp configuration file from the ubuntu unit.83# Read in the ntp configuration file from the ubuntu unit.
84configuration_file = ubuntu_unit.file_contents('/etc/ntp.conf')84configuration_file = ubuntu_unit_0.file_contents('/etc/ntp.conf')
85# This call will fail with an IO exception if the file does not exist.85# This call will fail with an IO exception if the file does not exist.
8686
87# Search for ntp server 0 in the config file, raise an exception if not found.87# Search for ntp server 0 in the config file, raise an exception if not found.

Subscribers

People subscribed via source and target branches

to all changes: