Code review comment for ~alextu/plainbox-provider-pc-sanity:give-tips-to-proc-thermal-test

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

with this patch, the failed case would print this message :

ubuntu@u-Wyse-5070-Extended-Thin-Client:/var/tmp/checkbox-providers/plainbox-provider-pc-sanity/units$ checkbox-cli run com.canonical.certification::miscellanea/proc_thermal_driver
Using sideloaded provider: plainbox-provider-pc-sanity, version 1.20.0.dev0 from /var/tmp/checkbox-providers/plainbox-provider-pc-sanity
===========================[ Running Selected Jobs ]============================
--------------[ Running job 1 / 1. Estimated time left: unknown ]---------------
---------------------[ check if thermal driver installed. ]---------------------
ID: com.canonical.certification::miscellanea/proc_thermal_driver
Category: com.canonical.plainbox::miscellanea
... 8< -------------------------------------------------------------------------
++ date +%Y%m%d%H%M
+ report_folder=/home/ubuntu/submission-report-202010131436
+ mkdir -p /home/ubuntu/submission-report-202010131436
+ lspci -nnk
+ grep driver
+ grep proc_thermal
+ dmesg
+ lspci -nnk
+ lsmod
+ echo 'Please refer to LP: #1874008 and LP: #1851506 for the next steps.'
+ exit 1
Please refer to LP: #1874008 and LP: #1851506 for the next steps.
------------------------------------------------------------------------- >8 ---
Outcome: job failed
Finalizing session that hasn't been submitted anywhere: checkbox-run-2020-10-13T06.36.35
==================================[ Results ]===================================
 ☒ : check if thermal driver installed.

« Back to merge proposal