Provider for Plainbox - Canonical Certification Server

Merge ~rodsmith/plainbox-provider-certification-server:disk-cpu-load into plainbox-provider-certification-server:master

  1. Git
  2. lp:~rodsmith/plainbox-provider-certification-server
  3. disk-cpu-load
  4. Merge into master
Proposed by Rod Smith
Status: Merged
Approved by: Jeff Lane 
Approved revision: 7c831f078e10148de6e9ab298679d085c93d4ef1
Merged at revision: 7c831f078e10148de6e9ab298679d085c93d4ef1
Proposed branch: ~rodsmith/plainbox-provider-certification-server:disk-cpu-load
Merge into: plainbox-provider-certification-server:master
Diff against target: 48 lines (+4/-0)
4 files modified
units/server-full-16.04.pxu (+1/-0)
units/server-functional-16.04.pxu (+1/-0)
units/storage-only.pxu (+1/-0)
units/virtual-machine-full-16.04.pxu (+1/-0)
Reviewer Review Type Date Requested Status
Jeff Lane  Approve
Review via email: mp+304663@code.launchpad.net

Description of the change

Modify the server units files to call the new disk_cpu_load test, which is part of the bzr merge at https://code.launchpad.net/~rodsmith/checkbox/storage-devices-cpu-load/+merge/304662.

To post a comment you must log in.
Revision history for this message
Jeff Lane  (bladernr) wrote :

need to be marked as non-blockers. As it's a new test, anything newly introduced must be a non-blocking (Greylist) item first and can be promoted to blocking in the next LTS.

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

OK, I've changed that.

Revision history for this message
Jeff Lane  (bladernr) wrote :

Ack. THanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/server-full-16.04.pxu b/units/server-full-16.04.pxu
2index 792cbd5..0c0d1a3 100644
3--- a/units/server-full-16.04.pxu
4+++ b/units/server-full-16.04.pxu
5@@ -57,6 +57,7 @@ include:
6 disk/smart_.* certification-status=non-blocker
7 disk/storage_device_.* certification-status=blocker
8 disk/disk_stress_ng_.* certification-status=blocker
9+ disk/disk_cpu_load_.* certification-status=non-blocker
10 ethernet/detect certification-status=non-blocker
11 ethernet/info_automated certification-status=non-blocker
12 ethernet/ethtool_info certification-status=non-blocker
13diff --git a/units/server-functional-16.04.pxu b/units/server-functional-16.04.pxu
14index d08fbed..0e812c6 100644
15--- a/units/server-functional-16.04.pxu
16+++ b/units/server-functional-16.04.pxu
17@@ -57,6 +57,7 @@ include:
18 disk/smart_.* certification-status=non-blocker
19 disk/storage_device_.* certification-status=blocker
20 disk/disk_stress_ng_.* certification-status=blocker
21+ disk/disk_cpu_load_.* certification-status=non-blocker
22 ethernet/detect certification-status=non-blocker
23 ethernet/info_automated certification-status=non-blocker
24 ethernet/ethtool_info certification-status=non-blocker
25diff --git a/units/storage-only.pxu b/units/storage-only.pxu
26index 1017f7f..7ef3e43 100644
27--- a/units/storage-only.pxu
28+++ b/units/storage-only.pxu
29@@ -44,6 +44,7 @@ include:
30 disk/smart_.* certification-status=non-blocker
31 disk/storage_device_.* certification-status=blocker
32 disk/disk_stress_ng_.* certification-status=blocker
33+ disk/disk_cpu_load_.* certification-status=non-blocker
34 mediacard/sd-preinserted certification-status=blocker
35 bootstrap_include:
36 device
37diff --git a/units/virtual-machine-full-16.04.pxu b/units/virtual-machine-full-16.04.pxu
38index 14562cf..e1c23de 100644
39--- a/units/virtual-machine-full-16.04.pxu
40+++ b/units/virtual-machine-full-16.04.pxu
41@@ -51,6 +51,7 @@ include:
42 disk/smart_.* certification-status=non-blocker
43 disk/storage_device_.* certification-status=blocker
44 disk/disk_stress_ng_.* certification-status=blocker
45+ disk/disk_cpu_load_.* certification-status=non-blocker
46 ethernet/detect certification-status=non-blocker
47 ethernet/info_automated certification-status=non-blocker
48 ethernet/ethtool_info certification-status=non-blocker

Subscribers

People subscribed via source and target branches

to all changes: