Merge ~alextu/plainbox-provider-pc-sanity:turbostat-dep-on-cpu-low-power-idle into plainbox-provider-pc-sanity:master

[Alex Tu]

for https://bugs.launchpad.net/somerville/+bug/1904142

[Alex Tu (alextu)]

because a system failed on power-management/cpu-low-power-idle don't able to get package PC10.

So, it's meanless to check if it stays in PC10 long enough.

[Alex Tu (alextu)]

pipline: https://gitlab.com/alex-tu-cc/plainbox-provider-pc-sanity/-/pipelines/220374117

[Kai-Chuan Hsieh (kchsieh)]

It makes sense to me. Do you want to add dependency of power-management/system-low-power-idle for miscellanea/turbostate-s2i-power-residency?

review: Needs Information

[Alex Tu (alextu)]

good suggestion , but not necessary to add it.

Because SYS%LPI is the latest item of check-turbostat-power-residency.sh

GFX%rc6,Pkg%pc2,Pkg%pc3,Pkg%pc6,Pkg%pc7,Pkg%pc8,Pkg%pc9,Pk%pc10,SYS%LPI

It won't impact this test case calculating the residency of PC10.

Say, even s0ix not enabled, this test case provides the visibility of PC10.

[Kai-Chuan Hsieh (kchsieh)]

Okay, LGTM.

review: Approve

diff --git a/units/pc-sanity/pc-sanity-power-consumption.pxu b/units/pc-sanity/pc-sanity-power-consumption.pxu
index 6e6f534..f6d7716 100644
--- a/units/pc-sanity/pc-sanity-power-consumption.pxu
+++ b/units/pc-sanity/pc-sanity-power-consumption.pxu
@@ -4,6 +4,7 @@ plugin: shell
 category_id: com.canonical.plainbox::miscellanea
 id: miscellanea/turbostate-s2i-power-residency
 after: miscellanea/turbostate-long-idle-c10
+depends: power-management/cpu-low-power-idle
 user: root
 command:
  set -x
@@ -16,6 +17,7 @@ _description:
 plugin: shell
 category_id: com.canonical.plainbox::miscellanea
 id: miscellanea/turbostate-short-idle-c10
+depends: power-management/cpu-low-power-idle
 command:
  set -x
  report_folder="$HOME/submission-report-$(date +%Y%m%d%H%M)"
@@ -40,6 +42,7 @@ _description:
 plugin: shell
 category_id: com.canonical.plainbox::miscellanea
 id: miscellanea/turbostate-long-idle-c10
+depends: power-management/cpu-low-power-idle
 command:
  set -x
  report_folder="$HOME/submission-report-$(date +%Y%m%d%H%M)"
diff --git a/units/pc-sanity/pc-sanity.pxu b/units/pc-sanity/pc-sanity.pxu
index f623d52..87e3a45 100644
--- a/units/pc-sanity/pc-sanity.pxu
+++ b/units/pc-sanity/pc-sanity.pxu
@@ -29,3 +29,12 @@ nested_part:
 bootstrap_include:
     device
     graphics_card
+
+id: sleep-state-auto
+_name: sleep-state-auto (Ubuntu Desktop)
+unit: test plan
+_description:
+    This test plan collected all auto test related to sleep state.
+include:
+    com.canonical.certification::miscellanea/turbostate-long-idle-c10
+    com.canonical.certification::miscellanea/turbostate-s2i-power-residency