Skip to content

Conversation

@jgphpc
Copy link
Contributor

@jgphpc jgphpc commented Mar 30, 2020

Will fix UES-721

@jgphpc jgphpc requested a review from kraushm March 30, 2020 09:36
@jgphpc jgphpc self-assigned this Mar 30, 2020
@jgphpc jgphpc requested a review from vkarak March 30, 2020 09:36
Copy link
Contributor

@kraushm kraushm left a comment

Choose a reason for hiding this comment

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

lgtm.

@jgphpc
Copy link
Contributor Author

jgphpc commented Mar 30, 2020

It fails because the fix is not on daint (yet). What's recommended here: exclude it from jenkins with-x NvprofCheck or add some python in the check ?

@teojgo
Copy link
Contributor

teojgo commented Mar 30, 2020

@jgphpc we can exclude it from the daily runs.

@vkarak vkarak added this to the ReFrame sprint 20.05 milestone Mar 30, 2020
@vkarak vkarak merged commit 862c55d into reframe-hpc:master Mar 31, 2020
@jgphpc jgphpc deleted the UES-721_nvprof branch April 1, 2020 07:44
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