Skip to content
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

Helm: updated local ray charts by the official ones #107

Merged
merged 3 commits into from
Jan 12, 2023
Merged

Helm: updated local ray charts by the official ones #107

merged 3 commits into from
Jan 12, 2023

Conversation

Tansito
Copy link
Member

@Tansito Tansito commented Jan 12, 2023

Summary

This PR replaces the current Ray chart configuration for the officials Charts published by Ray in its GitHub project: kuberay-helm.

Details and comments

Fix #99

The project is using currently: kuberay-operator and ray-cluster. For these charts we configured the last version: 0.4.0. Apart from that there is no special changes in the configuration:

  • I just updated our values.yaml with the last configuration that Ray provides.
  • Updated the documentation with references to the Ray documentation.

@Tansito Tansito requested a review from IceKhan13 January 12, 2023 13:32
@Tansito Tansito requested a review from IceKhan13 January 12, 2023 15:41
Copy link
Member

@IceKhan13 IceKhan13 left a comment

Choose a reason for hiding this comment

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

Great!

I can even feel how it became easier to breath :) Our repo is so much more lightweight now

@Tansito Tansito merged commit 98ee32a into main Jan 12, 2023
@Tansito Tansito deleted the ray-helm branch January 12, 2023 15:58
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.

Helm: remove ray charts in favor of artifactory
2 participants