System calls any ntp provider which has been installed.

[jorhett]

trafficstars

The system module currently requires a very minimal ntp provider. There are many, many other ntp providers which have significantly more features. Please make your module optional for this, not required.

Furthermore, the ntp.pp manifest invokes any top-level ntp class, which can lead to confusing and hilarious results.

If you really want to include ntp, why not remove the dependency from Moduleinfo and just require that any top-level ntp module be installed and pass all options through to it? You're pretty much there already. You could just blindly pass all parameters through, and thus allow the user to use any ntp module they want --yours or someone else's.

[kennedye]

It also appears ntp does not respect the system::schedule::ntp: 'never' option as even when this is set hosts still receive the default pool entries.

[bastelfreak]

Hi, based on a discussion in https://groups.io/g/voxpupuli/message/449 we decided to archive this repository. I'm going to close all issues and PRs. If you're interested in maintaining the module, please respond to our mailinglist.