feat: add new rtt per op field to nfsclient (#10787)

plugins/inputs/nfsclient/README.md

@@ -62,7 +62,8 @@ If `fullstat` is set, a great deal of additional metrics are collected, detailed
   - ops (integer, count) - The number of operations of this type executed.
   - retrans (integer, count) - The number of times an operation had to be retried (retrans = OP_trans - OP_ops.  See nfs_ops below)
   - exe (integer, miliseconds) - The number of miliseconds it took to process the operations.
-  - rtt (integer, miliseconds) - The round-trip time for operations.
+  - rtt (integer, miliseconds) - The total round-trip time for all operations.
+  - rtt_per_op (float, miliseconds) - The average round-trip time per operation.
 
 In addition enabling `fullstat` will make many more metrics available.
 

plugins/inputs/nfsclient/nfsclient.go

@@ -190,6 +190,10 @@ func (n *NFSClient) parseStat(mountpoint string, export string, version string,
 		fields["bytes"] = nline[3] + nline[4]
 		fields["rtt"] = nline[6]
 		fields["exe"] = nline[7]
+		fields["rtt_per_op"] = 0.0
+		if nline[0] > 0 {
+			fields["rtt_per_op"] = float64(nline[6]) / float64(nline[0])
+		}
 		tags["operation"] = first
 		acc.AddFields("nfsstat", fields, tags)
 	}

plugins/inputs/nfsclient/nfsclient_test.go

@@ -108,6 +108,7 @@ func TestNFSClientProcessStat(t *testing.T) {
 		"bytes":      uint64(1207),
 		"rtt":        uint64(606),
 		"exe":        uint64(607),
+		"rtt_per_op": float64(1.01),
 	}
 
 	readTags := map[string]string{
@@ -124,6 +125,7 @@ func TestNFSClientProcessStat(t *testing.T) {
 		"bytes":      uint64(1407),
 		"rtt":        uint64(706),
 		"exe":        uint64(707),
+		"rtt_per_op": float64(1.0085714285714287),
 	}
 
 	writeTags := map[string]string{