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

Update DCM scheduler #339

Merged
merged 9 commits into from
Oct 4, 2023
Merged

Update DCM scheduler #339

merged 9 commits into from
Oct 4, 2023

Conversation

hunhoffe
Copy link
Collaborator

@hunhoffe hunhoffe commented Sep 29, 2023

This PR:

  • Cleans up RPC library (remove overly verbose code in preparation to rework portions of the library)
  • Reduces RPC header size (required updating nrk-dcm-scheduler)
  • Adds support for different scheduler backends in integration tests (required updating nrk-dcm-scheduler)

This version of nrk-dcm-scheduler supports the following backends:

  • R (random scheduler)
  • RR (round robin scheduler)
  • LF (local-fill scheduler, previously called sticky scheduler)
  • DCMcap (DCM with only capacity policy - this performs poorly due to being under-constrained)
  • DCMloc (DCM with cap + locality policy - includes upgrade locality policy from previous version of DCM.

The dcm runner in the nrk integration tests is able to take an argument to specify the scheduler backend.

@hunhoffe hunhoffe changed the title Dev/rackscale dcm update Update DCM scheduler Sep 29, 2023
@hunhoffe hunhoffe marked this pull request as ready for review October 1, 2023 23:24
@hunhoffe hunhoffe merged commit 138fd92 into master Oct 4, 2023
10 checks passed
@hunhoffe hunhoffe deleted the dev/rackscale-dcm-update branch October 4, 2023 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants