address reviewer comments

neutrons · Sep 6, 2024 · 92d8dca · 92d8dca
1 parent 13603bb
commit 92d8dca
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 4 deletions.
diff --git a/Dockerfile.autoreducer b/Dockerfile.autoreducer
@@ -3,7 +3,7 @@ FROM registry.access.redhat.com/ubi9/ubi
 # install various dependencies
 RUN dnf install -y https://dl.fedoraproject.org/pub/epel/epel-release-latest-9.noarch.rpm
 RUN dnf updateinfo
-RUN dnf install -y procps-ng numpy # pgrep is used for health check, numpy for high memory test
+RUN dnf install -y procps-ng python3-numpy # pgrep is used for health check, numpy for high memory test
 
 # Inspection tools
 RUN dnf install -y vim

diff --git a/tests/data/PG3/shared/autoreduce/reduce_PG3.py b/tests/data/PG3/shared/autoreduce/reduce_PG3.py
@@ -6,5 +6,6 @@
 
 print("Running reduction for " + sys.argv[1] + " at " + datetime.isoformat(datetime.now()))
 
+# intentionally take up a lot of memory (used to test job memory monitoring)
 _ = np.random.rand(100000, 10000)
 time.sleep(5)
diff --git a/tests/test_autoreducer_high_memory.py b/tests/test_autoreducer_high_memory.py
@@ -3,7 +3,7 @@
 
 import time
 
-from tests.utils import db_utils
+import utils.db as db_utils
 
 
 class TestAutoreducerQueues:

diff --git a/tests/test_autoreducer_memory_limit.py b/tests/test_autoreducer_memory_limit.py
@@ -4,8 +4,7 @@
 """
 
 import time
-
-from tests.utils import db_utils
+import utils.db as db_utils
 
 
 class TestAutoreducerMemoryLimit:

diff --git a/tests/utils/db_utils.py → tests/utils/db.py b/tests/utils/db_utils.py → tests/utils/db.py