Perf.Measurements.TcRunMeasurement: remove run_success

This is equivalent to the measurement_success so this is no longer needed. Signed-off-by: Ondrej Lichtner <[email protected]>
LNST-project · Nov 20, 2024 · e8cf628 · e8cf628
1 parent e097351
commit e8cf628
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 14 deletions.
diff --git a/lnst/RecipeCommon/Perf/Measurements/Results/AggregatedTcRunMeasurementResults.py b/lnst/RecipeCommon/Perf/Measurements/Results/AggregatedTcRunMeasurementResults.py
@@ -34,10 +34,6 @@ def rule_install_rate(self) -> SequentialPerfResult:
     def rule_install_rate(self, result: SequentialPerfResult):
         self._rule_install_rate = result
 
-    @property
-    def run_success(self) -> bool:
-        return all((i.run_success for i in self.individual_results))
-
     @property
     def individual_results(self) -> list[TcRunMeasurementResults]:
         return self._individual_results

diff --git a/lnst/RecipeCommon/Perf/Measurements/Results/TcRunMeasurementResults.py b/lnst/RecipeCommon/Perf/Measurements/Results/TcRunMeasurementResults.py
@@ -17,7 +17,6 @@ def __init__(
         super().__init__(measurement, measurement_success, warmup_rules)
         self._device = device
         self._rule_install_rate: ParallelPerfResult = None
-        self._run_success: bool = None
 
     @property
     def metrics(self) -> list[str]:
@@ -39,14 +38,6 @@ def rule_install_rate(self) -> ParallelPerfResult:
     def rule_install_rate(self, result: ParallelPerfResult):
         self._rule_install_rate = result
 
-    @property
-    def run_success(self) -> bool:
-        return self._run_success
-
-    @run_success.setter
-    def run_success(self, v: bool):
-        self._run_success = v
-
     @property
     def description(self):
         return f"{self.device.host.hostid}.{self.device.name}" \

diff --git a/lnst/RecipeCommon/Perf/Measurements/TcRunMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/TcRunMeasurement.py
@@ -218,7 +218,6 @@ def collect_results(self) -> list[TcRunMeasurementResults]:
         run_result.rule_install_rate = ParallelPerfResult(
             [self._get_instance_interval(d) for d in instance_data]
         )
-        run_result.run_success = job.passed
 
         return [run_result]