Skip to content

Commit bdc3b13

Browse files
committed
fix: don't include veth NICs in summary field
Signed-off-by: Harper, Jason M <jason.m.harper@intel.com>
1 parent 4d9ad8a commit bdc3b13

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

internal/extract/nic.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,8 +231,8 @@ func NICSummaryFromOutput(outputs map[string]script.ScriptOutput) string {
231231
nics := ParseNicInfo(outputs[script.NicInfoScriptName].Stdout)
232232
modelCount := make(map[string]int)
233233
for _, nic := range nics {
234-
// don't include virtual functions or bridge devices in the summary as they are not physical NICs, and we want the summary to reflect physical hardware
235-
if nic.IsVirtual || strings.Contains(nic.Name, "(virtual)") || nic.Driver == "bridge" {
234+
// don't include virtual functions, bridge devices, or virtual Ethernet pairs (veth) devices in the summary as they are not physical NICs, and we want the summary to reflect physical hardware
235+
if nic.IsVirtual || strings.Contains(nic.Name, "(virtual)") || nic.Driver == "bridge" || nic.Driver == "veth" {
236236
continue
237237
}
238238
if strings.TrimSpace(nic.Model) == "" {

0 commit comments

Comments
 (0)