-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added new interface metrics: speed, mtu, duplex and isup. (#646) #837
Open
ccztux
wants to merge
16
commits into
NagiosEnterprises:maint
Choose a base branch
from
ccztux:issue-646
base: maint
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Installer: - Walk user through setting up Full Disk Access for terminal program - Install Python v2.7 if necessary - Walk user through authorizing unsigned Python executable - Added a lot more output to make it easier to debug failed installations - Added more checks to prevent failures for unexpected machine state - Fixed "upgrade" mode so that it actually works reliably - No longer requires plugin volume name to be hard coded in the installer - Added summary showing all added NCPA elements Uninstaller: - Fixed inability to remove nagios user and group - Added more checks to allow re-executing if some system state prevents completion - Added summary showing NCPA elements, so you can confirm they have been removed Added README.txt file to installer disk image, and updated build script.
…EADME to package.sh for MacOS.
* master: Update .gitignore to ignore build files on MacOS
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Example json output from an interface with the new metrics:
Example call of
check_ncpa.py
against the interface node: