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

Add maxdistance parameter to chrony #134

Merged
merged 6 commits into from
Dec 16, 2024

Conversation

davidphay
Copy link
Contributor

@davidphay davidphay commented Apr 10, 2024

This pull request allow users to edit maxdistance param in chrony config.

It usefull to fix the issue when the distance with the source is greater than the default value (3)

@davidphay
Copy link
Contributor Author

davidphay commented May 14, 2024

Hi @geerlingguy is there something I can do to get this PR merged?

defaults/main.yml Outdated Show resolved Hide resolved
@davidphay
Copy link
Contributor Author

@geerlingguy It should be fine now.

I update the readme & the param name

@davidphay
Copy link
Contributor Author

@geerlingguy I fixed the yaml lint syntax!

@davidphay
Copy link
Contributor Author

@geerlingguy What's the issue ?

@geerlingguy
Copy link
Owner

Looks like build failures due to the docker python library

@davidphay
Copy link
Contributor Author

Hi @geerlingguy, do you plan to fix this issue ?

Thanks!

Copy link

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label Dec 14, 2024
README.md Outdated
@@ -66,6 +66,12 @@ Whether to restart the cron daemon after the timezone has changed.

Enable tinker panic, which is useful when running NTP in a VM.

ntp_tinker_panic: true
Copy link
Owner

Choose a reason for hiding this comment

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

The ordering of these descriptions and examples is off, the ntp_chrony_maxdistance should be up here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can you check now, it shoudl be good, Thanks !

@geerlingguy geerlingguy merged commit b1ffa0d into geerlingguy:master Dec 16, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants