Skip to content

Conversation

@can-anyscale
Copy link
Contributor

@can-anyscale can-anyscale commented Nov 16, 2023

We need to build release test images for GCP as well. Add support for that. Currently GCP tests cannot run on PR because of this.

Test:

@can-anyscale can-anyscale force-pushed the can-rel-01 branch 4 times, most recently from 36a6979 to 1edd73b Compare November 17, 2023 02:08
@can-anyscale can-anyscale marked this pull request as ready for review November 17, 2023 15:40
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we make this also controllable by the env var please?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

totally

@can-anyscale
Copy link
Contributor Author

Make the gcp registry configurable through a env var. There is a yaml config that also contains this value, to make things configurable in another repo. However, the anyscale build code is this repo specific so I keep things simple for now. This might change on further unification.

@can-anyscale can-anyscale requested a review from a team as a code owner November 18, 2023 00:08
@can-anyscale can-anyscale merged commit 709dc1b into master Nov 18, 2023
@can-anyscale can-anyscale deleted the can-rel-01 branch November 18, 2023 00:18
liuxsh9 pushed a commit to liuxsh9/ray that referenced this pull request Nov 18, 2023
We need to build release test images for GCP as well. Add support for that. Currently GCP tests cannot run on PR because of this.

Signed-off-by: can <[email protected]>
Signed-off-by: Xiaoshuang Liu <[email protected]>
ujjawal-khare pushed a commit to ujjawal-khare-27/ray that referenced this pull request Nov 29, 2023
We need to build release test images for GCP as well. Add support for that. Currently GCP tests cannot run on PR because of this.

Signed-off-by: can <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants