From ef21ed5259c787b3993732df78e9d4c81b042f56 Mon Sep 17 00:00:00 2001 From: Satish Kamath Date: Thu, 6 Jun 2024 23:59:19 +0200 Subject: [PATCH] Using mem_required_per_node from the hooks. Tested on Snellius and it works properly. --- eessi/testsuite/tests/apps/espresso/espresso.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/eessi/testsuite/tests/apps/espresso/espresso.py b/eessi/testsuite/tests/apps/espresso/espresso.py index 9fbccf58..2fbb5ce3 100644 --- a/eessi/testsuite/tests/apps/espresso/espresso.py +++ b/eessi/testsuite/tests/apps/espresso/espresso.py @@ -83,8 +83,9 @@ def set_num_tasks_per_node(self): def set_mem(self): """ Setting an extra job option of memory. Here the assumption made is that HPC systems will contain at least 1 GB per core of memory.""" - mem_required_per_node = str(self.num_tasks_per_node * 1) + 'GB' - self.extra_resources = {'memory': {'size': mem_required_per_node}} + mem_required_per_node = self.num_tasks_per_node * 0.9 + hooks.req_memory_per_node(test=self, app_mem_req=mem_required_per_node) + @deferrable def assert_completion(self):