Add small sleep in some tests to reduce flakes in CI (#526)

This is to get around a docker-in-ci bug with how the host machine is not able to sync files to disk properly. see: #527
TraceMachina · Dec 19, 2023 · fd4e6a3 · fd4e6a3
1 parent a18d2d2
commit fd4e6a3
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/nativelink-worker/tests/local_worker_test.rs b/nativelink-worker/tests/local_worker_test.rs
@@ -496,6 +496,8 @@ mod local_worker_tests {
             drop(file);
             precondition_script
         };
+        // TODO(#527) Sleep to reduce flakey chances.
+        tokio::time::sleep(Duration::from_millis(250)).await;
         let local_worker_config = LocalWorkerConfig {
             experimental_precondition_script: Some(precondition_script),
             ..Default::default()

diff --git a/nativelink-worker/tests/running_actions_manager_test.rs b/nativelink-worker/tests/running_actions_manager_test.rs
@@ -1376,6 +1376,9 @@ exit 0
             test_wrapper_script
         };
 
+        // TODO(#527) Sleep to reduce flakey chances.
+        tokio::time::sleep(Duration::from_millis(250)).await;
+
         let running_actions_manager = Arc::new(RunningActionsManagerImpl::new(RunningActionsManagerArgs {
             root_work_directory: root_work_directory.clone(),
             execution_configuration: ExecutionConfiguration {
@@ -1505,6 +1508,9 @@ exit 0
             test_wrapper_script
         };
 
+        // TODO(#527) Sleep to reduce flakey chances.
+        tokio::time::sleep(Duration::from_millis(250)).await;
+
         let running_actions_manager = Arc::new(RunningActionsManagerImpl::new(RunningActionsManagerArgs {
             root_work_directory: root_work_directory.clone(),
             execution_configuration: ExecutionConfiguration {
@@ -1646,6 +1652,9 @@ exit 1
             test_wrapper_script
         };
 
+        // TODO(#527) Sleep to reduce flakey chances.
+        tokio::time::sleep(Duration::from_millis(250)).await;
+
         let running_actions_manager = Arc::new(RunningActionsManagerImpl::new(RunningActionsManagerArgs {
             root_work_directory: root_work_directory.clone(),
             execution_configuration: ExecutionConfiguration {