Age | Commit message (Collapse) | Author |
|
Remove variables, switch to inherits.
|
|
|
|
This commit removes all the local copies of variables in the subclasses
and simply inherits the main class in order to bring variables into the
right scope.
|
|
Instead of a boolean with hardcoded values we now just convert
restrict into an array of lines related to restrictions.
|
|
We're following (where possible) the pattern of having parameters only
exist on interface classes. As ntp::config and ntp::install are just
internal, private, classes, these don't make sense to allow changes to.
|
|
This adds:
`preferred_servers` [Array]: The servers to prefer.
As requested by Erik Dalén!
|
|
`driftfile` [String]: Allows the setting of the driftfile location.
`keys_enable` [Boolean]: Should the ntp keys option be enabled?
`keys_file` [String]: Where are the keys stored?
`keys_trusted` [Array]: List of keys to trust.
`keys_requestkey` [String]: Request key to use.
`keys_controlkey` [String]: Control key to use.
If keys_enable is set then the directory that keysfile is stored in is
created (recursively). Currently we don't manage the keys file but that
is an option to be added.
|
|
|
|
Correct deprecation notice to mention 'package_ensure'
|
|
|
|
As facts are always strings, quoting is required here.
|
|
Archlinux now also has an osfamily fact
|
|
|
|
|
|
Signed-off-by: Dave Simons <dave@inuits.eu>
|
|
|
|
ntp on the child classes.
|
|
manage_service param.
|
|
Add the anchor pattern.
Move the deprecation warning out of params to init.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
ntp: fixes #19418 - allow template override
|
|
Add test for package name and presence
Fix bad package name for Arch Linux
Fix missing comment in config template
|
|
This allows a template to be explicitly set.
|
|
Commit message from laurenrother
Before alterations, this content was the module author-determined
description of and instructions for use of the module.
As part of a joint Forge/Docs team effort to standardize formatting and
encourage quality module documentation, a best practices README
template was created via internal and external user testing. That
template was then applied to this module.
I pulled in content from the original README on GitHub as well as the
Forge Module Description. Standard headings were added (Overview,
Module Description, Setup, Usage,
Implementation, etc.) to organize content, existent content was moved
under its appropriate heading and edited for tone/flow/clarity, and
basic formatting was done to
adhere to template standards.
Extraneous, empty READMEs were deleted.
|
|
This commit comes to you via nrosier but I created a merge conflict
with my latest merge and already bothered him with a rebase once.
|
|
switch to the 'osfamily' fact
|
|
|
|
NTP servers should not restrict themselves to 127.0.0.1, and this commit
adds a `restrict` parameter to enable/disable this configuration block
in the relevent templates.
|
|
to start on boot.
|
|
|
|
|
|
Puppet-lint was complaining about "ensure found on line but it's not the
first attribute on line 109". Trivial fix (swapped the ensure and the
name).
Also, the README didn't mention Debian or CentOS, but the comment block
in init.pp did. I added both to the README. I happen to be using this
with Debian unstable, but I haven't tested it extensively enough to say
"yep, this works". It, however, works for me.
|
|
|
|
|
|
Add restrict parameter
|
|
NTP servers should not restrict themselves to 127.0.0.1, and this commit
adds a `restrict` parameter to enable/disable this configuration block
in the relevent templates.
|
|
|
|
|
|
Puppet-lint was complaining about "ensure found on line but it's not the
first attribute on line 109". Trivial fix (swapped the ensure and the
name).
Also, the README didn't mention Debian or CentOS, but the comment block
in init.pp did. I added both to the README. I happen to be using this
with Debian unstable, but I haven't tested it extensively enough to say
"yep, this works". It, however, works for me.
|
|
|
|
|
|
to reflect Puppet Labs best practices
|
|
|