Skip to content

Conversation

@teojgo
Copy link
Contributor

@teojgo teojgo commented Feb 11, 2019

  • Flexible node allocations take into account only nodes belonging to
    the default partition if a partition is not specified.

Closes #663

* Flexible node allocations take into account only nodes belonging to
  the `default` partition if a partition is not specified.
@teojgo teojgo added this to the Upcoming sprint milestone Feb 11, 2019
@teojgo teojgo self-assigned this Feb 11, 2019
@teojgo teojgo requested review from victorusu and vkarak February 11, 2019 09:06
@codecov-io
Copy link

codecov-io commented Feb 11, 2019

Codecov Report

Merging #676 into master will decrease coverage by 0.02%.
The diff coverage is 58.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #676      +/-   ##
==========================================
- Coverage   91.85%   91.83%   -0.03%     
==========================================
  Files          77       77              
  Lines        9445     9455      +10     
==========================================
+ Hits         8676     8683       +7     
- Misses        769      772       +3
Impacted Files Coverage Δ
unittests/test_schedulers.py 97.63% <100%> (+0.01%) ⬆️
reframe/core/schedulers/slurm.py 60% <50%> (-0.67%) ⬇️
reframe/core/config.py 84.48% <0%> (+1.72%) ⬆️

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 d632088...df26b2c. Read the comment docs.

@vkarak vkarak changed the title [bugfix] Use the 'default' partition with 'flex-alloc-tasks' [bugfix] Use the default Slurm partition with --flex-alloc-tasks if none is specified Feb 15, 2019
@vkarak vkarak merged commit f701ca5 into reframe-hpc:master Feb 19, 2019
@teojgo teojgo added this to the ReFrame sprint 2019w07 milestone Feb 25, 2019
@teojgo teojgo deleted the bugfix/flex_alloc_default_partition branch March 13, 2019 14:13
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