Skip to content

[History Server] Replace "runtime class name" flag and env var with "storage backend" - #5085

Open
machichima wants to merge 2 commits into
ray-project:masterfrom
machichima:remove-runtime-class-name
Open

[History Server] Replace "runtime class name" flag and env var with "storage backend"#5085
machichima wants to merge 2 commits into
ray-project:masterfrom
machichima:remove-runtime-class-name

Conversation

@machichima

@machichima machichima commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Why are these changes needed?

We currently have both "runtime class name" (old one) and "storage backend" referring to the name of the storage backend we used. This PR removes the old "runtime class name".

This PR do following changes across code / docs / YAMLs:

  1. Rename --runtime-class-name flag to --storage-backend
  2. Remove RUNTIME_CLASS_NAME env var
  3. Rename --runtime-class-config-path flag to --storage-backend-config-path

Related issue number

Related to ray-project/ray#65139 (comment)

Labels

  • If this PR has user-facing changes that require documentation updates at release time, I have added the doc-updates-required label.
  • If this PR contains breaking changes, I have added the breaking-change label.

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

Manual test instructions

…ckend

Signed-off-by: machichima <nary12321@gmail.com>

@win5923 win5923 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment thread historyserver/cmd/collector/main.go Outdated

@win5923 win5923 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

cc @chiayi

@chiayi chiayi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! I think this can also be merged in first before #5080 or which ever is ready first.

@machichima
machichima force-pushed the remove-runtime-class-name branch from 42c7867 to cd6ccf2 Compare August 7, 2026 02:59
@machichima

Copy link
Copy Markdown
Collaborator Author

cc @rueian or @andrewsykim to merge when you have time. Thanks!

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit cd6ccf2. Configure here.

Comment thread historyserver/cmd/collector/main.go
…untime-class-name

Signed-off-by: machichima <nary12321@gmail.com>
@machichima
machichima force-pushed the remove-runtime-class-name branch from cd6ccf2 to 9636f35 Compare August 7, 2026 05:39
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.

3 participants