Skip to content

Commit

Permalink
Merge pull request #739 from sebkelle1/checks/affinity
Browse files Browse the repository at this point in the history
[test] Sort cores list in affinity test before comparing
  • Loading branch information
vkarak committed Apr 6, 2019
2 parents dd4d6ba + 9d4213f commit 3ad7afc
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions cscs-checks/prgenv/affinity_check.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,12 @@ def __init__(self, variant):
'dom:gpu', 'dom:mc']
self.valid_prog_environs = ['PrgEnv-gnu']
self.build_system = 'Make'
self.build_system.options = ['--directory=affinity', 'MPI=1']
self.build_system.options = ['-C affinity', 'MPI=1']
# The github URL can not be specifid as `self.sourcedir` as that
# would prevent the src folder from being copied to stage which is
# necessary since these tests need files from it.
self.prebuild_cmd = ['git clone https://github.com/vkarak/affinity']
self.executable = 'affinity/affinity'
self.executable = './affinity/affinity'
self.variant = variant
self.maintainers = ['RS', 'VK']
self.tags = {'production', 'scs', 'maintenance'}
Expand Down Expand Up @@ -50,7 +50,8 @@ def parse_cpus(x):
self.sanity_patterns = sn.all([
sn.assert_eq(self.aff_thrds, self.ref_thrds),
sn.assert_eq(self.aff_ranks, self.ref_ranks),
sn.assert_eq(self.aff_cores, self.ref_cores)])
sn.assert_eq(sn.sorted(self.aff_cores), sn.sorted(self.ref_cores))
])

super().setup(partition, environ, **job_opts)

Expand Down

0 comments on commit 3ad7afc

Please sign in to comment.