Skip to content

Conversation

@teojgo
Copy link
Contributor

@teojgo teojgo commented Nov 2, 2018

  • Add the -a option when performing scontrol show to list hidden
    partitions as well.

This makes it possible to filter nodes with respect to the hidden partitions like cn-regression.

* Add the `-a` option when performing `scontrol show` to list hidden
  partitions as well.
@teojgo teojgo added this to the ReFrame sprint 2018w44 milestone Nov 2, 2018
@teojgo teojgo self-assigned this Nov 2, 2018
@teojgo teojgo requested a review from vkarak November 2, 2018 12:47
@codecov-io
Copy link

codecov-io commented Nov 2, 2018

Codecov Report

Merging #553 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #553   +/-   ##
=======================================
  Coverage   91.56%   91.56%           
=======================================
  Files          72       72           
  Lines        8954     8954           
=======================================
  Hits         8199     8199           
  Misses        755      755
Impacted Files Coverage Δ
reframe/core/schedulers/slurm.py 60.61% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bee39aa...15171e3. Read the comment docs.

@vkarak vkarak changed the title [bugfix] Show also hidden partition in slurm [bugfix] Show also hidden partitions in Slurm, when trying to flexibly allocate tasks Nov 2, 2018
Copy link
Contributor

@vkarak vkarak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@vkarak vkarak merged commit e6f6083 into reframe-hpc:master Nov 2, 2018
@teojgo teojgo deleted the bugfix/slurm_show_hidden_partitions branch November 22, 2018 08:45
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.

3 participants