[sfputil] Display 'N/A' for non-SFP ports (#1078)

Currently, "Not Present" will be shown at non-sfp ports. This enhancement will show "N/A" instead of "Not Present" at non-sfp ports.
sonic-net · Nov 2, 2020 · ef5f212 · ef5f212
1 parent 8af9aee
commit ef5f212
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/sfputil/main.py b/sfputil/main.py
@@ -412,10 +412,11 @@ def presence(port):
                 click.echo("This functionality is currently not implemented for this platform")
                 sys.exit(5)
 
-            if presence:
-                output_table.append([port_name, "Present"])
+            if platform_sfputil._is_valid_port(physical_port):
+                status_string = "Present" if presence else "Not present"
+                output_table.append([port_name, status_string])
             else:
-                output_table.append([port_name, "Not present"])
+                output_table.append([port_name, "N/A"])
 
             i += 1