Skip to content

fix ray-service.different-port.yaml#3721

Merged
kevin85421 merged 4 commits into
ray-project:masterfrom
zjx20:patch-1
Jun 11, 2025
Merged

fix ray-service.different-port.yaml#3721
kevin85421 merged 4 commits into
ray-project:masterfrom
zjx20:patch-1

Conversation

@zjx20
Copy link
Copy Markdown
Contributor

@zjx20 zjx20 commented May 31, 2025

Why are these changes needed?

#3262 cleaned up the ports field in the YAML files. However, the ray-service.different-port.yaml should at least retain the configuration for the serve port, as this YAML is specifically demonstrating how to customize the serve port.

Related issue number

Checks

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

zjx20 added 3 commits May 31, 2025 12:13
Signed-off-by: zjx20 <zhoujianxiong2@gmail.com>
Signed-off-by: zjx20 <zhoujianxiong2@gmail.com>
Signed-off-by: zjx20 <zhoujianxiong2@gmail.com>
@zjx20
Copy link
Copy Markdown
Contributor Author

zjx20 commented Jun 4, 2025

Hi @MortalHappiness, could you take a look?

@MortalHappiness
Copy link
Copy Markdown
Member

Thanks for your contribution!

Signed-off-by: zjx20 <zhoujianxiong2@gmail.com>
@zjx20
Copy link
Copy Markdown
Contributor Author

zjx20 commented Jun 11, 2025

Hello, if there are no further issues, please consider merging this PR :) @MortalHappiness @kevin85421

@kevin85421
Copy link
Copy Markdown
Member

Oh, I didn’t realize this was a bug fix. We’re currently working on the release and we’ve already cut the branch, so I haven’t been reviewing new PRs lately. I will merge this PR and cherry pick to the release branch.

@kevin85421 kevin85421 merged commit 50678f6 into ray-project:master Jun 11, 2025
25 checks passed
MortalHappiness pushed a commit to MortalHappiness/kuberay that referenced this pull request Jun 11, 2025
chipspeak pushed a commit to chipspeak/kuberay that referenced this pull request Jul 2, 2025
kryanbeane pushed a commit to kryanbeane/kuberay that referenced this pull request Jul 2, 2025
CathalOConnorRH pushed a commit to CathalOConnorRH/kuberay that referenced this pull request Oct 2, 2025
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