Skip to content
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
wants to merge 16 commits into
base: maint
Choose a base branch
from

Conversation

ccztux
Copy link
Contributor

@ccztux ccztux commented Dec 19, 2021

Example json output from an interface with the new metrics:

{
    "enp0s3": {
        "dropin": [
            0,
            "packets"
        ],
        "packets_recv": [
            178160,
            "packets"
        ],
        "bytes_recv": [
            0.2,
            "GiB"
        ],
        "duplex": "full",
        "dropout": [
            0,
            "packets"
        ],
        "bytes_sent": [
            0.02,
            "GiB"
        ],
        "errout": [
            0,
            "errors"
        ],
        "mtu": 1500,
        "packets_sent": [
            88313,
            "packets"
        ],
        "isup": true,
        "errin": [
            0,
            "errors"
        ],
        "speed": [
            1000,
            "Mbit/s"
        ]
    }
}

Example call of check_ncpa.py against the interface node:

[user@cl01]: ncpa$ ./client/check_ncpa.py -H localhost -t mytoken -M interface/enp0s3
OK: Bytes_sent was 0.02 GiB (Dropin: 0 packets, Bytes_recv: 0.20 GiB, Packets_recv: 178237 packets, Dropout: 0 packets, Errout: 0 errors, Errin: 0 errors, Packets_sent: 88404 packets, Speed: 1000 Mbit/s) | 'bytes_sent'=0.02GiB;;; 'dropin'=0;;; 'bytes_recv'=0.20GiB;;; 'packets_recv'=178237;;; 'dropout'=0;;; 'errout'=0;;; 'errin'=0;;; 'packets_sent'=88404;;; 'speed'=1000;;;

@ccztux ccztux changed the base branch from master to maint December 19, 2021 16:41
czet and others added 15 commits December 19, 2021 17:46
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.
* 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants