-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat(blueprint): Ray Blueprint #871
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
askulkarni2
requested review from
kcoleman731,
vara-bonthu,
bryantbiggs and
Zvikan
as code owners
August 19, 2022 07:07
…ks-blueprints into feature-ray-addon
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
reviewed
Aug 21, 2022
bryantbiggs
approved these changes
Aug 22, 2022
Are there any operational, observability, and security considerations to add to the README ? |
Yep, please see https://github.com/aws-ia/terraform-aws-eks-blueprints/tree/feature-ray-addon/examples/ai-ml/ray#monitoring |
allamand
pushed a commit
to allamand/terraform-aws-eks-blueprints
that referenced
this pull request
Dec 15, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR adds an experimental blueprint for running Ray Clusters using the KubeRay operator which is provided as a new add-on.
Fixes #491
Motivation
Ray is a popular AI/ML framework for running distributed ML training and inference. Customers who wish to deploy Ray Clusters on EKS can use this example as a reference implementation. The example shows how Ray Clusters can be deployed along with Ingress for model serving as well as a sample monitoring stack using kube-prometheus-stack. Over time we will work towards adding more robustness to the Ray cluster configuration and features such as autoscaling, multi-tenancy, etc.
More
pre-commit run -a
with this PRNote: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators
Additional Notes
The repo local helm chart for kuberay-operator is temporary as the upstream project is still working on having the helm chart published. See ray-project/kuberay#475