Merge pull request #91 from apenney/inherits

Remove variables, switch to inherits.

Merge pull request #92 from meltwater/keyfile_params

Update default keys file param variable to match init.pp's

Update default keys file param variable to match init.pp's

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.

Merge pull request #81 from apenney/20-release

2.0-rc1 release.

Add in parameters that may be effected.

Add `restrict`.

Prepare an rc release and add the CHANGELOG.

Merge pull request #88 from apenney/restrict

Add specs for restrict.

Add specs for restrict.

Merge pull request #87 from apenney/restrict

Convert restrict to an array of restrictions.

Convert restrict to an array of restrictions.

Instead of a boolean with hardcoded values we now just convert
restrict into an array of lines related to restrictions.

Merge pull request #80 from apenney/single_template

Merge all the templates together, add a few new features to help with the merge.

Rewrite the spec testing.

A lot of changes here, primarily to unify the spec testing in a single
spec file to reflect the changes to the private ntp::config and
::install classes.  As a side effect make sure we properly test each
supported distribution (including Gentoo properly) and prefer to use
'content' => /blah/ rather than other methods of testing file contents.

Convert some classes to not be parameterized.

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.

Add documentation for new parameters.

Add preferred_servers feature.

This adds:

`preferred_servers` [Array]: The servers to prefer.

As requested by Erik Dalén!

Add tests for the new keys options.

Remove all the distribution specific templates.

Add a single ntp.conf.erb as the standard template going forward,
to ease the maintenance burden of adding features to this module.

Add new parameters to allow more flexible configuration.

`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.

Merge pull request #79 from 3flex/fix-bug-number-typo

correct anchor pattern bug number

correct anchor pattern bug number

Merge pull request #75 from apenney/101-release

Prepare a quick 1.0.1 release to fix a quoting issue.

Prepare a quick 1.0.1 release to fix a quoting issue.

Merge pull request #73 from gwis/update-deprecation-notice

Correct deprecation notice to mention 'package_ensure'

Merge pull request #72 from mmoll/virtual_panic

fix panic parameter in virtual machines

Correct deprecation notice to mention 'package_ensure'

fix panic parameter in virtual machines

As facts are always strings, quoting is required here.

Merge pull request #71 from apenney/100-release

Prepare the 1.0.0 release.

Prepare the 1.0.0 release.

Merge pull request #69 from simonsd/master

Archlinux now also has an osfamily fact

fixup spec tests for Archlinux

Signed-off-by: Dave Simons <dave@inuits.eu>

Merge pull request #70 from nanliu/cleanup

Clean up README to use puppet highlight.

Clean up README to use puppet highlight.

Merge pull request #68 from apenney/rc-fixes

Formatting fixes for the rc

Use scope.lookupvar for this to look in the right place.

Fix extra blank line.

Fix examples to use single quotes.

Remove placeholder file.

Ensure the spacing between class and ( is consistent.

Archlinux now also has osfamily fact

Signed-off-by: Dave Simons <dave@inuits.eu>

Merge pull request #67 from apenney/rc

Fix version.

Fix the travis secure field.

Fix version.

Merge pull request #66 from apenney/refactor-round-1

WIP: Refactoring of NTP module

Bump to rc candidate.

Add panic as a new parameter in case people want to allow clock skews.

Fix up changelog.

Fix up travis for more testing.

Add service checks.

Improve the rspec-system tests, making sure we test for the package
being installed and the config file at least makes basic sense.

Add some basic tests.

Add rspec-system framework.

Rewrite the README and prepare the CHANGELOG.

Rename parameters, remove outdated documentation, and adds inherit to
ntp on the child classes.

Rewrite the spec tests to cover the 4 different classes and improve a
few missing gaps in the testing.

Ensure the rename is done everywhere and wrap the service in a
manage_service param.

Rename package_ensure to ensure_package.
Add the anchor pattern.
Move the deprecation warning out of params to init.

Add Gentoo support. (And reformat archlinux vars)

Fix specs now things are renamed.

Some syntax fixes as well as deprecate autoupdate.

Ensure that users can pass in templates from outside of the ntp module.

Rework specs to pass with renamed params.

Rename all the servers_real functions to servers.

Add ntp::service, a class to manage the ntp service.

Refactor out the main ntp class to break it up into 4 smaller ones.

Add ntp::params, a place to centralize all configuration information.

Add ntp::install, a class to manage the ntp packages.

Add ntp::config, a class to manage the ntp configuration file.

Merge pull request #62 from c00w/master

Fix NTP templates to use @

User @servers_real instead of servers_real to avoid warnings

Merge pull request #54 from puppetlabs/spec_test/pull_49

Add test for virtual machines and local clocks

Merge pull request #49 from alexjfisher/feature/master/disable_local_clock_for_vms

Omit configuration for a local clock as time source if running on a VM.

Add test for virtual machines and local clocks

This commit provides a spec test for pull request #49 and commit
01273c53f1cf2015c3ff5a9f704828225d900e03 which specifies that virtual
machines should not use the local clock. This test attempts to protect
against a regression on that statement.

Merge pull request #53 from puppetlabs/release_0.3.0

Release version 0.3.0

Release version 0.3.0

Release 0.3.0 includes fixes to style, a couple bug fixes and adds
support for Arch Linux by way of Matt Rogers.

Merge pull request #52 from puppetlabs/travis_update

Update Travis.YML

Update Travis.YML

This commit updates to a more modern test matrix and adds the
configuration necessary to publish in an automated fashion.

Merge pull request #47 from frimik/feature-19418-ntp-template

ntp: fixes #19418 - allow template override

fixes #19418 - missing fixtures for spec tests

Omit configuration for a local clock as time source if running on a VM.

Quoting http://kb.vmware.com/selfservice/microsites/search.do?language=en_US&cmd=displayKC&externalId=1006427
'It is also important not to use the local clock as a time source, often referred to as the Undisciplined Local Clock.'

Merge branch 'mattr--feature/add-archlinux-support'

* mattr--feature/add-archlinux-support:
  Add support for Arch Linux

closes #40

Add support for Arch Linux

Add test for package name and presence

Fix bad package name for Arch Linux

Fix missing comment in config template

Merge pull request #48 from richardc/patch-1

Restrict the versions and add 3.1

Restrict the versions and add 3.1

~> 2.6 will match 2.7

ntp: fixes #19418 - allow template override

This allows a template to be explicitly set.

Merge pull request #44 from puppetlabs/readme_update

Update Documentation and Purge Extras

Update Documentation and Purge Extras

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.

Merge pull request #42 from blkperl/update_travis

Update travis and gemfile to puppetlabs standard

Add build status to the README

Update travis and gemfile to puppetlabs standard

Merge pull request #39 from mmoll/fix_cp_error

fix copy/paste error in spec file

fix copy/paste error in spec file

Merge pull request #38 from puppetlabs/contributing_guide

Add CONTRIBUTING.md

Add CONTRIBUTING.md

This commit adds a brief contributing guide that really links back
to the wiki page hosted on projects.puppetlabs.com. This file was
added mainly for the GitHub functionality described in this blog
post: https://github.com/blog/1184-contributing-guidelines

Merge pull request #37 from puppetlabs/doh/nrosier/patch-1

Update package-format for support of FreeBSD 9.x

Update package-format for support of FreeBSD 9.x

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.

Merge pull request #29 from mmoll/osfamily

switch to the 'osfamily' fact

Merge pull request #33 from codingtony/master

Make service ntp started on boot -> service 'ntp' { enable=>$enable...

Merge remote-tracking branch 'origin_master/master'

Conflicts:
manifests/init.pp