Merge ~liaou3/checkbox-iiotg/+git/checkbox-provider-intliotg:exclude_mtd_test into ~checkbox-dev/checkbox-iiotg/+git/checkbox-provider-intliotg:master

Proposed by Vincent Liao
Status: Merged
Approved by: Vincent Liao
Approved revision: 76e44250d1d673299ac3720c2cb0049dbf634779
Merged at revision: 7629916920030c0b4cccbb9e827b1b030ae76459
Proposed branch: ~liaou3/checkbox-iiotg/+git/checkbox-provider-intliotg:exclude_mtd_test
Merge into: ~checkbox-dev/checkbox-iiotg/+git/checkbox-provider-intliotg:master
Diff against target: 60 lines (+6/-0)
3 files modified
units/test-plan-core.pxu (+2/-0)
units/test-plan-desktop.pxu (+2/-0)
units/test-plan-server.pxu (+2/-0)
Reviewer Review Type Date Requested Status
Vic Liu Approve
Vincent Liao Pending
StanleyHuang Pending
Patrick Liu Pending
PeiYao Chang Pending
Rick Wu Pending
Review via email: mp+447737@code.launchpad.net

This proposal supersedes a proposal from 2023-07-26.

Commit message

Modify: Exclude check mtd count test from automated test plan

Description of the change

According to [LP#2026403](https://bugs.launchpad.net/lookout-canyon/+bug/2026403), this test should be excluded to prevent from read/write operation may sabotaging the boot firmware.

To post a comment you must log in.
Revision history for this message
Vic Liu (zongminl) wrote : Posted in a previous version of this proposal

Please exclude all the MTD related tests, which are:

ce-oem-mtd/check-total-numbers
ce-oem-mtd/read-write-test-.*
also-after-suspend-ce-oem-mtd/check-total-numbers
also-after-suspend-ce-oem-mtd/read-write-test-.*

review: Needs Fixing
Revision history for this message
Vincent Liao (liaou3) wrote :

@Vic
I write it in regular expression `com.canonical.qa.ceoem::.*ce-oem-mtd.*`. So we could exclude any mtd related job.

Revision history for this message
Vic Liu (zongminl) wrote :

+1, thank you

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/test-plan-core.pxu b/units/test-plan-core.pxu
2index dc7a11e..0cae768 100644
3--- a/units/test-plan-core.pxu
4+++ b/units/test-plan-core.pxu
5@@ -103,6 +103,7 @@ nested_part:
6 exclude:
7 com.canonical.certification::tpm2.0_4.1.1/.*
8 com.canonical.certification::watchdog/.*
9+ com.canonical.qa.ceoem::.*ce-oem-mtd.*
10
11 id: intliotg-ubuntucore-20-automated
12 unit: test plan
13@@ -129,6 +130,7 @@ nested_part:
14 exclude:
15 com.canonical.certification::tpm2.0_4.1.1/.*
16 com.canonical.certification::watchdog/.*
17+ com.canonical.qa.ceoem::.*ce-oem-mtd.*
18
19 id: intliotg-ubuntucore-22-stress
20 unit: test plan
21diff --git a/units/test-plan-desktop.pxu b/units/test-plan-desktop.pxu
22index 4938653..c1553e0 100644
23--- a/units/test-plan-desktop.pxu
24+++ b/units/test-plan-desktop.pxu
25@@ -104,6 +104,7 @@ nested_part:
26 after-suspend-power-management-cpu-cstate-automated
27 com.canonical.qa.ceoem::after-suspend-ce-oem-automated
28 exclude:
29+ com.canonical.qa.ceoem::.*ce-oem-mtd.*
30
31 id: intliotg-desktop-20-04-automated
32 unit: test plan
33@@ -125,6 +126,7 @@ nested_part:
34 com.canonical.certification::client-cert-desktop-20-04-automated
35 after-suspend-power-management-cpu-cstate-automated
36 exclude:
37+ com.canonical.qa.ceoem::.*ce-oem-mtd.*
38
39 id: intliotg-desktop-22-04-stress
40 unit: test plan
41diff --git a/units/test-plan-server.pxu b/units/test-plan-server.pxu
42index 1871951..87b8ef8 100644
43--- a/units/test-plan-server.pxu
44+++ b/units/test-plan-server.pxu
45@@ -93,6 +93,7 @@ nested_part:
46 com.canonical.qa.ceoem::after-suspend-ce-oem-automated
47 exclude:
48 com.canonical.certification::watchdog/.*
49+ com.canonical.qa.ceoem::.*ce-oem-mtd.*
50
51 id: intliotg-server-20-04-automated
52 unit: test plan
53@@ -114,6 +115,7 @@ nested_part:
54 after-suspend-power-management-cpu-cstate-automated
55 exclude:
56 com.canonical.certification::watchdog/.*
57+ com.canonical.qa.ceoem::.*ce-oem-mtd.*
58
59 id: intliotg-server-22-04-stress
60 unit: test plan

Subscribers

People subscribed via source and target branches