Skip to content

actually use the stats graph for ntp-server_freq now, which as missed in #14834#14884

Merged
Jellyfrog merged 1 commit intolibrenms:masterfrom
VVelox:minor_ntp_fix
Mar 9, 2023
Merged

actually use the stats graph for ntp-server_freq now, which as missed in #14834#14884
Jellyfrog merged 1 commit intolibrenms:masterfrom
VVelox:minor_ntp_fix

Conversation

@VVelox
Copy link
Copy Markdown
Contributor

@VVelox VVelox commented Mar 9, 2023

Derp! Updated the graph include for the client graph in #14834 , but I forgot to update that line for ntp-server_freq.

This fixes that.

DO NOT DELETE THE UNDERLYING TEXT

Please note

Please read this information carefully. You can run ./lnms dev:check to check your code before submitting.

  • Have you followed our code guidelines?
  • If my Pull Request does some changes/fixes/enhancements in the WebUI, I have inserted a screenshot of it.
  • If my Pull Request makes discovery/polling/yaml changes, I have added/updated test data.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

@VVelox VVelox changed the title actually use the stats graph for ntp-server_freq now, whichw as missed in #14834 actually use the stats graph for ntp-server_freq now, which as missed in #14834 Mar 9, 2023
@Jellyfrog Jellyfrog merged commit 6b56e78 into librenms:master Mar 9, 2023
@librenms-bot
Copy link
Copy Markdown

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/23-4-0-changelog/21194/1

TheMysteriousX pushed a commit to TheMysteriousX/librenms that referenced this pull request Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants