openstack
/
tripleo-heat-templates
Code Issues Proposed changes

Specify multiple NtpServers by default 

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.

Change-Id: I5b82d77cbf0f2e8c2a59645a72aa533d7d2c86b8
Closes-Bug: #1806521
This commit is contained in:
Alex Schultz 2018-12-03 20:20:49 -07:00
parent b01b1a7f72
commit 90d3723175
8 changed files with 26 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
deployment/timesync/chrony-ansible.yaml
View File

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

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

@ -41,9 +41,9 @@ parameter_defaults:
# Type: comma_delimited_list # Type: comma_delimited_list
DnsServers: [] 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 # 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 # Name of the flavor for Ceph nodes
# Type: string # Type: string

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

@ -41,9 +41,9 @@ parameter_defaults:
# Type: comma_delimited_list # Type: comma_delimited_list
DnsServers: [] 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 # 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 # Name of the flavor for Ceph nodes
# Type: string # Type: string

4
environments/composable-roles/standalone.yaml
View File

@ -54,9 +54,9 @@ parameter_defaults:
# Type: number # Type: number
NetworkerCount: 2 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 # 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 # Name of the flavor for Ceph nodes
# Type: string # Type: string

4
environments/standalone/standalone-overcloud.yaml
View File

@ -26,9 +26,9 @@ parameter_defaults:
# Type: comma_delimited_list # Type: comma_delimited_list
DnsServers: [] 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 # 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 # Name of the flavor for Standalone nodes
# Type: string # Type: string

4
environments/standalone/standalone-tripleo.yaml
View File

@ -36,9 +36,9 @@ parameter_defaults:
# Type: boolean # Type: boolean
EnablePackageInstall: True 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 # 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. # How the server should receive the metadata required for software configuration.
# Type: string # Type: string

6
puppet/services/time/ntp.yaml
View File

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

10
releasenotes/notes/change-ntp-default-servers-5c4cb51c10ffe60e.yaml Normal file
View 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