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

Ray Integration to Stacks #52

Merged
merged 5 commits into from
Feb 12, 2025
Merged

Ray Integration to Stacks #52

merged 5 commits into from
Feb 12, 2025

Conversation

elamaran11
Copy link
Contributor

This PR adds a new RAY Stack integration to Stacks. This PR includes the following:

Ray Operator CRDS
Ray Operator
Backstage Manifests to Ray Serve ML thru Backstage templates
ReadMe with instructions

Copy link

@punkwalker punkwalker left a comment

Choose a reason for hiding this comment

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

Thanks @elamaran11 for this stack. I will wait for the requested changes and then approve it.

```bash
idpbuilder create \
--use-path-routing \
--p https://github.com/cnoe-io/stacks//ray-integration

Choose a reason for hiding this comment

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

The shorthand flag has to be -p and longhand is --package

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

```bash
idpbuilder create \
--use-path-routing \
--p https://github.com/cnoe-io/stacks//ref-implementation

Choose a reason for hiding this comment

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

This command is missing multiline command format

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@@ -0,0 +1,95 @@
# Make sure to increase resource requests and limits before using this example in production.
# For examples with more realistic resource configuration, see
# ray-cluster.complete.large.yaml and

Choose a reason for hiding this comment

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

do we have references to these files?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No longer required, references are removed.

Comment on lines 1 to 17
apiVersion: backstage.io/v1alpha1
kind: Location
metadata:
name: basic-example-templates
description: A collection of example templates
spec:
targets:
- ./ray-serve/template-ray-serve.yaml
---
apiVersion: backstage.io/v1alpha1
kind: Location
metadata:
name: basic-organization
description: Basic organization data
spec:
targets:
- ./organization/guests.yaml

Choose a reason for hiding this comment

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

we should template-ray-serve.yaml to : https://github.com/cnoe-io/stacks/tree/main/ref-implementation/backstage-templates/entities

OR

Remove basic-organization location and instructions to Register an existing component using catalog-info.yaml from this stack.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.


train_config = {
"lr": 1e-3,
"epochs": 1, # artifically set low to finish quickly

Choose a reason for hiding this comment

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

nit: artifically ==> artificially

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

Signed-off-by: Elamaran Shanmugam <[email protected]>
Signed-off-by: Elamaran Shanmugam <[email protected]>
Signed-off-by: Elamaran Shanmugam <[email protected]>
@elamaran11
Copy link
Contributor Author

@punkwalker All required changes and comments are incorporated. Please review and merge.

@punkwalker punkwalker merged commit 7c10de9 into cnoe-io:main Feb 12, 2025
2 checks passed
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.

2 participants