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 new chrony serverstats counters to exposed serverstats metrics #87

Merged

Conversation

raphaelthomas
Copy link
Contributor

@raphaelthomas raphaelthomas commented Jul 1, 2024

Exposes the additional NTP packet timestamping counter serverstats
metrics introduced in chronyd version 4.4.

@raphaelthomas
Copy link
Contributor Author

Not overly proficient in Go, feedback is much appreciated.

collector/serverstats.go Outdated Show resolved Hide resolved
@raphaelthomas raphaelthomas marked this pull request as draft July 27, 2024 18:34
@SuperQ
Copy link
Owner

SuperQ commented Oct 9, 2024

I've created a minimal fix PR: #96.

If you want to work on adding additional stats, that'd be great.

@raphaelthomas
Copy link
Contributor Author

I've created a minimal fix PR: #96.

If you want to work on adding additional stats, that'd be great.

Wonderful, thanks a lot and apologies for the long silence 🙏🏼 I'll gladly add the missing stats then once #96 is merged.

@raphaelthomas raphaelthomas force-pushed the raphaelthomas/fix-serverstats-chrony-4.5 branch 2 times, most recently from 10d144c to de46b26 Compare October 16, 2024 19:34
@raphaelthomas raphaelthomas marked this pull request as ready for review October 18, 2024 08:25
@SuperQ
Copy link
Owner

SuperQ commented Oct 23, 2024

This needs a DCO sign-off. You can use git commit -s --amend to add it.

collector/serverstats.go Show resolved Hide resolved
@SuperQ SuperQ mentioned this pull request Oct 23, 2024
@raphaelthomas raphaelthomas force-pushed the raphaelthomas/fix-serverstats-chrony-4.5 branch from de46b26 to f3bcd1a Compare October 27, 2024 16:22
@raphaelthomas raphaelthomas requested a review from SuperQ October 27, 2024 16:30
collector/serverstats.go Outdated Show resolved Hide resolved
Exposes the additional NTP packet timestamping counter serverstats
metrics introduced in `chronyd` version `4.4`.

Signed-off-by: Raphael Seebacher <[email protected]>
@raphaelthomas raphaelthomas force-pushed the raphaelthomas/fix-serverstats-chrony-4.5 branch from f3bcd1a to c34ec8f Compare October 27, 2024 18:19
@raphaelthomas raphaelthomas requested a review from SuperQ October 27, 2024 18:20
Copy link
Owner

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

@SuperQ SuperQ merged commit e90bda9 into SuperQ:main Oct 27, 2024
2 checks passed
SuperQ added a commit that referenced this pull request Oct 27, 2024
* [CHANGE] Update logging to slog #95
* [CHANGE] Only expose serverstats for correct veersions #100
* [ENHANCEMENT] Add support for Chrony >= 4.5 server stats #87 #96

Signed-off-by: SuperQ <[email protected]>
@SuperQ SuperQ mentioned this pull request Oct 27, 2024
SuperQ added a commit that referenced this pull request Oct 27, 2024
* [CHANGE] Update logging to slog #95
* [CHANGE] Only expose serverstats for correct veersions #100
* [ENHANCEMENT] Add support for Chrony >= 4.5 server stats #87 #96

Signed-off-by: SuperQ <[email protected]>
SuperQ added a commit that referenced this pull request Oct 27, 2024
* [CHANGE] Update logging to slog #95
* [CHANGE] Only expose serverstats for correct veersions #100
* [ENHANCEMENT] Add support for Chrony >= 4.5 server stats #87 #96

Signed-off-by: SuperQ <[email protected]>
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.

2 participants