Skip to content

net/chrony: Improve configuration options#5121

Open
Matthew-Otto wants to merge 13 commits intoopnsense:masterfrom
Matthew-Otto:chrony-config
Open

net/chrony: Improve configuration options#5121
Matthew-Otto wants to merge 13 commits intoopnsense:masterfrom
Matthew-Otto:chrony-config

Conversation

@Matthew-Otto
Copy link
Copy Markdown

The current chrony plugin provides insufficient options for configuration.

This PR adds support for the following source options on a per-source basis:

  • NTS
  • prefer
  • iburst
  • min/max poll
  • interleaving

It additionally adds support for pool-type sources, an NTP data diagnostics tab, and the local stratum feature as requested in #5115

@fichtner
Copy link
Copy Markdown
Member

fichtner commented Jan 6, 2026

What happened to fallbackpeers? It looks like you may need a migration script for the model and I want some of the reordering out of the current diff because it makes it harder than necessary.

@fichtner fichtner self-assigned this Jan 6, 2026
@Matthew-Otto
Copy link
Copy Markdown
Author

Matthew-Otto commented Jan 6, 2026

Fallbackpeers is unnecessary now. Simply add a source without NTS marked. This is equivalent to how it works currently (authselectmode mix is the default behavior of chrony).

A migration script seems necessary. I have no idea how to implement that though. Can you point me to any resources on how that should work in opnsense?

Can you elaborate on what you mean by reordering? I don't follow.

@HidingCherry
Copy link
Copy Markdown

I would rather suggest to have optionally a form whether to manually enter all servers (either line for line or a big form with full control).
Alternatively - add a include /usr/local/etc/chrony.d/*.conf to add a custom config file, though - the user would be required to ensure validity.

@Matthew-Otto
Copy link
Copy Markdown
Author

I don't have much time to work on this atm but if you're willing to implement that, I'd happily close this PR. I just want to see improved functionality of chrony in opnsense.

@HidingCherry
Copy link
Copy Markdown

I'll give it a try, though I've never worked on opnsense yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants