-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
fix(inputs.chrony): Support local (reference) sources #15551
Conversation
Thanks so much for the pull request! |
!signed-cla |
01cff75
to
5205e0a
Compare
5205e0a
to
b834e14
Compare
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.
Thanks @Frankkkkk! Just a small comment regarding the comment. ;-) Maybe we could also move the peer-extraction to a function to avoid having to comment the special cases in two places?
b834e14
to
8f47c6b
Compare
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.
Thanks for the quick update! Please move all source-name logic into the now function as this makes the code in gather more readable.
I don't understand what you mean by "source-name logic into the now function" as they are different:
Cheers |
@Frankkkkk I see, didn't notice that the type is different. So keep the code locations but remove the link to the other repository! |
8f47c6b
to
cc9e21c
Compare
Local sources are directly connected to the server (PPS inputs from oscillators, GPS, etc..). They can be seen as stratum-0 sources. The standard way of getting their name doesn't work as it is stored into their RefID instead. See the following links for more details: - https://gitlab.com/chrony/chrony/-/blob/05bd4898a9a64fe1e5432a78939c4b4b82619c2f/client.c?page=3#L2046 - https://gitlab.com/chrony/chrony/-/blob/05bd4898a9a64fe1e5432a78939c4b4b82619c2f/client.c?page=3#L2152 This fixes that Signed-off-by: Frank Villaro-Dixon <frank@villaro-dixon.eu>
cc9e21c
to
1a29326
Compare
@srebhan done |
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 👍 This pull request doesn't change the Telegraf binary size 📦 Click here to get additional PR build artifactsArtifact URLs |
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.
Thanks for your contribution @Frankkkkk!
(cherry picked from commit 5a46c0a)
Summary
Local sources are directly connected to the server (PPS inputs from oscillators, GPS, etc..). They can be seen as stratum-0 sources.
The standard way of getting their name doesn't work as it is stored into their RefID instead.
This fixes that
Checklist
Related issues
resolves #15550