Add verbose option to backend_factory (useful to avoid useless lo…

…g output when used in short-lived CLI commands)

server/parsec/backend.py

@@ -61,10 +61,11 @@
 
 
 @asynccontextmanager
-async def backend_factory(config: BackendConfig) -> AsyncGenerator[Backend, None]:
+async def backend_factory(config: BackendConfig, verbose: bool) -> AsyncGenerator[Backend, None]:
     # Log the server version and the backend configuration
-    logger.info("Parsec version", version=server_version)
-    logger.info("Backend configuration", **config.logging_kwargs())
+    if verbose:
+        logger.info("Parsec version", version=server_version)
+        logger.info("Backend configuration", **config.logging_kwargs())
 
     if config.db_config.is_mocked():
         components_factory = mocked_components_factory

server/parsec/cli/run.py

@@ -480,7 +480,7 @@ async def _run_backend(
             retry_policy.new_attempt()
 
             # Run the backend app (and connect to the database)
-            async with backend_factory(config=app_config) as backend:
+            async with backend_factory(config=app_config, verbose=True) as backend:
                 # Connection is successful, reset the retry policy
                 retry_policy.success()
 

server/parsec/cli/testbed.py

@@ -247,7 +247,7 @@ async def testbed_backend_factory(
         administration_token="s3cr3t",
         organization_spontaneous_bootstrap=True,
     )
-    async with backend_factory(config=config) as backend:
+    async with backend_factory(config=config, verbose=True) as backend:
         yield TestbedBackend(backend=backend)
 
 

server/tests/common/backend.py

@@ -65,7 +65,7 @@ async def backend(
 
     async def _run_backend():
         nonlocal backend
-        async with backend_factory(config=backend_config) as backend:
+        async with backend_factory(config=backend_config, verbose=False) as backend:
             started.set()
             try:
                 await should_stop.wait()

server/tests/test_cross_server_event.py

@@ -13,8 +13,8 @@
 @pytest.mark.postgresql
 async def test_cross_server_event(backend_config: BackendConfig) -> None:
     async with (
-        backend_factory(config=backend_config) as b1,
-        backend_factory(config=backend_config) as b2,
+        backend_factory(config=backend_config, verbose=False) as b1,
+        backend_factory(config=backend_config, verbose=False) as b2,
     ):
         b2_received_events = Queue()