Skip to content

Release/5.1.0#546

Closed
tdeva14 wants to merge 3 commits into
mainfrom
release/5.1.0
Closed

Release/5.1.0#546
tdeva14 wants to merge 3 commits into
mainfrom
release/5.1.0

Conversation

@tdeva14
Copy link
Copy Markdown
Contributor

@tdeva14 tdeva14 commented May 14, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 14, 2026 16:56
@tdeva14 tdeva14 requested a review from a team as a code owner May 14, 2026 16:56
@tdeva14 tdeva14 closed this May 14, 2026
@tdeva14 tdeva14 deleted the release/5.1.0 branch May 14, 2026 16:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Release 5.1.0 updates getPartnerProperty.sh to replace the legacy NTP/Chrony TR-181 property lookups with a new per-server Chrony.NTPServer.N.Settings scheme and a single Chrony.Makestep parameter, and adds the 5.1.0 entry to the changelog.

Changes:

  • Removed handlers for ChronyEnable, NTPMinpoll, NTPMaxpoll, NTPServer{1-5}Directive, and NTPMaxstep.
  • Added handlers for ntpHost{1-5}Settings mapping to Device.Time.Chrony.NTPServer.N.Settings and ntpMakestep mapping to Device.Time.Chrony.Makestep.
  • Added 5.1.0 entry in CHANGELOG.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
lib/rdk/getPartnerProperty.sh Replaces multiple legacy NTP/Chrony properties with consolidated per-server Settings and a Makestep lookup.
CHANGELOG.md Adds 5.1.0 release notes entry.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions github-actions Bot locked and limited conversation to collaborators May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants