Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Doc] Add a Doc and YAML for RayServe grpc support #1915

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

evalaiyc98
Copy link
Contributor

Why are these changes needed?

This is an example of setting up gRPC on KubeRay, and this example is consistent with that on Ray Serve.

The following are the steps:

  1. Use the command python -m grpc_tools.protoc -I./protos --python_out=. --pyi_out=. --grpc_python_out=. ./protos/user_defined_protos.proto which is provided in the proto file to generate the Python code.
    This command would generate three files, including user_defined_protos_pb2.py, user_defined_protos_pb2.pyi, user_defined_protos_pb2_grpc.py.

  2. Use the YAML file ray-serve-grpc.yaml to create the RayService CR.

  3. Use client.py to test whether the gRPC works well or not.

Related issue number

#1555

Checks

The screenshot of the result:

image

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant