Skip to content

[Feat][Core/Dashboard] Add CPU and memory metrics for subprocesses#52208

Merged
jjyao merged 3 commits intoray-project:masterfrom
MortalHappiness:feature/dashboard-subprocess-cpu-ram-metrics
Apr 11, 2025
Merged

[Feat][Core/Dashboard] Add CPU and memory metrics for subprocesses#52208
jjyao merged 3 commits intoray-project:masterfrom
MortalHappiness:feature/dashboard-subprocess-cpu-ram-metrics

Conversation

@MortalHappiness
Copy link
Copy Markdown
Member

@MortalHappiness MortalHappiness commented Apr 10, 2025

Why are these changes needed?

Emit CPU and memory metrics for dashboard subprocess modules.

image

Related issue number

N/A

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Signed-off-by: Chi-Sheng Liu <chishengliu@chishengliu.com>
@MortalHappiness MortalHappiness added the go add ONLY when ready to merge, run all tests label Apr 10, 2025
@MortalHappiness MortalHappiness marked this pull request as ready for review April 10, 2025 13:02
Signed-off-by: Chi-Sheng Liu <chishengliu@chishengliu.com>
@jjyao jjyao merged commit fe8b8b7 into ray-project:master Apr 11, 2025
5 checks passed
han-steve pushed a commit to han-steve/ray that referenced this pull request Apr 11, 2025
…ay-project#52208)

Signed-off-by: Chi-Sheng Liu <chishengliu@chishengliu.com>
Signed-off-by: Steve Han <stevehan2001@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-backlog go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants