Log results for all tests, not only perf tests

Author: ekouts

reframe/core/logging.py

@@ -933,14 +933,10 @@ def _update_check_extras(self):
             self.extra['check_job_completion_time_unix'], r'%FT%T%:z'
         )
 
-    def log_performance(
-            self, level, task, msg=None, multiline=False, log_sanity=False
+    def log_result(
+            self, level, task, msg=None, multiline=False
     ):
-        if (
-            self.check is None or
-            not self.check.is_performance_check() and
-            not log_sanity
-        ):
+        if self.check is None:
             return
 
         _, part, env = task.testcase

reframe/frontend/executors/__init__.py

@@ -235,9 +235,6 @@ def __init__(self, case, listeners=None, timeout=None):
         self._perflog_compat = runtime.runtime().get_option(
             'logging/0/perflog_compat'
         )
-        self._log_sanity_results = runtime.runtime().get_option(
-            'logging/0/log_sanity_results'
-        )
 
     def duration(self, phase):
         # Treat pseudo-phases first
@@ -528,11 +525,8 @@ def finalize(self):
         self._current_stage = 'finalize'
         self._notify_listeners('on_task_success')
         try:
-            self._perflogger.log_performance(
-                logging.INFO, self,
-                multiline=self._perflog_compat,
-                log_sanity=self._log_sanity_results
-            )
+            self._perflogger.log_result(logging.INFO, self,
+                                        multiline=self._perflog_compat)
         except LoggingError as e:
             getlogger().warning(
                 f'could not log performance data for {self.testcase}: {e}'
@@ -556,11 +550,8 @@ def _wait_job(job):
         self._exc_info = exc_info or sys.exc_info()
         self._notify_listeners(callback)
         try:
-            self._perflogger.log_performance(
-                logging.INFO, self,
-                multiline=self._perflog_compat,
-                log_sanity=self._log_sanity_results
-            )
+            self._perflogger.log_result(logging.INFO, self,
+                                        multiline=self._perflog_compat)
         except LoggingError as e:
             getlogger().warning(
                 f'could not log performance data for {self.testcase}: {e}'

reframe/schemas/config.json

@@ -440,7 +440,6 @@
                 "properties": {
                     "level": {"$ref": "#/defs/loglevel"},
                     "perflog_compat": {"type": "boolean"},
-                    "log_sanity_results": {"type": "boolean"},
                     "handlers": {
                         "type": "array",
                         "items": {
@@ -622,7 +621,6 @@
         "general/verbose": 0,
         "logging/level": "undefined",
         "logging/perflog_compat": false,
-        "logging/log_sanity_results": false,
         "logging/target_systems": ["*"],
         "logging/handlers": [],
         "logging/handlers_perflog": [],