Ryan O'Leary

Results 63 comments of Ryan O'Leary

This PR was manually tested as follows: ### Prerequisites: 1. GKE cluster with TPU quota and Node Autoprovisioning enabled, or a v4 2x2x2 TPU nodepool already created. 2. [Ray TPU...

> Could you share more details about the detached actor and add more details about why you expect the cluster to look like this at each step? Sure, I edited...

> ![image](https://private-user-images.githubusercontent.com/20109646/351971747-a34d1c06-beaa-48d1-a1e8-4f3c585527d9.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE4ODQ2ODMsIm5iZiI6MTcyMTg4NDM4MywicGF0aCI6Ii8yMDEwOTY0Ni8zNTE5NzE3NDctYTM0ZDFjMDYtYmVhYS00OGQxLWExZTgtNGYzYzU4NTUyN2Q5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI1VDA1MTMwM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWU1MDRmM2RhY2QyMTMzNjhiY2QwNjhhNTZlMTQxODljZjBlMzFlMzVmZTE1YzQ5ZWYzOGM0NmRmMDAwN2NlNWQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.ounCrFWndeX7H6QI1Q2BFyb2KMH7lUgAqvbttOWdsxQ) > > @ryanaoleary can you fix the CI error? Fixed in be33993. ![image](https://github.com/user-attachments/assets/3b7a4a72-d156-4435-a7cc-55e0203b394d)

> 2\. PodSlice I switched all usages of pod -> Pod, but I think we should keep the naming as "Pod slice" rather than "PodSlice" since that's what the GKE...

> @ryanaoleary would you mind fixing the CI error? Thanks! Can you paste the CI error? I'm not able to see it.

I just pulled in the latest changes, it looks like the build for [e2fe072](https://github.com/ray-project/ray/pull/45924/commits/e2fe0720674aca7f7ce0ba09c0eaeb5a510c7168) was passing the CI @angelinalg

> Do we need a RayService YAML in the repo with region tags that you can reference in the GCP docs? Yeah that sounds good. I'm still testing out the...