Ensure all temp files are deleted after the compatibility test

[Yicheng-Lu-llll]

Why are these changes needed?

1. This PR ensures all temp files are removed after the compatibility tests are completed. In the current implementation, some temporary files will not be deleted after the process exits due to:

   kuberay/tests/kuberay_utils/utils.py

   Line 44 in 683f0fa

   with tempfile.NamedTemporaryFile('w', delete=False) as ray_cluster_yaml:

   With the option delete=False, the temporary files will not be deleted after the process exits, and need to be deleted manually. See NamedTemporaryFile for more details. Check /tmp to see the temporary files after the tests are completed.

   In this PR, temporary files will be deleted automatically once we close the files. See how NamedTemporaryFile works.

   with tempfile.NamedTemporaryFile('w', suffix = '_ray_cluster_yaml') as ray_cluster_yaml:
           ray_cluster_yaml.write(yamlfile)
           ray_cluster_yaml.flush()

• Note: We need to add a flush manually. Otherwise, the file will be empty when we try to read it because the flush normally does not happen until the file is closed.

2. This PR removes redundant try/catch. RayServiceAddCREvent and RayClusterAddCREvent already have the logic to raise and log errors.

Related issue number

Checks

Make sure the compatibility tests still work. This can be proven through the CI test.

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

[kevin85421]

LGTM

[kevin85421]

The CI failure is due to #852.