New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Check for `pool` or `server` in chrony.conf #97

Closed

nvwls wants to merge 1 commit into dev-sec:master from nvwls:chrony-pool

Contributor

nvwls commented Dec 23, 2020

The chrony cookbook uses pool https://github.com/chef-cookbooks/chrony/blob/master/templates/chrony_client.conf.erb#L8

pool is also referenced in
https://chrony.tuxfamily.org/faq.html#_what_is_the_minimum_recommended_configuration_for_an_ntp_client


          Check for pool or server in chrony.conf

e2c74f8

The chrony cookbook uses `pool` https://github.com/chef-cookbooks/chrony/blob/master/templates/chrony_client.conf.erb#L8

`pool` is also referenced in
https://chrony.tuxfamily.org/faq.html#_what_is_the_minimum_recommended_configuration_for_an_ntp_client

Signed-off-by: joe.nuspl <joe.nuspl@workday.com>

deric4 mentioned this pull request

Travis CI status check failing #98

Closed

Member

deric4 commented Jan 5, 2021 •

edited

Loading

Nice! The audit command in the Benchmark also checks for pool in the configuration file:

# grep -E "^(server|pool)" /etc/chrony.conf
server <remote-server>

@nvwls I think if you update with a rebase from master, the Travis check should pass (see #99 )

nvwls closed this

nvwls deleted the chrony-pool branch

January 5, 2021 21:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet