Merge lp:~bladernr/checkbox/1585802-udev_resource-virtual-disks into lp:checkbox

[Jeff Lane ]

Add a bit to have udevadm parser ignore devices where product is "Virtual Floppy" or "Virtual SDD/HDD" as those are fake devices created by Cisco CIMC to pass files from remote management connections.

[lp:~bladernr/checkbox/1585802-udev_resource-virtual-disks]

4380. By Jeff Lane  on 2016-05-26

Whoops, forgot the parenthesis

4381. By Jeff Lane  on 2016-05-26

Added a string I missed for Virtual CDROM

[Sylvain Pineau (sylvain-pineau)]

Thanks, I hope you covered all of them ;)

review: Approve

=== modified file 'checkbox-support/checkbox_support/parsers/udevadm.py'
--- checkbox-support/checkbox_support/parsers/udevadm.py	2016-05-18 07:21:31 +0000
+++ checkbox-support/checkbox_support/parsers/udevadm.py	2016-05-26 17:23:23 +0000
@@ -857,6 +857,13 @@
         if device.vendor == "iDRAC":
             return True
 
+        # Ignore virtual devices created by Cisco CIMC manager
+        # See pad.lv/1585802
+        if (device.product == "Virtual FDD/HDD" or
+            device.product == "Virtual Floppy" or
+            device.product == "Virtual CD/DVD"):
+            return True
+
         return False
 
     def getAttributes(self, path):