Checkbox Provider - Resource

~sylvain-pineau/plainbox-provider-resource:more_snapd_fixes

  1. Git
  2. lp:~sylvain-pineau/plainbox-provider-resource
  3. more_snapd_fixes
Last commit made on 2017-12-10
Get this branch:
git clone -b more_snapd_fixes https://git.launchpad.net/~sylvain-pineau/plainbox-provider-resource
Only Sylvain Pineau can upload to this branch. If you are Sylvain Pineau please log in for upload directions.

Branch merges

Branch information

Name:
more_snapd_fixes
Repository:
lp:~sylvain-pineau/plainbox-provider-resource

Recent commits

77d80b1... by Sylvain Pineau

snapd_resource: return an empty response instead of exiting with 0

Allowing the following job requirement to work as expected on system w/o snapd:

ID: com.canonical.certification::sysfs_attachment
Category: com.canonical.plainbox::info
Job cannot be started because:
 - resource expression 'model_assertion.model != "dragonboard"' evaluates to false
Outcome: job cannot be started

4bdac3f... by Sylvain Pineau

snapd_resource: Stop raising exception if snapd is not installed

0892562... by PMR <pmr@pmr-lander>

Merge #333979 from ~sylvain-pineau/plainbox-provider-resource:udev_infiniband

d43ba09... by Sylvain Pineau

udev_resource: Add INFINIBAND to the list of filter options

90c280c... by PMR <pmr@pmr-lander>

Merge #333604 from ~kissiel/plainbox-provider-resource:autoswitch-gpus

2d8def9... by Maciej Kisielewski

graphics resource: report cmd to switch GPUs

Signed-off-by: Maciej Kisielewski <email address hidden>

e8a3466... by PMR <pmr@pmr-lander>

Merge #333582 from ~kissiel/plainbox-provider-resource:fix-1731660-bad-driver-reported

98bc8d2... by Maciej Kisielewski

override the driver name to amdgpu-pro only for vendor == amd

previously, if the system had vulkan-amdgpu-pro pkg installed, the script
reported 'amdgpu-pro' as the driver for all the GPUs.

Now it's amdgpu-pro is reported only for GPUs from vendor 4098 - AMD.
This is still not perfect, as theretically there might be multiple amd gpus
in the system, not all of them amdgpu-pro, but it still makes it closer to the
truth and right for the typical intel+radeon configuration.

This patch also rearranges the logic for the dpkg-query call.
if not `some commands run` intuitively should mean "if the command failed"
which in the case of subprocess.call(...) is casted to boolean, therefore
`if not subprocess.call(...)` means success. I added explicit comparison to 0.

Fixes: LP:1731660

Signed-off-by: Maciej Kisielewski <email address hidden>

ec0f42d... by Maciej Kisielewski

graphics: report how to switch GPUs in the record

Signed-off-by: Maciej Kisielewski <email address hidden>

6d0d68e... by PMR <pmr@pmr-lander>

Merge #332706 from plainbox-provider-resource:release

Release_2017_Week43