Merge lp:~julian-edwards/maas/remove-hardcoded-ip into lp:~maas-committers/maas/trunk

Proposed by Julian Edwards
Status: Merged
Approved by: Julian Edwards
Approved revision: no longer in the source branch.
Merged at revision: 2501
Proposed branch: lp:~julian-edwards/maas/remove-hardcoded-ip
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 25 lines (+2/-2)
2 files modified
src/maasserver/forms_settings.py (+1/-1)
src/maasserver/models/config.py (+1/-1)
To merge this branch: bzr merge lp:~julian-edwards/maas/remove-hardcoded-ip
Reviewer Review Type Date Requested Status
Julian Edwards (community) Approve
Review via email: mp+225432@code.launchpad.net

Commit message

Use the domain and not hard-coded IP for default NTP server

To post a comment you must log in.
Revision history for this message
Julian Edwards (julian-edwards) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/maasserver/forms_settings.py'
2--- src/maasserver/forms_settings.py 2014-06-27 20:02:49 +0000
3+++ src/maasserver/forms_settings.py 2014-07-03 07:02:22 +0000
4@@ -173,7 +173,7 @@
5 'required': False,
6 'help_text': (
7 "NTP server address passed to nodes via a DHCP response. "
8- "e.g. for ntp.ubuntu.com: '91.189.94.4'")
9+ "e.g. ntp.ubuntu.com")
10 }
11 },
12 'default_osystem': {
13
14=== modified file 'src/maasserver/models/config.py'
15--- src/maasserver/models/config.py 2014-06-20 13:38:39 +0000
16+++ src/maasserver/models/config.py 2014-07-03 07:02:22 +0000
17@@ -53,7 +53,7 @@
18 'default_distro_series': DEFAULT_OS.get_default_release(),
19 'http_proxy': None,
20 'upstream_dns': None,
21- 'ntp_server': '91.189.94.4', # ntp.ubuntu.com
22+ 'ntp_server': 'ntp.ubuntu.com',
23 # RPC configuration.
24 'rpc_region_certificate': None,
25 # Third Party