Provider for Plainbox - PC Sanity

Merge ~alextu/plainbox-provider-pc-sanity:nvidia-rtd3-checking-v3 into plainbox-provider-pc-sanity:master

  1. Git
  2. lp:~alextu/plainbox-provider-pc-sanity
  3. nvidia-rtd3-checking-v3
  4. Merge into master
Proposed by Alex Tu
Status: Merged
Merged at revision: 700d198784023a60176825571c84bb5997165829
Proposed branch: ~alextu/plainbox-provider-pc-sanity:nvidia-rtd3-checking-v3
Merge into: plainbox-provider-pc-sanity:master
Diff against target: 29 lines (+4/-1)
2 files modified
units/pc-sanity/pc-sanity-nvidia.pxu (+3/-1)
units/pc-sanity/pc-sanity.pxu (+1/-0)
Reviewer Review Type Date Requested Status
Cyrus Lien Approve
Yuan-Chen Cheng Pending
OEM Solutions Group: Engineers Pending
Review via email: mp+394617@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alex Tu (alextu) wrote :

# on rkl (nvidia 455 + kernel 5.10)
# It correctly captured the know issue LP: # 1904762
https://pastebin.canonical.com/p/R32dNVwmGW/

# on I+N Nvidia mode (nvidia 455 + kernel 5.10)
https://pastebin.canonical.com/p/224Hp3TKCb/

# on I+N Intel mode (nvidia 455 + kernel 5.10)
# It correctly captured the know issue LP: # 1904762
https://pastebin.canonical.com/p/Xp5ZY5Swg8/

# on I+N Intel mode (nvidia 455 + kernel 4.6)
# LP: # 1904762 not happens to this case, so test case reflect correctly
https://pastebin.canonical.com/p/XX5fxgZSQ8/

# on UMA machine
https://pastebin.canonical.com/p/NRKsYXHhcb/

gitlab pipeline is running:
https://gitlab.com/alex-tu-cc/plainbox-provider-pc-sanity/-/pipelines/222977814

Revision history for this message
Alex Tu (alextu) wrote :

this is the report of `checkbox-run-plan pc-sanity-smoke-test -b`
https://oem-share.canonical.com/partners/somerville/share/temp/plainbox-provider-pc-sanity_mp394617/

The miscellanea/check-nvidia will be executed before other graphic testing so it will not be impacted by LP: # 1904762.

In this case on rkl (nvidia 455 + kernel 5.10)
It correctly captured the know issue LP: # 1904762 in miscellanea/check-nvidia.
And other graphic passed as expected for the environment at Intel and Nvidia mode.

Revision history for this message
Alex Tu (alextu) wrote :

a test build based this change is triggering on rklstaging test pool.
http://10.101.46.50:8080/view/auto-sanity/job/sanity-0-dell-bto-focal-fossa-rklstaging-generic-staging-testing/

Revision history for this message
Cyrus Lien (cyruslien) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/pc-sanity/pc-sanity-nvidia.pxu b/units/pc-sanity/pc-sanity-nvidia.pxu
2index 3d7153a..12a6e02 100644
3--- a/units/pc-sanity/pc-sanity-nvidia.pxu
4+++ b/units/pc-sanity/pc-sanity-nvidia.pxu
5@@ -1,9 +1,11 @@
6 plugin: shell
7 category_id: com.canonical.plainbox::miscellanea
8+requires:
9+ graphics_card.driver in ['nvidia', 'pcieport']
10 id: miscellanea/check-nvidia
11 command:
12 set -x
13- check-nvidia.sh
14+ check-nvidia.sh --out "$PLAINBOX_SESSION_SHARE"
15 _summary: check if nvidia dgpu behave as we expected.
16 _description:
17 Nvidia introduce runtime D3 (runtime pm) from Nvidia driver version 450. This test case is used to check if everything work as expected.
18diff --git a/units/pc-sanity/pc-sanity.pxu b/units/pc-sanity/pc-sanity.pxu
19index 87e3a45..c06622e 100644
20--- a/units/pc-sanity/pc-sanity.pxu
21+++ b/units/pc-sanity/pc-sanity.pxu
22@@ -5,6 +5,7 @@ _description:
23 This test plan is for smoke test as sanity before release image.
24 include:
25 com.canonical.certification::somerville-installation
26+ com.canonical.certification::miscellanea/check-nvidia
27 com.canonical.certification::miscellanea/debsums
28 com.canonical.certification::miscellanea/turbostate-long-idle-c10
29 com.canonical.certification::miscellanea/turbostate-s2i-power-residency

Subscribers

People subscribed via source and target branches