Series-Relative Test IDs

lib/pavilion/builder.py

@@ -460,7 +460,7 @@ def build(self, test_id: str, tracker: BuildTracker,
         directory into our test directory, and note that we've used the given
         build.
 
-        :param test_id: The test 'full_id' for the test initiating this build.
+        :param test_id: The test id for the test initiating this build.
         :param tracker: A thread-safe tracker object for keeping info on what the
             build is doing.
         :param cancel_event: Allows builds to tell each other
@@ -570,7 +570,7 @@ def _build(self, build_dir, cancel_event, test_id, tracker: BuildTracker) -> boo
         :param Path build_dir: The directory in which to perform the build.
         :param threading.Event cancel_event: Event to signal that the build
             should stop.
-        :param test_id: The 'full_id' of the test initiating the build.
+        :param test_id: The ID of the test initiating the build.
         :param tracker: Build tracker for this build.
         :returns: True or False, depending on whether the build appears to have
             been successful.
@@ -592,7 +592,8 @@ def _build(self, build_dir, cancel_event, test_id, tracker: BuildTracker) -> boo
             # Do the build, and wait for it to complete.
             with self.tmp_log_path.open('w') as build_log:
                 # Build scripts take the test id as a first argument.
-                cmd = [self._script_path.as_posix(), test_id]
+                cmd = [self._script_path.as_posix(), str(test_id)]
+
                 proc = subprocess.Popen(cmd,
                                         cwd=build_dir.as_posix(),
                                         stdout=build_log,
@@ -797,10 +798,6 @@ def _setup_build_dir(self, dest: Path, tracker: BuildTracker) -> None:
             raise TestBuilderError("Error extracting file '{}'\n  {}"
                                    .format(src_path.as_posix(), extract_error))
 
-        tracker.update(
-            state=STATES.BUILDING,
-            note="Generating dynamically created files.")
-
         # Create build time file(s).
         for file, contents in self._config.get('create_files', {}).items():
             try:

lib/pavilion/cancel_utils.py

@@ -100,7 +100,7 @@ def cancel_tests(pav_cfg: PavConfig, tests: Iterable[TestRun], outfile: TextIO,
             title="Cancelling {} test{}".format(test_count, 's' if test_count > 1 else ''),
             outfile=outfile,
             fields=['name', 'id', 'state', 'series'],
-            rows=[{'name': test.name, 'id': test.full_id,
+            rows=[{'name': test.name, 'id': test.id,
                    'state': test.status.current().state, 'series': test.series}
                   for test in cancelled_test_info])
     else: