[Refactor] Remove cleanupInvalidVolumeMounts

[kevin85421]

Why are these changes needed?

cleanupInvalidVolumeMounts verifies and modifies the Pod spec directly which may introduce some unknown unknowns for users. KubeRay should only verify the RayCluster, RayJob, and RayCluster specs. For other resource specs, such as Kubernetes Pods, Jobs, and Services, we should not verify them; instead, we should only propagate the errors.

Related issue number

Checks

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

• Step 1: Create a RayCluster with https://gist.github.com/kevin85421/22c0f1792fe475bb80c3ec7a3180915a. The Ray container in the head Pod mounts a non-existent volume.

  volumeMounts:
    - mountPath: /home/ray/samples
      name: ray-example-configmap
    - mountPath: /home/ray/
      name: not-exist

• Step 2: We can find the error by running kubectl describe raycluster xxxx.
  

[kevin85421]

I can't understand the logic to compare the number of init containers and RayContainerIndex.