Code review comment for ~bladernr/plainbox-provider-resource:1705745-dmi-supported-resource

Revision history for this message
Rod Smith (rodsmith) wrote :

I've tested this on a couple of systems, tweaking the filename to simulate the non-presence of /sys/devices/virtual/dmi, and it worked as expected. (I also had to change some job definitions to use this modified test rather than check architecture type.)

review: Approve

« Back to merge proposal