Skip to content

Remove Slurm reservation check from serve_overview

Thomas Kluyver requested to merge serve-overview-rm-reservation into master

Description

We're no longer using a Slurm reservation for calibration jobs - we have partitions (upex-high & upex-middle) to give calibration priority instead. This removes some unnecessary work in serve_overview to check how many nodes are in a reservation and recommend whether to use it or not.

I've left in code to check for upex_* reservations for now, but I don't think we use those either. We can simplify the code more if we ignore those as well.

How Has This Been Tested?

Installed on max-exfl017, checked that the http://max-exfl017.desy.de:8008 loads OK:

image

Types of changes

  • Removal of feature no longer needed

Checklist:

  • My code follows the code style of this project.

Reviewers

@danilevc @roscar

Edited by Thomas Kluyver

Merge request reports