Skip to content

Conversation

@bcfriesen
Copy link
Contributor

ReFrame 3 upstreamed support for UPC and UPC++ launchers from Berkeley Lab. This PR fixes a missed change to the launcher schemes. Without this, tests which request the upcrun or upcxx-run launchers will fail because ReFrame cannot find them in this list.

@jenkins-cscs
Copy link
Collaborator

Can I test this patch?

@codecov-commenter
Copy link

codecov-commenter commented Jun 28, 2020

Codecov Report

Merging #1393 into master will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1393      +/-   ##
==========================================
+ Coverage   91.66%   91.75%   +0.08%     
==========================================
  Files          83       83              
  Lines       12673    12712      +39     
==========================================
+ Hits        11617    11664      +47     
+ Misses       1056     1048       -8     
Impacted Files Coverage Δ
reframe/core/meta.py 96.77% <0.00%> (-3.23%) ⬇️
reframe/core/schedulers/__init__.py 98.83% <0.00%> (ø)
reframe/frontend/cli.py 78.15% <0.00%> (+0.17%) ⬆️
reframe/core/schedulers/slurm.py 57.54% <0.00%> (+1.72%) ⬆️
unittests/test_schedulers.py 93.82% <0.00%> (+2.00%) ⬆️

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 ec92346...558bdfc. Read the comment docs.

@vkarak vkarak changed the title schemas: add missing launchers upcrun and upcxx-run [bugfix] Add missing launchers upcrun and upcxx-run in configuration schema Jun 28, 2020
@vkarak vkarak self-requested a review June 28, 2020 08:27
@vkarak vkarak self-assigned this Jun 28, 2020
@vkarak vkarak added this to the ReFrame sprint 20.10 milestone Jun 28, 2020
@vkarak vkarak requested a review from ekouts June 28, 2020 08:29
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.

Thanks @bcfriesen for the fix!

@vkarak
Copy link
Contributor

vkarak commented Jun 28, 2020

ok to test

@vkarak vkarak merged commit 37cbafc into reframe-hpc:master Jun 30, 2020
@bcfriesen bcfriesen deleted the bcf/fix-upc-launcher-schema branch August 5, 2020 03:38
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.

5 participants