Fix --with-testbed option in parsec sequester CLI when the testbed template needs to create blocks

Author: touilleMan

server/parsec/cli/inspect.py

@@ -17,7 +17,12 @@
 from parsec.cli.utils import cli_exception_handler, start_backend
 from parsec.components.realm import RealmGrantedRole
 from parsec.components.user import UserDump
-from parsec.config import BaseDatabaseConfig, DisabledBlockStoreConfig, LogLevel
+from parsec.config import (
+    BaseDatabaseConfig,
+    DisabledBlockStoreConfig,
+    LogLevel,
+    MockedBlockStoreConfig,
+)
 
 
 class DevOption(click.Option):
@@ -93,7 +98,11 @@ async def _human_accesses(
     user_filter: str,
 ) -> None:
     # Can use a dummy blockstore config since we are not going to query it
-    blockstore_config = DisabledBlockStoreConfig()
+    if with_testbed is None:
+        blockstore_config = DisabledBlockStoreConfig()
+    else:
+        # Testbed template might need to create some blocks
+        blockstore_config = MockedBlockStoreConfig()
 
     async with start_backend(
         db_config=db_config,

server/parsec/cli/sequester_create.py

@@ -30,7 +30,12 @@
     SequesterServiceConfig,
     SequesterServiceType,
 )
-from parsec.config import BaseDatabaseConfig, DisabledBlockStoreConfig, LogLevel
+from parsec.config import (
+    BaseDatabaseConfig,
+    DisabledBlockStoreConfig,
+    LogLevel,
+    MockedBlockStoreConfig,
+)
 
 SEQUESTER_SERVICE_CERTIFICATE_PEM_HEADER = "-----BEGIN PARSEC SEQUESTER SERVICE CERTIFICATE-----"
 SEQUESTER_SERVICE_CERTIFICATE_PEM_FOOTER = "-----END PARSEC SEQUESTER SERVICE CERTIFICATE-----"
@@ -322,7 +327,11 @@ async def _create_service(
     sequester_service_config: SequesterServiceConfig,
 ) -> None:
     # Can use a dummy blockstore config since we are not going to query it
-    blockstore_config = DisabledBlockStoreConfig()
+    if with_testbed is None:
+        blockstore_config = DisabledBlockStoreConfig()
+    else:
+        # Testbed template might need to create some blocks
+        blockstore_config = MockedBlockStoreConfig()
 
     async with start_backend(
         db_config=db_config,

server/parsec/cli/sequester_list.py

@@ -17,7 +17,12 @@
     SequesterGetOrganizationServicesBadOutcome,
     WebhookSequesterService,
 )
-from parsec.config import BaseDatabaseConfig, DisabledBlockStoreConfig, LogLevel
+from parsec.config import (
+    BaseDatabaseConfig,
+    DisabledBlockStoreConfig,
+    LogLevel,
+    MockedBlockStoreConfig,
+)
 
 
 class DevOption(click.Option):
@@ -102,7 +107,11 @@ async def _list_services(
     organization_id: OrganizationID,
 ) -> None:
     # Can use a dummy blockstore config since we are not going to query it
-    blockstore_config = DisabledBlockStoreConfig()
+    if with_testbed is None:
+        blockstore_config = DisabledBlockStoreConfig()
+    else:
+        # Testbed template might need to create some blocks
+        blockstore_config = MockedBlockStoreConfig()
 
     async with start_backend(
         db_config=db_config,

server/parsec/cli/sequester_revoke.py

@@ -27,7 +27,12 @@
     SequesterRevokeServiceStoreBadOutcome,
     SequesterRevokeServiceValidateBadOutcome,
 )
-from parsec.config import BaseDatabaseConfig, DisabledBlockStoreConfig, LogLevel
+from parsec.config import (
+    BaseDatabaseConfig,
+    DisabledBlockStoreConfig,
+    LogLevel,
+    MockedBlockStoreConfig,
+)
 
 SEQUESTER_SERVICE_REVOCATION_CERTIFICATE_PEM_HEADER = (
     "-----BEGIN PARSEC SEQUESTER SERVICE REVOCATION CERTIFICATE-----"
@@ -280,7 +285,11 @@ async def _revoke_service(
     service_revocation_certificate_pem: str,
 ) -> None:
     # Can use a dummy blockstore config since we are not going to query it
-    blockstore_config = DisabledBlockStoreConfig()
+    if with_testbed is None:
+        blockstore_config = DisabledBlockStoreConfig()
+    else:
+        # Testbed template might need to create some blocks
+        blockstore_config = MockedBlockStoreConfig()
 
     async with start_backend(
         db_config=db_config,