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

[YUNIKORN-2087] reinstate node utilisation rest endpoint #145

Closed
wants to merge 1 commit into from

Conversation

wilfred-s
Copy link
Contributor

What is this PR for?

The node utilisation endpoint is required to be exposed for the dashboard to work after YUNIKORN-325
Moving the endpoint to /ws/v1/scheduler/node-utilization The endpoint will need to change as part of YUNIKORN-2088

What type of PR is it?

  • - Bug Fix

What is the Jira issue?

How should this be tested?

Dashboard fails to show without the fix
Deploy YuniKorn from master with the web UI from master needs to include fix from YUNIKORN-2087 also: dashboard works

Screenshots (if appropriate)

NA

The node utilisation endpoint is required to be exposed for the
dashboard to work after YUNIKORN-325
Moving the endpoint to /ws/v1/scheduler/node-utilization
The endpoint will need to change as part of YUNIKORN-2088
@codecov-commenter
Copy link

Codecov Report

Merging #145 (3188adc) into master (d69a737) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #145   +/-   ##
=======================================
  Coverage   66.66%   66.66%           
=======================================
  Files           1        1           
  Lines          30       30           
=======================================
  Hits           20       20           
  Misses          7        7           
  Partials        3        3           

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@wilfred-s wilfred-s closed this in 271bb72 Nov 8, 2023
@wilfred-s wilfred-s deleted the YUNIKORN-2087 branch November 8, 2023 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants