[PR #168] [MERGED] Add all servers to non-numbered zones #205

Closed
opened 2026-02-28 14:16:40 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/abh/ntppool/pull/168
Author: @mlichvar
Created: 4/10/2019
Status: Merged
Merged: 4/10/2019
Merged by: @abh

Base: masterHead: duplicate-sntp


📝 Commits (1)

  • 87c3a7f Add all servers to non-numbered zones

📊 Changes

1 file changed (+8 additions, -1 deletions)

View changed files

📝 lib/NP/Model/DnsRoot.pm (+8 -1)

📄 Description

Instead of splitting servers between non-numbered SNTP and numbered NTP
zones, always duplicate all servers in the SNTP zones. This should
stabilize the rate of requests that servers get as the SNTP zones will
have more servers and they will not be switching between SNTP and NTP
zone.

I'm not sure how much it would actually stabilize the request rate as I don't have my own pool to test it :). It would be an interesting experiment.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/abh/ntppool/pull/168 **Author:** [@mlichvar](https://github.com/mlichvar) **Created:** 4/10/2019 **Status:** ✅ Merged **Merged:** 4/10/2019 **Merged by:** [@abh](https://github.com/abh) **Base:** `master` ← **Head:** `duplicate-sntp` --- ### 📝 Commits (1) - [`87c3a7f`](https://github.com/abh/ntppool/commit/87c3a7ff4cbb8332346dc859c2317aa4a2cfcd57) Add all servers to non-numbered zones ### 📊 Changes **1 file changed** (+8 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `lib/NP/Model/DnsRoot.pm` (+8 -1) </details> ### 📄 Description Instead of splitting servers between non-numbered SNTP and numbered NTP zones, always duplicate all servers in the SNTP zones. This should stabilize the rate of requests that servers get as the SNTP zones will have more servers and they will not be switching between SNTP and NTP zone. I'm not sure how much it would actually stabilize the request rate as I don't have my own pool to test it :). It would be an interesting experiment. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 14:16:40 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ntppool#205
No description provided.