Skip to content

[202311][cherry-pick][NTP] Configure NTP according to extended configuration#3080

Closed
fastiuk wants to merge 2 commits intosonic-net:202311from
fastiuk:dev-ntp-configuration
Closed

[202311][cherry-pick][NTP] Configure NTP according to extended configuration#3080
fastiuk wants to merge 2 commits intosonic-net:202311from
fastiuk:dev-ntp-configuration

Conversation

@fastiuk
Copy link
Contributor

@fastiuk fastiuk commented Dec 13, 2023

What I did

Fix NTP CLI according to new extended configuration abilities

How I did it

Additionally configure association_type and resolve_as when adding new NTP servers.

How to verify it

  • Add a new server
  • Show NTP sync show ntp. It should show you are sync with new server
  • Delete NTP server
  • Show NTP sync show ntp. It should show you are unsync

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

Not changed

Signed-off-by: Yevhen Fastiuk <yfastiuk@nvidia.com>
@fastiuk fastiuk self-assigned this Dec 13, 2023
@fastiuk fastiuk changed the base branch from master to 202311 December 13, 2023 23:56
@fastiuk fastiuk changed the base branch from 202311 to master December 13, 2023 23:56
@fastiuk fastiuk changed the base branch from master to 202311 December 13, 2023 23:57
@fastiuk
Copy link
Contributor Author

fastiuk commented Dec 14, 2023

Closing, as it is part of this branch already

@fastiuk fastiuk closed this Dec 14, 2023
@fastiuk fastiuk deleted the dev-ntp-configuration branch December 14, 2023 00:06
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.

NTP: Additional NTP configuration knobs + NTP server provisioning

1 participant