Skip to content

One ntp to rule#612

Open
MarlinMr wants to merge 1 commit into
librenms:masterfrom
MarlinMr:one-ntp-to-rule
Open

One ntp to rule#612
MarlinMr wants to merge 1 commit into
librenms:masterfrom
MarlinMr:one-ntp-to-rule

Conversation

@MarlinMr

@MarlinMr MarlinMr commented Mar 11, 2026

Copy link
Copy Markdown
Contributor
  • agree on variables to send back
  • extend to functionality from ntp-server.py

@MarlinMr MarlinMr changed the title WIP: One ntp to rule One ntp to rule Mar 21, 2026
@MarlinMr

Copy link
Copy Markdown
Contributor Author

See librenms/librenms#19291

@VVelox VVelox left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Needs to be portable. Use a flag for which should be used.

@VVelox VVelox left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Generally best to leave the old ones in place still if they work.

Comment thread snmp/ntp-client

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Also we generally leave older versions of the extends laying around if they work in case any one has anything that depends on those as part of a ansible/rex job or the like.

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.

2 participants