[Feature] Why doesn’t serveConfigV2 support file inputs
Search before asking
- [x] I had searched in the issues and found no similar feature requirement.
Description
Could someone explain the design decision behind this limitation? thanks
Use case
No response
Related issues
No response
Are you willing to submit a PR?
- [ ] Yes I am willing to submit a PR!
- what do you mean file inputs? can you give an example?
spec: serveConfigV2: application.yaml
like this
RayService’s serveConfigV2 currently only accepts inline YAML strings, for the following reasons:
-
In the CRD, serveConfigV2 is defined as a string, which means it is designed to contain the full YAML content directly. https://github.com/ray-project/kuberay/blob/aa8d8301ce0526f07d03691a3edd64b7d7c07bd1/ray-operator/apis/ray/v1/rayservice_types.go#L105-L109
-
The controller simply treats this string as YAML and unmarshals it.
https://github.com/ray-project/kuberay/blob/aa8d8301ce0526f07d03691a3edd64b7d7c07bd1/ray-operator/controllers/ray/rayservice_controller.go#L1233-L1240
To support file-based inputs, maybe we can add a serveConfigV2From field to load the YAML from a referenced ConfigMap.
this is worth discussing, will put it in my topic to committers next time we have meeting, thank you @win5923