Revert "Changed default for InmantaBootloader's logging config argume…

…nt (PR #8530)"

This reverts commit dd19c6e.

src/inmanta/app.py

@@ -99,7 +99,7 @@ def start_server(options: argparse.Namespace) -> None:
     tracing.configure_logfire("server")
     util.ensure_event_loop()
 
-    ibl = InmantaBootloader(configure_logging=False)
+    ibl = InmantaBootloader()
 
     setup_signal_handlers(ibl.stop)
 
@@ -900,7 +900,7 @@ def default_logging_config(options: argparse.Namespace) -> None:
 
     if options.cmd == "server":
         # Upgrade with extensions
-        ibl = InmantaBootloader(configure_logging=False)
+        ibl = InmantaBootloader()
         ibl.start_loggers_for_extensions(component_config)
 
     assert component_config._loaded_config is not None  # make mypy happy
@@ -922,7 +922,7 @@ def default_logging_config(options: argparse.Namespace) -> None:
 def print_versions_installed_components_and_exit() -> None:
     # coroutine to make sure event loop is running for server slices
     async def print_status() -> None:
-        bootloader = InmantaBootloader(configure_logging=False)
+        bootloader = InmantaBootloader()
         app_context = bootloader.load_slices()
         product_metadata = app_context.get_product_metadata()
         extension_statuses = app_context.get_extension_statuses()

src/inmanta/server/bootloader.py

@@ -82,11 +82,10 @@ class InmantaBootloader:
     # Cache field for available extensions
     AVAILABLE_EXTENSIONS: Optional[dict[str, str]] = None
 
-    def __init__(self, *, configure_logging: bool = True) -> None:
+    def __init__(self, configure_logging: bool = False) -> None:
         """
         :param configure_logging: This config option is used by the tests to configure the logging framework.
                                   In normal execution, the logging framework is configured by the app.py
-                                  Defaults to true for backwards compatibility.
         """
         self.restserver = Server()
         self.started = False

tests/test_extension_loading.py

@@ -286,7 +286,7 @@ async def test_register_setting() -> None:
 
     config.server_enabled_extensions.set("testlogextender")
     with splice_extension_in("test_module_path"):
-        ibl = InmantaBootloader(configure_logging=False)
+        ibl = InmantaBootloader()
         ibl.start_loggers_for_extensions()
         logging.info("This is a log line")
     assert "TEST TEST TEST" in io.getvalue()