Project

General

Profile

Actions

Feature #7311

closed

[Tests] run-tests.sh should put all temp install dirs inside a single --temp

Added by Tom Clegg over 10 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Category:
Tests
Target version:
-
Story points:
0.5

Description

Currently, saving time by reusing temp dirs involves passing a growing list of arguments to run-tests like GOPATH=/tmp/1 VENVDIR=/tmp/2. If you let run-tests.sh do its own thing, it makes a number of temp dirs in /tmp/.

Better way:
  • When no override args are given, make one tempdir and put VENVDIR, GOPATH, etc. inside that. Call them VENVDIR, GOPATH, etc.
  • When a --temp arg is given, use that as the tempdir. Create whatever VENVDIR, GOPATH, etc. dirs don't already exist there.
Old recipe for reusing your tmp dirs:
  1. Run once with --leave-temp
  2. Copy & paste the GOPATH="/tmp/foo" etc. from the end of run-tests's output into your next command line, and run that from now on
New recipe for reusing your tmp dirs:
  1. Run "run-tests.sh --temp ~/.cache/arvados-build" one or more times
Actions #1

Updated by Ward Vandewege over 10 years ago

Reviewing 7311-temp-base

  • Looks like the temporary directories are not being pre-created in the new --tempdir case; is that on purpose? Line 308:

    eval "$tmpdir"="$temp/$tmpdir"

Actions #2

Updated by Tom Clegg over 10 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados-dev|commit:9afc5f2d9ca0af29c34ff15e58b66387c3c149a6.

Actions

Also available in: Atom PDF