Skip to content

Add hyp create ray-dashboard-connection command - #446

Open
jchatter321 wants to merge 1 commit into
aws:mainfrom
jchatter321:add-hyp-create-ray-dashboard-connection-command
Open

Add hyp create ray-dashboard-connection command#446
jchatter321 wants to merge 1 commit into
aws:mainfrom
jchatter321:add-hyp-create-ray-dashboard-connection-command

Conversation

@jchatter321

@jchatter321 jchatter321 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Adds a CLI command to create a RayDashboardConnection CR and return the dashboard URL.

Usage:

hyp create ray-dashboard-connection --cluster-name <name> --namespace <ns>

Returns the connection URL from the operator's extension API. Flag --namespace defaults to default namespace if not provided.

Testing

Installed CLI with pip install -e . and successfully generated a RayDashboardConnection using hyp create ray-dashboard-connection --cluster-name my-raycluster.

@jchatter321
jchatter321 requested a review from a team as a code owner August 11, 2026 16:57
if connection_url:
click.echo(connection_url)
else:
click.echo(f"RayDashboardConnection created in namespace '{namespace}' for cluster '{cluster_name}'")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This can never happen because agg APIs are sychronous. Lets fail with an error message 'contact your administrator' or something when URL is null / empty.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added exception.

"""Create a RayDashboardConnection to get a dashboard URL for a RayCluster."""
api_client = _get_eks_api_client()

body = {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can we handle the case where the ray endpoint operator is not installed gracefully? If kind RayDashboardConnection is not recognized by the API server, lets instruct customers to install ray-endpoint-operator (we'll come back and link the docs when it's ready)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added exception below.

@jchatter321
jchatter321 force-pushed the add-hyp-create-ray-dashboard-connection-command branch from bebb5de to b4aeb21 Compare August 11, 2026 17:25
@jchatter321
jchatter321 force-pushed the add-hyp-create-ray-dashboard-connection-command branch from b4aeb21 to 4281c4a Compare August 11, 2026 17:29
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