commit e3c0df695330359dbf26bfe98a87233195bc4014
author Alex Klein <saklein@chromium.org> Wed Nov 13 13:32:13 2019 -0700
committer Commit Bot <commit-bot@chromium.org> Fri Nov 22 09:28:08 2019 +0000
tree 40ca1deb5c00f7e89ffe2cb9ba17a4fb763f64a9
parent a4a9f0235249c3ec4afac73f9fbf926fc2f8ddab

test controller: Fix moblab vm test result path.

The path is a tempdir in the chroot, but was being passed in as
the absolute path outside the chroot. Fix to be a chroot-absolute
path.

BUG=chromium:998155
TEST=run_tests

Change-Id: I5167ba724a8ce7bfb852e96832099fb409d5f6bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1914507
Tested-by: Alex Klein <saklein@chromium.org>
Auto-Submit: Alex Klein <saklein@chromium.org>
Reviewed-by: Michael Mortensen <mmortensen@google.com>
Commit-Queue: Alex Klein <saklein@chromium.org>

api/controller/test.py

diff --git a/api/controller/test.py b/api/controller/test.py
index 2e8892c..2f26f35 100644
--- a/api/controller/test.py
+++ b/api/controller/test.py
@@ -203,9 +203,11 @@
 
   # Now we can run the tests.
   with chroot.tempdir() as workspace_dir, chroot.tempdir() as results_dir:
+    # Convert the results directory to an absolute chroot directory.
+    chroot_results_dir = '/%s' % os.path.relpath(results_dir, chroot.path)
     vms = test.CreateMoblabVm(workspace_dir, chroot.path, image_payload_dir)
     cache_dir = test.PrepareMoblabVmImageCache(vms, builder, cache_payload_dirs)
-    test.RunMoblabVmTest(chroot, vms, builder, cache_dir, results_dir)
+    test.RunMoblabVmTest(chroot, vms, builder, cache_dir, chroot_results_dir)
     test.ValidateMoblabVmTest(results_dir)