Merge lp:~cypressyew/checkbox/server-fwts-hwe-attachment into lp:checkbox

Proposed by Po-Hsu Lin
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3968
Merged at revision: 3969
Proposed branch: lp:~cypressyew/checkbox/server-fwts-hwe-attachment
Merge into: lp:checkbox
Diff against target: 18 lines (+8/-0)
1 file modified
providers/plainbox-provider-checkbox/jobs/miscellanea.txt.in (+8/-0)
To merge this branch: bzr merge lp:~cypressyew/checkbox/server-fwts-hwe-attachment
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+269627@code.launchpad.net

Description of the change

Add the fwts hwe attachment job, as the server provider is using "fwts_results" as the log filename, which is different than the one we used in the client provider.

We can merge these two sets later (the one in firmware/ and miscellanea/)

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-checkbox/jobs/miscellanea.txt.in'
2--- providers/plainbox-provider-checkbox/jobs/miscellanea.txt.in 2015-08-18 22:25:21 +0000
3+++ providers/plainbox-provider-checkbox/jobs/miscellanea.txt.in 2015-08-31 10:40:26 +0000
4@@ -62,6 +62,14 @@
5 [[ -e ${PLAINBOX_SESSION_SHARE}/fwts_results.log ]] && cat ${PLAINBOX_SESSION_SHARE}/fwts_results.log
6 _description: Attaches the FWTS results log to the submission
7
8+plugin: attachment
9+estimated_duration: 0.5
10+id: miscellanea/fwts_results_hwe.log
11+command:
12+ [ -f $PLAINBOX_SESSION_SHARE/fwts_results_hwe.log ] && gzip -c $PLAINBOX_SESSION_SHARE/fwts_results_hwe.log | base64
13+_description: Attaches the FWTS results log to the submission (to HWE)
14+_summary: Attach FWTS log to submission (to HWE)
15+
16 plugin: shell
17 id: miscellanea/ipmi_test
18 requires: package.name == 'ipmitool'

Subscribers

People subscribed via source and target branches