openstack/tripleo-heat-templates
Code Issues Proposed changes

Merge "Specify multiple NtpServers by default"

Browse Source
This commit is contained in:
Zuul 2018-12-06 22:14:21 +00:00 committed by Gerrit Code Review
commit 7e754dcf14
8 changed files with 26 additions and 16 deletions

6
deployment/timesync/chrony-ansible.yaml

@ -34,9 +34,9 @@ parameters:
via parameter_defaults in the resource registry.
type: json
NtpServer:
default: ['pool.ntp.org']
description: NTP servers list. Defaulted to pool.ntp.org in order to
have a sane default for Pacemaker deployments when
default: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']
description: NTP servers list. Defaulted to a set of pool.ntp.org servers
in order to have a sane default for Pacemaker deployments when
not configuring this parameter by default.
type: comma_delimited_list
NtpPool:

4
environments/composable-roles/monolithic-ha.yaml

@ -41,9 +41,9 @@ parameter_defaults:
# Type: comma_delimited_list
DnsServers: []
# NTP servers list. Defaulted to pool.ntp.org in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# NTP servers list. Defaulted to a set of pool.ntp.org servers in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# Type: comma_delimited_list
NtpServer: ['pool.ntp.org']
NtpServer: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']
# Name of the flavor for Ceph nodes
# Type: string

4
environments/composable-roles/monolithic-nonha.yaml

@ -41,9 +41,9 @@ parameter_defaults:
# Type: comma_delimited_list
DnsServers: []
# NTP servers list. Defaulted to pool.ntp.org in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# NTP servers list. Defaulted to a set of pool.ntp.org servers in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# Type: comma_delimited_list
NtpServer: ['pool.ntp.org']
NtpServer: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']
# Name of the flavor for Ceph nodes
# Type: string

4
environments/composable-roles/standalone.yaml

@ -54,9 +54,9 @@ parameter_defaults:
# Type: number
NetworkerCount: 2
# NTP servers list. Defaulted to pool.ntp.org in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# NTP servers list. Defaulted to a set of pool.ntp.org servers in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# Type: comma_delimited_list
NtpServer: ['pool.ntp.org']
NtpServer: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']
# Name of the flavor for Ceph nodes
# Type: string

4
environments/standalone/standalone-overcloud.yaml

@ -26,9 +26,9 @@ parameter_defaults:
# Type: comma_delimited_list
DnsServers: []
# NTP servers list. Defaulted to pool.ntp.org in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# NTP servers list. Defaulted to a set of pool.ntp.org servers in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# Type: comma_delimited_list
NtpServer: ['pool.ntp.org']
NtpServer: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']
# Name of the flavor for Standalone nodes
# Type: string

4
environments/standalone/standalone-tripleo.yaml

@ -36,9 +36,9 @@ parameter_defaults:
# Type: boolean
EnablePackageInstall: True
# NTP servers list. Defaulted to pool.ntp.org in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# NTP servers list. Defaulted to a set of pool.ntp.org servers in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
# Type: comma_delimited_list
NtpServer: ['pool.ntp.org']
NtpServer: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']
# How the server should receive the metadata required for software configuration.
# Type: string

6
puppet/services/time/ntp.yaml

@ -34,9 +34,9 @@ parameters:
via parameter_defaults in the resource registry.
type: json
NtpServer:
default: ['pool.ntp.org']
description: NTP servers list. Defaulted to pool.ntp.org in order to
have a sane default for Pacemaker deployments when
default: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']
description: NTP servers list. Defaulted to a set of pool.ntp.org servers
in order to have a sane default for Pacemaker deployments when
not configuring this parameter by default.
type: comma_delimited_list
NtpIburstEnable:

10
releasenotes/notes/change-ntp-default-servers-5c4cb51c10ffe60e.yaml Normal file

@ -0,0 +1,10 @@
---
fixes:
- |
The NtpServer default set now includes multiple pool.ntp.org hosts to ensure
that the time can be properly synced during the deployment. Having only a
single timesource can lead to deployment failures if the time source is
unavailable during the deployment. It is recommended that you either
set multiple NtpServers or use the NtpPool configuration to ensure that
enough time sources are available for the hosts. Note that the NtpPool
configuration is only available when using chrony. See LP#1806521