Remove partial dependency in PCSContainerService (#1428)

Summary:
Pull Request resolved: #1428

X-link: facebookresearch/fbpcp#405

We decided that validate_container_definitions should not be a public API in container service. This diff is to remove partial dependency in PCSContainerService

The validation logic has been moved in D38629555, to ensure the backward compatibility, we firstly changed the function body to ``pass``.

Differential Revision: D38604420

fbshipit-source-id: 74e63e0bd150e398b1a427fe35339b9ab0de6b39

fbpcs/CHANGELOG.md

@@ -17,7 +17,7 @@ Types of changes
 ### Added
 
 ### Changed
-
+  - Mark validate_container_definition as deprecated in PCSContainerService since it is no longer a public method in ContainerService in fbpcp
 ### Removed
 
 ## [1.10.0] - 2022-08-12

fbpcs/common/service/pcs_container_service.py

@@ -15,6 +15,8 @@
 from fbpcs.common.entity.pcs_container_instance import PCSContainerInstance
 from fbpcs.experimental.cloud_logs.log_retriever import CloudProvider, LogRetriever
 
+from fbpcs.private_computation.service.utils import deprecated
+
 
 class PCSContainerService(ContainerService):
     def __init__(self, inner_container_service: ContainerService) -> None:
@@ -80,7 +82,8 @@ def cancel_instances(self, instance_ids: List[str]) -> List[Optional[PcpError]]:
     def get_current_instances_count(self) -> int:
         return self.inner_container_service.get_current_instances_count()
 
+    @deprecated(
+        "validate_container_definition is no longer a public method in container service"
+    )
     def validate_container_definition(self, container_definition: str) -> None:
-        return self.inner_container_service.validate_container_definition(
-            container_definition
-        )
+        pass