Fix too-low heartbeat threshold in test (#3647)

benclifford · web-flow · commit bcea1fae48c0 · 2024-10-22T19:16:09.000-04:00
The threshold was the same as the heartbeat period, leading to
races in CI where the manager would be expired before the heartbeat
was received.
diff --git a/parsl/tests/test_serialization/test_3495_deserialize_managerlost.py b/parsl/tests/test_serialization/test_3495_deserialize_managerlost.py
@@ -32,7 +32,7 @@ def test_manager_lost_system_failure(tmpd_cwd):
         cores_per_worker=1,
         worker_logdir_root=str(tmpd_cwd),
         heartbeat_period=1,
-        heartbeat_threshold=1,
+        heartbeat_threshold=3,
     )
     c = Config(executors=[hte], strategy='simple', strategy_period=0.1)
 

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ def test_manager_lost_system_failure(tmpd_cwd):`
`32`	`32`	`cores_per_worker=1,`
`33`	`33`	`worker_logdir_root=str(tmpd_cwd),`
`34`	`34`	`heartbeat_period=1,`
`35`		`- heartbeat_threshold=1,`
	`35`	`+ heartbeat_threshold=3,`
`36`	`36`	`)`
`37`	`37`	`c = Config(executors=[hte], strategy='simple', strategy_period=0.1)`
`38`	`38`