ci: Lint fixes

Removed old stuff, and pulled out MutatorType to make the
RallyEcsFormatter definition shorter

esrally/log.py

@@ -43,11 +43,14 @@ def configure_utc_formatter(*args: typing.Any, **kwargs: typing.Any) -> logging.
     return formatter
 
 
+MutatorType = typing.Callable[[logging.LogRecord, typing.Dict[str, typing.Any]], None]
+
+
 class RallyEcsFormatter(ecs_logging.StdlibFormatter):
     def __init__(
         self,
-        mutators: typing.Optional[typing.List[typing.Callable[[typing.Dict[str, typing.Any]], None]]] = None,
         *args: typing.Any,
+        mutators: typing.Optional[typing.List[MutatorType]] = None,
         **kwargs: typing.Any,
     ):
         super().__init__(*args, **kwargs)
@@ -95,15 +98,6 @@ def configure_ecs_formatter(*args: typing.Any, **kwargs: typing.Any) -> ecs_logg
     return formatter
 
 
-def configure_json_formatter(*args: typing.Any, **kwargs: typing.Any) -> RallyJsonFormatter:
-    """
-    JSON Logging formatter
-    """
-    formatter = jsonlogger.JsonFormatter(*args, **kwargs)
-
-    return formatter
-
-
 def log_config_path():
     """
     :return: The absolute path to Rally's log configuration file.