Merge ~alextu/plainbox-provider-pc-sanity:thermald_debug_tips into plainbox-provider-pc-sanity:master

[Alex Tu]

update debug tips for bugs like LP: #1895776

[Kai-Chuan Hsieh (kchsieh)]

For thermald part, LGTM.

[Alex Tu (alextu)]

cleaned up non-related changes.
Sorry for that miss commit.

diff --git a/units/pc-sanity/pc-sanity-thermal.pxu b/units/pc-sanity/pc-sanity-thermal.pxu
index 264fe9a..5dc49ea 100644
--- a/units/pc-sanity/pc-sanity-thermal.pxu
+++ b/units/pc-sanity/pc-sanity-thermal.pxu
@@ -105,7 +105,12 @@ command:
  RETVAL=$?
  if [ $RETVAL -ne 0 ];
  then
-     journalctl -b -u thermald
+     journalctl -b -u thermald > $PLAINBOX_SESSION_SHARE/journalctl.log
+     echo "The log for thermald: $PLAINBOX_SESSION_SHARE/journalctl.log"
+     grep -q "Unsupported cpu model" < $PLAINBOX_SESSION_SHARE/journalctl.log && \
+     echo "Debugging Tips:"
+     echo "Please check the upstream code(https://github.com/intel/thermal_daemon/blob/master/src/thd_engine.cpp) for the target id in id_table[] " && \
+     echo "If the ID not there, then open a upstream bug like https://github.com/intel/thermal_daemon/issues/275 "
      exit $RETVAL
  fi
 _summary: Check if thermald is active