Move log_sanity_results to logger instead of handlers

Author: ekouts

reframe/core/logging.py

@@ -604,7 +604,6 @@ def _create_httpjson_handler(site_config, config_prefix):
     debug = site_config.get(f'{config_prefix}/debug')
     backoff_intervals = site_config.get(f'{config_prefix}/backoff_intervals')
     retry_timeout = site_config.get(f'{config_prefix}/retry_timeout')
-    log_sanity_results = site_config.get(f'{config_prefix}/log_sanity_results')
 
     parsed_url = urllib.parse.urlparse(url)
     if parsed_url.scheme not in {'http', 'https'}:
@@ -647,7 +646,7 @@ def _create_httpjson_handler(site_config, config_prefix):
 
     return HTTPJSONHandler(url, extras, ignore_keys, json_formatter,
                            extra_headers, debug, backoff_intervals,
-                           retry_timeout, log_sanity_results)
+                           retry_timeout)
 
 
 def _record_to_json(record, extras, ignore_keys):
@@ -698,8 +697,7 @@ class HTTPJSONHandler(logging.Handler):
 
     def __init__(self, url, extras=None, ignore_keys=None,
                  json_formatter=None, extra_headers=None,
-                 debug=False, backoff_intervals=(1, 2, 3), retry_timeout=0,
-                 log_sanity_results=False):
+                 debug=False, backoff_intervals=(1, 2, 3), retry_timeout=0):
         super().__init__()
         self._url = url
         self._extras = extras
@@ -725,7 +723,6 @@ def __init__(self, url, extras=None, ignore_keys=None,
         self._debug = debug
         self._timeout = retry_timeout
         self._backoff_intervals = backoff_intervals
-        self._log_sanity_results = log_sanity_results
 
     def emit(self, record):
         # Convert tags to a list to make them JSON friendly

reframe/frontend/executors/__init__.py

@@ -235,6 +235,9 @@ 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

reframe/schemas/config.json

@@ -181,7 +181,6 @@
                             "type": "array",
                             "items": {"type": "number"}
                         },
-                        "log_sanity_results": {"type": "boolean"},
                         "retry_timeout": {"type": "number"}
                     },
                     "required": ["url"]
@@ -441,6 +440,7 @@
                 "properties": {
                     "level": {"$ref": "#/defs/loglevel"},
                     "perflog_compat": {"type": "boolean"},
+                    "log_sanity_results": {"type": "boolean"},
                     "handlers": {
                         "type": "array",
                         "items": {
@@ -622,6 +622,7 @@
         "general/verbose": 0,
         "logging/level": "undefined",
         "logging/perflog_compat": false,
+        "logging/log_sanity_results": false,
         "logging/target_systems": ["*"],
         "logging/handlers": [],
         "logging/handlers_perflog": [],
@@ -648,7 +649,6 @@
         "logging/handlers_perflog/httpjson_extra_headers": {},
         "logging/handlers_perflog/httpjson_debug": false,
         "logging/handlers_perflog/httpjson_backoff_intervals": [0.1, 0.2, 0.4, 0.8, 1.6, 3.2],
-        "logging/handlers_perflog/httpjson_log_sanity_results": false,
         "logging/handlers_perflog/httpjson_retry_timeout": 0,
         "modes/options": [],
         "modes/target_systems": ["*"],