From 5e8b3f9215c61c53f37cbd07c155d7a6d0bb10a6 Mon Sep 17 00:00:00 2001 From: Johan Schreurs Date: Mon, 12 Dec 2022 16:26:24 +0100 Subject: [PATCH] Issue #84: Cleanup left over from workaround that forces sentinelhub for secondary service creation --- src/openeo_aggregator/backend.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/openeo_aggregator/backend.py b/src/openeo_aggregator/backend.py index 8736e8b9..7d00e57a 100644 --- a/src/openeo_aggregator/backend.py +++ b/src/openeo_aggregator/backend.py @@ -682,7 +682,7 @@ def service_types(self) -> dict: return {name: data["service_type"] for name, data, in cached_info.items()} def _find_backend_id_for_service_type(self, service_type: str) -> str: - """Returns the ID of the backend that provides the service_type or None if no backend supports that service.""" + """Returns the ID of the backend that provides the service_type.""" cached_info = self._get_service_types_cached() backend_id = cached_info.get(service_type, {}).get("backend_id") if backend_id is None: @@ -800,16 +800,13 @@ def service_info(self, user_id: str, service_id: str) -> ServiceMetadata: service_json["id"] = ServiceIdMapping.get_aggregator_service_id(service_json["id"], con.id) return ServiceMetadata.from_dict(service_json) - def create_service(self, user_id: str, process_graph: dict, service_type: str, api_version: str, + def _create_service(self, user_id: str, process_graph: dict, service_type: str, api_version: str, configuration: dict) -> str: """ https://openeo.org/documentation/1.0/developers/api/reference.html#operation/create-service """ # TODO: configuration is not used. What to do with it? - # TODO: Strictly speaking it would be better to override _create_service instead of create_service - # but for now we override create_service so we can keep the special case for the "sentinelhub" backend. - backend_id = self._find_backend_id_for_service_type(service_type) process_graph = self._processing.preprocess_process_graph(process_graph, backend_id=backend_id)