pkg/tcp_metrics/inetdiag/structs.go (8 lines): - line 64: // TODO should we just make this explicit in the code? - line 68: // TODO - should use binary functions instead? - line 90: // TODO - remove all these. - line 122: // TODO make this unexported - line 190: // TODO should use more net.IP code instead of custom code. - line 191: // TODO: reconcile this encoding of v4-in-v6 with the encoding used in https://golang.org/src/net/ip.go?s=1216:1245#L35 - line 260: // TODO - why using rtaAlign on InetDiagMsg ??? - line 277: // TODO - why using rtaAlign on InetDiagMsg ??? pkg/tcp_metrics/parser/parse.go (6 lines): - line 22: // TODO - populate this field if any message is ignored, or not fully parsed. - line 33: // TODO Do we need to record whether these are present and zero, vs absent? - line 38: // TODO Do we need to record present and zero, vs absent? - line 42: // TODO Do we need to record present and zero, vs absent? - line 122: // TODO - add metric so we can alert on these. - line 143: // TODO - why using rtaAlign on InetDiagMsg ??? pkg/tcp_metrics/parser/parser_linux.go (2 lines): - line 18: // TODO - get these from sys/unix or syscall - line 28: // TODO use unix instead. pkg/tcp_metrics/inetdiag/inetdiag.go (2 lines): - line 69: // TODO - Should check whether this matches the current linux header. - line 122: // TODO - Figure out why we aren't seeing INET_DIAG_DCTCPINFO or INET_DIAG_BBRINFO messages. pkg/metrics/collector/netlink_metrics.go (1 line): - line 307: // TODO avoid the copy. pkg/metrics/collector/pod_ip_metrics.go (1 line): - line 240: // TODO: #351 - fix lint errors