Refactor log performance function

Author: ekouts

reframe/core/logging.py

@@ -933,11 +933,13 @@ 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):
+    def log_performance(
+            self, level, task, msg=None, multiline=False, log_sanity=False
+    ):
         if (
             self.check is None or
             not self.check.is_performance_check() and
-            not getattr(self.logger, '_log_sanity_results', False)
+            not log_sanity
         ):
             return
 

reframe/frontend/executors/__init__.py

@@ -528,8 +528,11 @@ 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)
+            self._perflogger.log_performance(
+                logging.INFO, self,
+                multiline=self._perflog_compat,
+                log_sanity=self._log_sanity_results
+            )
         except LoggingError as e:
             getlogger().warning(
                 f'could not log performance data for {self.testcase}: {e}'
@@ -553,8 +556,11 @@ 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)
+            self._perflogger.log_performance(
+                logging.INFO, self,
+                multiline=self._perflog_compat,
+                log_sanity=self._log_sanity_results
+            )
         except LoggingError as e:
             getlogger().warning(
                 f'could not log performance data for {self.testcase}: {e}'