Skip to content
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.

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

use RHEL conf for chrony #343

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Conversation

tomkuba
Copy link
Contributor

@tomkuba tomkuba commented Feb 7, 2024

Overall Review of Changes:
replace debian chrony.conf by RHEL8 default conf.

Issue Fixes:
#227

Enhancements:
N/A

How has this been tested?:
N/A

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congrats on opening your first pull request and thank you for taking the time to help improve Ansible-Lockdown!
Please join in the conversation happening on the Discord Server as well.

Signed-off-by: Tomáš Kuba <tom.kuba@gmail.com>
# Note that if Chrony tries to go "online" and dns lookup of the servers
# fails they will be discarded. Thus under some circumstances it is
# better to use IP numbers than host names.
## This file is managed by Ansible, YOUR CHANGES WILL BE LOST!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not use conventional {{ ansible_managed | comment }}? The user can redefine whatever the text should be.

Signed-off-by: Tomáš Kuba <tom.kuba@gmail.com>
@uk-bolly uk-bolly merged commit 74dc606 into ansible-lockdown:devel Feb 19, 2024
4 checks passed
@uk-bolly
Copy link
Member

Great update, thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants