Requesting changes as this is a breaking change. We should add a deprecation warning for a couple of releases before removing the optional dependency, since it breaks --distributed-executor-backend ray
Let's additionally discuss the deprecation in an RFC first + with the community
Originally posted by @tlrmchlsmth in #33351 (review)
vLLM v1 PP can run via the multiprocessing backend; Ray is only required when users explicitly choose the Ray executor backend. Keeping Ray as a default dependency on CUDA/ROCm causes confusion and unnecessarily pulls Ray into environments that don’t use it.
Originally posted by @tlrmchlsmth in #33351 (review)
vLLM v1 PP can run via the multiprocessing backend; Ray is only required when users explicitly choose the Ray executor backend. Keeping Ray as a default dependency on CUDA/ROCm causes confusion and unnecessarily pulls Ray into environments that don’t use it.