Skip to content

Conversation

@lucamar
Copy link
Contributor

@lucamar lucamar commented Mar 25, 2020

Removing kesch:pn from FlexAlltoallTest, as suggested by @vkarak in https://jira.cscs.ch/browse/UES-740.
Please note that if we want to keep it, I can just export the correct path for MV2_USE_GPUDIRECT_GDRCOPY in the check, as I have reported in https://jira.cscs.ch/browse/UES-533.
As a matter of fact, daint:mc, dom:mc, arolla:pn and tsa:pn are also listed in the check.

@pep8speaks
Copy link

pep8speaks commented Mar 25, 2020

Hello @lucamar, Thank you for updating!

Cheers! There are no PEP8 issues in this Pull Request!Do see the ReFrame Coding Style Guide

Comment last updated at 2020-03-25 09:20:59 UTC

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 @lucamar for the PR. I would recommend to remove also tsa:pn and arolla:pn from the supported systems. @kraushm What do you think?

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.

As discussed in UES-740, we will keep the tsa:pn and arolla:pn.

@kraushm
Copy link
Contributor

kraushm commented Mar 25, 2020

I just ran the test on the RH7.7 partition (although with limited resources) in order to validate, whether the modified setup of the software stack (CUDA - non-CUDA) would have any impact. It passed, hence it seems ok from that perspective to keep the tests for tsa:pn and arolla:pn.

@vkarak vkarak changed the title [test] Removing kesch:pn from FlexAlltoallTest [test] Remove kesch:pn from supported systems of FlexAlltoallTest Mar 25, 2020
@vkarak vkarak merged commit 7fe7bec into reframe-hpc:master Mar 25, 2020
@lucamar lucamar deleted the flexalltoall branch March 25, 2020 19:54
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.

4 participants