Provider for Plainbox - PC Sanity

Merge ~alextu/plainbox-provider-pc-sanity:bootstrap-pc-sanity-smoke-test into plainbox-provider-pc-sanity:master

  1. Git
  2. lp:~alextu/plainbox-provider-pc-sanity
  3. bootstrap-pc-sanity-smoke-test
  4. Merge into master
Proposed by Alex Tu
Status: Merged
Merged at revision: 4f78b8e2a46223ccf4e16e7cc3246452e527b39c
Proposed branch: ~alextu/plainbox-provider-pc-sanity:bootstrap-pc-sanity-smoke-test
Merge into: plainbox-provider-pc-sanity:master
Diff against target: 27 lines (+8/-0)
2 files modified
units/pc-sanity/pc-sanity-misc.pxu (+7/-0)
units/pc-sanity/pc-sanity.pxu (+1/-0)
Reviewer Review Type Date Requested Status
Cyrus Lien Approve
Yuan-Chen Cheng (community) Approve
jeremyszu (community) Needs Information
Kai-Chuan Hsieh Pending
OEM Solutions Group: Engineers Pending
Review via email: mp+396041@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alex Tu (alextu) wrote :

an example of added attachement could be found there: http://10.101.46.50:8080/job/sanity-3-testflinger-dell-bto-focal-fossa-201810-26535-staging/317/

Revision history for this message
StanleyHuang (stanley31) wrote :

I know we would like to list down the job order, but how can we make sure the order is correct during the testing?

Revision history for this message
jeremyszu (os369510) wrote :

I didn't see the result of "miscellanea/bootstrap-pc-sanity-smoke-test" in your shared jenkins job.
http://10.101.46.50:8080/job/sanity-3-testflinger-dell-bto-focal-fossa-201810-26535-staging/317/testReport/com.canonical/

Could you please point out where is it?

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

please refer to:
 -> http://10.101.46.50:8080/job/sanity-3-testflinger-dell-bto-focal-fossa-201810-26535-staging/317/artifact/artifacts/submission.html
 -> http://10.101.46.50:8080/job/sanity-3-testflinger-dell-bto-focal-fossa-201810-26535-staging/317/artifact/artifacts/submission.html#attachment-30-log

Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

LGTM

review: Approve
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-misc.pxu b/units/pc-sanity/pc-sanity-misc.pxu
2index 1b51696..caf6317 100644
3--- a/units/pc-sanity/pc-sanity-misc.pxu
4+++ b/units/pc-sanity/pc-sanity-misc.pxu
5@@ -93,3 +93,10 @@ command: dpkg -l
6 _description:
7 attache log from dpkg -l
8
9+plugin: attachment
10+category_id: com.canonical.plainbox::miscellanea
11+id: miscellanea/bootstrap-pc-sanity-smoke-test
12+estimated_duration: 1.0
13+command: checkbox-cli list-bootstrapped com.canonical.certification::pc-sanity-smoke-test || true
14+_description:
15+ Attache the order of executed order of test cases.
16diff --git a/units/pc-sanity/pc-sanity.pxu b/units/pc-sanity/pc-sanity.pxu
17index 286af19..77640c8 100644
18--- a/units/pc-sanity/pc-sanity.pxu
19+++ b/units/pc-sanity/pc-sanity.pxu
20@@ -5,6 +5,7 @@ _description:
21 This test plan is for smoke test as sanity before release image.
22 include:
23 com.canonical.certification::miscellanea/dpkg-l
24+ com.canonical.certification::miscellanea/bootstrap-pc-sanity-smoke-test
25 com.canonical.certification::somerville-installation
26 com.canonical.certification::somerville/platform-meta-test
27 com.canonical.certification::misc/generic/grub_boothole

Subscribers

People subscribed via source and target branches