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

correct spell problem : retransmitted #4

Merged
merged 1 commit into from
Jan 25, 2018
Merged

Conversation

Jijun
Copy link
Contributor

@Jijun Jijun commented Jan 25, 2018

correct spell problem : "retransmitted" should be retransmited

correct spell problem : "retransmitted" should be retransmited
@kloczek
Copy link
Owner

kloczek commented Jan 25, 2018

Hi,

Can you show me "netstat -V" on your system?

@Chewbakka-Wakka
Copy link
Contributor

It should be one of either; "retransmitted" "retransmit" or "retransmits".

@kloczek kloczek merged commit 6071c25 into kloczek:master Jan 25, 2018
@kloczek
Copy link
Owner

kloczek commented Jan 25, 2018

I was aware that this item was for some time in the unsupported state.
I remember that it was working on el6. Looks like in earlier versions of the net-tools was the typo and only recently it was corrected.

However, I think that generally, I'll change regexp in this item to the new one:

$ /bin/netstat -s|/bin/sed -n 's/\(.*\) segments retransmitted*/\1/ p'
    582
$ /bin/netstat -s|/bin/sed -n 's/\( *\)\(.*\) segments retransmitted/\2/ p'
582
$

PS. I must finish import templates script.

@Jijun
Copy link
Contributor Author

Jijun commented Jan 26, 2018

i am confused about that
my online server version is CentOS Linux release 7.2.1511:

netstat -V
net-tools 2.10-alpha
Fred Baumgarten, Alan Cox, Bernd Eckenfels, Phil Blundell, Tuan Hoang, Brian Micek and others
+NEW_ADDRT +RTF_IRTT +RTF_REJECT +FW_MASQUERADE +I18N +SELINUX
AF: (inet) +UNIX +INET +INET6 +IPX +AX25 +NETROM +X25 +ATALK +ECONET +ROSE -BLUETOOTH
HW:  +ETHER +ARC +SLIP +PPP +TUNNEL -TR +AX25 +NETROM +X25 +FR +ROSE +ASH +SIT +FDDI +HIPPI +HDLC/LAPB +EUI64 

netstat -s|grep segment
     25184028994 segments received
     25363957817 segments send out
     88295058 segments retransmited
     133814 bad segments received.

my computer is archlinux :

net-tools 2.10-alpha
Fred Baumgarten, Alan Cox, Bernd Eckenfels, Phil Blundell, Tuan Hoang, Brian Micek and others
+NEW_ADDRT +RTF_IRTT +RTF_REJECT +FW_MASQUERADE -I18N
AF: (inet) +UNIX +INET +INET6 +IPX +AX25 +NETROM +X25 +ATALK -ECONET +ROSE -BLUETOOTH
HW:  +ETHER +ARC +SLIP +PPP +TUNNEL -TR +AX25 +NETROM +X25 +FR +ROSE +ASH +SIT +FDDI +HIPPI +HDLC/LAPB +EUI64 


netstat -s|grep segment
    437274 segments received
    450478 segments sent out
    3203 segments retransmitted
    97 bad segments received

so that make me confused.

@kloczek
Copy link
Owner

kloczek commented Jan 26, 2018

On the bottom of the "OS Linux" template description is:

Notes:
- compliant with EL7 and compatible

CentOS 7.x is EL7 compliant.
So don't worry Jijun because according to this note your fix is correct.

Thank you for your patch :)

@kloczek
Copy link
Owner

kloczek commented Jan 26, 2018

Looks like archlinux version which you are using is not fully EL7 compliant.

@Chewbakka-Wakka
Copy link
Contributor

Chewbakka-Wakka commented Jan 26, 2018 via email

@kloczek
Copy link
Owner

kloczek commented Jan 26, 2018

I've been testing this on Fedora rawhide.

Nevertheless, it is yet another detail here.
In both cases is reported the same version of the netstat.
In my case it is:

$ /bin/netstat -V
net-tools 2.10-alpha
Fred Baumgarten, Alan Cox, Bernd Eckenfels, Phil Blundell, Tuan Hoang, Brian Micek and others
+NEW_ADDRT +RTF_IRTT +RTF_REJECT +FW_MASQUERADE +I18N +SELINUX
AF: (inet) +UNIX +INET +INET6 +IPX +AX25 +NETROM +X25 +ATALK -ECONET +ROSE +BLUETOOTH
HW:  +ETHER +ARC +SLIP +PPP +TUNNEL -TR +AX25 +NETROM +X25 +FR +ROSE +ASH +SIT +FDDI +HIPPI +HDLC/LAPB +EUI64 

To be honest, now I think that this item should be removed from OS Linux template and replaced by item from another template which will be built around /proc/net/snmp, /proc/net/netstat, /proc/net/sctp/snmp content.
Reading those files and exposing red data as the metrics could be done over zabbix agent loadable module.
This will be better as it will be not using system.run[] and will allow hiding differences in netstat output.

@kloczek
Copy link
Owner

kloczek commented Jan 26, 2018

Another solution is to change regexp to only common part of both spellings.
That will be working in both cases.

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.

3 participants