Skip to content

Add support for NTP server #904

Closed

Description

@troglobit

Description

Currently Infix only supports running chronyd as an NTP client via configuration in ietf-system.yang.

This is a proposal to use RFC 9249, in a stripped down form, to add NTP server support as well using chronyd.

Original tree view, from the RFC, is available here: https://github.com/dhruvdhody/ietf-yang/blob/master/ietf-ntp.tree

Tree View

This is an initial, heavily cut down (deviated), tree view based on the RFC.

module: ietf-ntp
  +--rw ntp!
     +--rw port?                  inet:port-number {ntp-port}?
     +--rw refclock-master!
     |  +--rw master-stratum?     ntp-stratum
     +--rw interfaces
     |  +--rw interface* [name]
     |     +--rw name             if:interface-ref
     +--ro ntp-statistics
        +--ro packet-sent?        yang:counter32
        +--ro packet-sent-fail?   yang:counter32
        +--ro packet-received?    yang:counter32
        +--ro packet-dropped?     yang:counter32

Additional Information

No response

General Information

Anyone can help out by sponsoring development of new features or contributing pull requests.
Please use this issue for discussions related to the feature.

Activity

  1. troglobit commented on Jan 23, 2025

    @troglobit
    ContributorAuthor

    CCB decision: great feature, needs to be investigated further which model to use though. Aiming for milestone: future, for now.

  2. added this to the FUTURE milestone on Jan 23, 2025
  3. removed
    triagePending investigation & classification (CCB)
    on Jan 23, 2025
  4. mattiaswal commented on Jan 31, 2025

    @mattiaswal
    Contributor

    From the ietf-ntp model:

     container ntp {
        when 'false() = boolean(/sys:system/sys:ntp)' {
          description
            "Applicable when the system /sys/ntp/ is not used.";
        }
        presence "NTP is enabled and system should attempt to
                  synchronize the system clock with an NTP server
                  from the 'ntp/associations' list.";
        description
          "Configuration parameters for NTP";
        leaf port {
          if-feature "ntp-port";
          type inet:port-number {
            range "123 | 1024..max";
          }
          default "123";
          description
            "Specify the port used to send and receive NTP packets.";
    
    
  5. added
    enhancementNew feature or request
    and removed on Feb 7, 2025
  6. self-assigned this
    on Nov 27, 2025
  7. moved this to In progress in Infix & C:oon Nov 27, 2025
  8. added 3 commits that reference this issue on Dec 3, 2025
    83d422b
    6735cbc
    d00a450
  9. linked a pull request that will close this issueNTP server support #1302on Dec 5, 2025
  10. moved this from In progress to Blocked in Infix & C:oon Dec 5, 2025
  11. added 2 commits that reference this issue on Jan 3, 2026
    5a5f7c7
    efe4903
  12. moved this from Blocked to In progress in Infix & C:oon Jan 4, 2026
  13. moved this from In progress to Done in Infix & C:oon Jan 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions