Revert "Custom protoc plugin to generate a grpclib wrapper" (#2189)

Revert "Custom protoc plugin to generate a grpclib wrapper (#2181)"

This reverts commit 10282b3.

.github/workflows/ci-cd.yml

@@ -88,9 +88,8 @@ jobs:
         run: |
           python -m venv venv
           source venv/bin/activate
-          pip install grpcio-tools==1.59.2 grpclib==0.4.7
+          pip install grpcio-tools==1.59.2
           python -m grpc_tools.protoc --python_out=. --grpclib_python_out=. --grpc_python_out=. -I . modal_proto/api.proto modal_proto/options.proto
-          python -m grpc_tools.protoc --plugin=protoc-gen-modal-grpclib-python=protoc_plugin/plugin.py --modal-grpclib-python_out=. -I . modal_proto/api.proto modal_proto/options.proto
           deactivate
 
       - name: Check entrypoint import

modal/_utils/blob_utils.py

@@ -14,7 +14,7 @@
 from aiohttp.abc import AbstractStreamWriter
 
 from modal_proto import api_pb2
-from modal_proto.modal_api_grpc import ModalClientModal
+from modal_proto.api_grpc import ModalClientStub
 
 from ..exception import ExecutionError
 from .async_utils import TaskContext, retry
@@ -287,7 +287,7 @@ async def _blob_upload(
     return blob_id
 
 
-async def blob_upload(payload: bytes, stub: ModalClientModal) -> str:
+async def blob_upload(payload: bytes, stub: ModalClientStub) -> str:
     if isinstance(payload, str):
         logger.warning("Blob uploading string, not bytes - auto-encoding as utf8")
         payload = payload.encode("utf8")
@@ -296,7 +296,7 @@ async def blob_upload(payload: bytes, stub: ModalClientModal) -> str:
 
 
 async def blob_upload_file(
-    file_obj: BinaryIO, stub: ModalClientModal, progress_report_cb: Optional[Callable] = None
+    file_obj: BinaryIO, stub: ModalClientStub, progress_report_cb: Optional[Callable] = None
 ) -> str:
     upload_hashes = get_upload_hashes(file_obj)
     return await _blob_upload(upload_hashes, file_obj, stub, progress_report_cb)
@@ -316,15 +316,15 @@ async def _download_from_url(download_url: str) -> bytes:
         return await s3_resp.read()
 
 
-async def blob_download(blob_id: str, stub: ModalClientModal) -> bytes:
+async def blob_download(blob_id: str, stub: ModalClientStub) -> bytes:
     # convenience function reading all of the downloaded file into memory
     req = api_pb2.BlobGetRequest(blob_id=blob_id)
     resp = await retry_transient_errors(stub.BlobGet, req)
 
     return await _download_from_url(resp.download_url)
 
 
-async def blob_iter(blob_id: str, stub: ModalClientModal) -> AsyncIterator[bytes]:
+async def blob_iter(blob_id: str, stub: ModalClientStub) -> AsyncIterator[bytes]:
     req = api_pb2.BlobGetRequest(blob_id=blob_id)
     resp = await retry_transient_errors(stub.BlobGet, req)
     download_url = resp.download_url

modal/_utils/grpc_utils.py

@@ -9,21 +9,14 @@
 from typing import (
     Any,
     AsyncIterator,
-    Collection,
     Dict,
-    Generic,
-    Mapping,
     Optional,
-    Tuple,
     TypeVar,
-    Union,
 )
 
 import grpclib.client
 import grpclib.config
 import grpclib.events
-import grpclib.protocol
-import grpclib.stream
 from google.protobuf.message import Message
 from grpclib import GRPCError, Status
 from grpclib.exceptions import StreamTerminatedError
@@ -118,52 +111,11 @@ async def send_request(event: grpclib.events.SendRequest) -> None:
     return channel
 
 
-_Value = Union[str, bytes]
-_MetadataLike = Union[Mapping[str, _Value], Collection[Tuple[str, _Value]]]
-
-
-class UnaryUnaryWrapper(Generic[RequestType, ResponseType]):
-    wrapped_method: grpclib.client.UnaryUnaryMethod[RequestType, ResponseType]
-
-    def __init__(self, wrapped_method: grpclib.client.UnaryUnaryMethod[RequestType, ResponseType]):
-        self.wrapped_method = wrapped_method
-
-    @property
-    def name(self) -> str:
-        return self.wrapped_method.name
-
-    async def __call__(
-        self,
-        req: RequestType,
-        *,
-        timeout: Optional[float] = None,
-        metadata: Optional[_MetadataLike] = None,
-    ) -> ResponseType:
-        # TODO: implement Client tracking and retries
-        return await self.wrapped_method(req, timeout=timeout, metadata=metadata)
-
-
-class UnaryStreamWrapper(Generic[RequestType, ResponseType]):
-    wrapped_method: grpclib.client.UnaryStreamMethod[RequestType, ResponseType]
-
-    def __init__(self, wrapped_method: grpclib.client.UnaryStreamMethod[RequestType, ResponseType]):
-        self.wrapped_method = wrapped_method
-
-    def open(
-        self,
-        *,
-        timeout: Optional[float] = None,
-        metadata: Optional[_MetadataLike] = None,
-    ) -> grpclib.client.Stream[RequestType, ResponseType]:
-        # TODO: implement Client tracking and unary_stream-wrapper
-        return self.wrapped_method.open(timeout=timeout, metadata=metadata)
-
-
 async def unary_stream(
-    method: UnaryStreamWrapper[RequestType, ResponseType],
-    request: RequestType,
+    method: grpclib.client.UnaryStreamMethod[_SendType, _RecvType],
+    request: _SendType,
     metadata: Optional[Any] = None,
-) -> AsyncIterator[ResponseType]:
+) -> AsyncIterator[_RecvType]:
     """Helper for making a unary-streaming gRPC request."""
     async with method.open(metadata=metadata) as stream:
         await stream.send_message(request, end=True)
@@ -172,7 +124,7 @@ async def unary_stream(
 
 
 async def retry_transient_errors(
-    fn: UnaryUnaryWrapper[RequestType, ResponseType],
+    fn: grpclib.client.UnaryUnaryMethod[RequestType, ResponseType],
     *args,
     base_delay: float = 0.1,
     max_delay: float = 1,

modal/client.py

@@ -10,7 +10,7 @@
 from grpclib import GRPCError, Status
 from synchronicity.async_wrap import asynccontextmanager
 
-from modal_proto import api_grpc, api_pb2, modal_api_grpc
+from modal_proto import api_grpc, api_pb2
 from modal_version import __version__
 
 from ._utils import async_utils
@@ -99,11 +99,11 @@ def __init__(
         self.image_builder_version: Optional[str] = None
         self._pre_stop: Optional[Callable[[], Awaitable[None]]] = None
         self._channel: Optional[grpclib.client.Channel] = None
-        self._stub: Optional[modal_api_grpc.ModalClientModal] = None
+        self._stub: Optional[api_grpc.ModalClientStub] = None
         self._snapshotted = False
 
     @property
-    def stub(self) -> modal_api_grpc.ModalClientModal:
+    def stub(self) -> api_grpc.ModalClientStub:
         """mdmd:hidden"""
         assert self._stub
         return self._stub
@@ -125,8 +125,7 @@ async def _open(self):
         assert self._stub is None
         metadata = _get_metadata(self.client_type, self._credentials, self.version)
         self._channel = create_channel(self.server_url, metadata=metadata)
-        grpclib_stub = api_grpc.ModalClientStub(self._channel)
-        self._stub = modal_api_grpc.ModalClientModal(grpclib_stub)
+        self._stub = api_grpc.ModalClientStub(self._channel)  # type: ignore
 
     async def _close(self, prep_for_restore: bool = False):
         if self._pre_stop is not None:

modal/functions.py

@@ -30,8 +30,7 @@
 from synchronicity.combined_types import MethodWithAio
 
 from modal._output import FunctionCreationStatus
-from modal_proto import api_pb2
-from modal_proto.modal_api_grpc import ModalClientModal
+from modal_proto import api_grpc, api_pb2
 
 from ._location import parse_cloud_provider
 from ._output import OutputManager
@@ -100,9 +99,9 @@
 class _Invocation:
     """Internal client representation of a single-input call to a Modal Function or Generator"""
 
-    stub: ModalClientModal
+    stub: api_grpc.ModalClientStub
 
-    def __init__(self, stub: ModalClientModal, function_call_id: str, client: _Client):
+    def __init__(self, stub: api_grpc.ModalClientStub, function_call_id: str, client: _Client):
         self.stub = stub
         self.client = client  # Used by the deserializer.
         self.function_call_id = function_call_id  # TODO: remove and use only input_id

modal_proto/__init__.py

@@ -1 +1 @@
-# Copyright Modal Labs 2024
+# Copyright Modal Labs 2022