nixos/tests/archiver-appliance: finalize

epics-extensions · Aug 4, 2023 · 482f859 · 482f859
1 parent cb161da
commit 482f859
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/nixos/tests/archiver-appliance/default.nix b/nixos/tests/archiver-appliance/default.nix
@@ -43,9 +43,6 @@
       };
 
       networking.firewall.allowedTCPPorts = [8080];
-
-      # Else phoebus-olog gets killed by the OOM killer
-      # virtualisation.memorySize = 2047;
     };
   };
 

diff --git a/nixos/tests/archiver-appliance/test_script.py b/nixos/tests/archiver-appliance/test_script.py
@@ -207,8 +207,8 @@ def static_deadband_has_more_data(_):
         ), "staticDeadband's additional datapoint should be 10"
 
 with subtest("static processed record"):
-    with subtest("wait until staticDeadband is being archived"):
-        retry(lambda _: check_pv_archived("staticDeadband"))
+    with subtest("wait until staticProcessed is being archived"):
+        retry(lambda _: check_pv_archived("staticProcessed"))
 
     with subtest("json of staticProcessed is valid"):
         data = get_data("staticProcessed")
@@ -332,6 +332,7 @@ def waveform_has_data(_):
     mts_content = server.succeed("ls -1 /arch/mts/ArchiverStore")
     assert len(mts_content.strip().split("\n")) > 0
 
-# TODO: check controlling PV
+# TODO: check controlling PV, which seems to be buggy?
+# Added PV sometimes not linked to the controlling PV
 # TODO: check renaming PV
 # TODO: check different policies