-
Notifications
You must be signed in to change notification settings - Fork 432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Release][Doc] Add instructions to release Go modules. #1546
Conversation
cc @pingsutw |
Thank you so much! |
@@ -74,6 +74,27 @@ You will prompted for a commit reference and an image tag. The commit reference | |||
git push upstream v0.5.0-rc.0 | |||
``` | |||
|
|||
* Tag the tip of the release branch as `ray-operator/vX.Y.Z-rc.0`. Without this tag, users won't be able to use `github.com/ray-project/kuberay/[email protected]` to install the Go module. KubeRay starts supporting Go modules from v0.6.0. | |||
```sh | |||
git tag ray-operator/v0.5.0-rc.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can just add it to the CI. https://github.com/ray-project/kuberay/blob/master/.github/workflows/image-release.yaml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Open an issue: #1549
Why are these changes needed?
Some KubeRay users, like Flyte, want to use the generated Golang client by
go install github.com/ray-project/kuberay/[email protected]
. However, there is nogo.mod
in the root directory of KubeRay repo, so users cannot do that until we provide a specific tag forray-operator/
directory.Related issue number
Checks