Copy resultsdir contents from DUT to Server before server-side test ends. |
|||||
Issue descriptionThe server-side test, enterprise_LongevityTrackerServer, calls the client-side longevity_Tracker 5 times. After that, it must collect the results from the DUT, and copy it to the server-side results directory. It should do this before cleanup and resetting the TPM on the device. This change will add a file copy method to enterprise_LongevityTrackerServer, which will copy the results files (i.e., keyval, perf.csv, and results-chart.json) from the DUT's results directory (typically at /usr/local/autotest/results/default/longevity_Tracker/results/), to the server-side results directory (path stored in self.resultsdir, in the runtime scope of the server-side test). The server-side resultsdir is typically at /tmp/test_that_results_<RUNID>/results-1-enterprise_LongevityTrackerServer.chrome_sign_builder/enterprise_LongevityTrackerServer.chrome_sign_builder/results. This copy method will utilize the get_file() utility, defined in abstract_ssh.py (at https://cs.corp.google.com/chromeos_public/src/third_party/autotest/files/server/hosts/abstract_ssh.py).
,
Aug 31 2016
CL https://chromium-review.googlesource.com/#/c/376867 Note: After some experimentation, an easier method of copying files, using shutil.copy(), was used. Thanks to Danny for suggesting method.
,
Sep 8 2016
,
Sep 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/24b242ef09856eb56cc2fc48a07758de215adec3 commit 24b242ef09856eb56cc2fc48a07758de215adec3 Author: Scott Cunningham <scunningham@chromium.org> Date: Wed Aug 31 00:45:13 2016 Copy resultsdir from DUT to Server at end of test. Add a file copy method to enterprise_LongevityTrackerServer that will copy the results files (i.e., keyval, perf.csv, and results-chart.json) from the DUT's results directory (typically at /usr/local/autotest/results/default/longevity_Tracker/results/), to the server-side results directory (path stored in self.resultsdir, in the runtime scope of the server-side test). My local branch name: longsrvrcopy BUG= chromium:641762 TEST=test_that <IP> f:.*enterprise_LongevityTrackerServer/ control.chromesign Change-Id: Ie38ba04f66a514c7e4868dc11217201a17f2a476 Reviewed-on: https://chromium-review.googlesource.com/376867 Commit-Ready: Scott Cunningham <scunningham@chromium.org> Tested-by: Scott Cunningham <scunningham@chromium.org> Reviewed-by: Scott Cunningham <scunningham@chromium.org> Reviewed-by: Harpreet Grewal <harpreet@chromium.org> [modify] https://crrev.com/24b242ef09856eb56cc2fc48a07758de215adec3/client/site_tests/longevity_Tracker/longevity_Tracker.py [modify] https://crrev.com/24b242ef09856eb56cc2fc48a07758de215adec3/server/site_tests/enterprise_LongevityTrackerServer/control.chromesign [modify] https://crrev.com/24b242ef09856eb56cc2fc48a07758de215adec3/client/site_tests/longevity_Tracker/control.chromesign [modify] https://crrev.com/24b242ef09856eb56cc2fc48a07758de215adec3/server/site_tests/enterprise_LongevityTrackerServer/enterprise_LongevityTrackerServer.py
,
Sep 14 2016
,
Mar 3 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by scunning...@chromium.org
, Aug 31 2016