[Bugfix] to shared storage not working with dynamic PV provisioning (#933)

* helm(shared-storage): only create PV when hostPath/NFS provided and no storageClass; set PVC.volumeName then

Signed-off-by: Niccolo Tosato <niccolo.tosato@yahoo.it>

* helm(config): set HF_HOME to shared mount when sharedPvcStorage.enabled

Signed-off-by: Niccolo Tosato <niccolo.tosato@yahoo.it>

* Remove fix from raycluster

Signed-off-by: Niccolo Tosato <niccolo.tosato@yahoo.it>

* Fix discussion

Signed-off-by: Niccolo Tosato <niccolo.tosato@yahoo.it>

---------

Signed-off-by: Niccolo Tosato <niccolo.tosato@yahoo.it>
Co-authored-by: Rui Zhang <51696593+ruizhang0101@users.noreply.github.com>

Author: NiccoloTosato

helm/templates/deployment-vllm-multi.yaml

@@ -214,7 +214,9 @@ spec:
           - name: PYTHONHASHSEED
             value: "123"
           - name: HF_HOME
-            {{- if hasKey $modelSpec "pvcStorage" }}
+            {{- if .Values.sharedPvcStorage.enabled }}
+            value: /data/shared-pvc-storage
+            {{- else if hasKey $modelSpec "pvcStorage" }}
             value: /data
             {{- else }}
             value: /tmp