tests: don't use tmp as the default log dir with run.py

The log file directory is configurable with run.py using the
--log-dir argument. This patch removes the use of /tmp as
the default dir for storing all test logs. The default
log dir is now set to show the year, month and day
of the test run. This provides a more meaningful aggregation
of test logs for effective troubleshooting. The default log
dir is set to <CWD>/test-run-YYYY-MM-DD.

Type: improvement
Change-Id: I6c9002e961f6e06fc953ca42d86febf4f218e566
Signed-off-by: Naveen Joy <najoy@cisco.com>

diff --git a/.gitignore b/.gitignore
index 06c57c3..aba513c 100644 (file)
--- a/.gitignore
+++ b/.gitignore
@@ -119,3 +119,7 @@ compile_commands.json
 
 # debian packaging
 .pc
+
+# No test log files
+**/test-run-*-*-*
+

diff --git a/test/run.py b/test/run.py
index 646354a..ddb7995 100755 (executable)
--- a/test/run.py
+++ b/test/run.py
@@ -25,6 +25,7 @@ from subprocess import Popen, PIPE, STDOUT, call
 import sys
 import time
 import venv
+import datetime
 
 
 # Required Std. Path Variables
@@ -300,6 +301,7 @@ def run_tests_in_venv(
         f"--filter={test}",
         f"--jobs={jobs}",
         f"--log-dir={log_dir}",
+        f"--tmp-dir={log_dir}",
     ]
     if running_vpp:
         args = args + [f"--use-running-vpp"]
@@ -376,9 +378,9 @@ if __name__ == "__main__":
     parser.add_argument(
         "--log-dir",
         action="store",
-        default="/tmp",
+        default=os.path.abspath(f"./test-run-{datetime.date.today()}"),
         help="directory where to store directories "
-        "containing log files (default: /tmp)",
+        "containing log files (default: ./test-run-YYYY-MM-DD)",
     )
     parser.add_argument(
         "--jobs",