-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
timesync: make poll interval configurable #7268
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, but one nit on the documentation
man/timesyncd.conf.xml
Outdated
@@ -103,10 +103,20 @@ | |||
|
|||
<varlistentry> | |||
<term><varname>RootDistanceMaxSec=</varname></term> | |||
<listitem><para>Maximum acceptable root distance in seconds. | |||
<listitem><para>Maximum acceptable root distance. Takes time (in seconds). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe "Takes a time value (in seconds)"?
man/timesyncd.conf.xml
Outdated
<term><varname>PollIntervalMinSec=</varname></term> | ||
<term><varname>PollIntervalMaxSec=</varname></term> | ||
<listitem><para>The minimum and maximum poll intervals for NTP messages. | ||
Take time (in seconds). <varname>PollIntervalMinSec=</varname> must not be smaller than 16 seconds. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as above
This adds PollIntervalMinSec= and PollIntervalMaxSec= to timesyncd.conf Closes systemd#7262.
@poettering Thanks. Updated. |
lgtm. thanks! |
This adds
PollIntervalMinSec=
andPollIntervalMaxSec=
to timesyncd.confCloses #7262.
I proposed this in #7215. But at that time no one requests these options, then I dropped them. After that, in #7262 @conqp requests max and min poll intervals configurable. So, I resubmit this.
Note that the relevant options exist in both chrony.conf and ntp.conf (
minpoll
andmaxpoll
).