[mesos] branch revert-433-fix-SlaveRecoveryTest.PingTimeoutDuringRecovery created (now 7d5928919)

[cf...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

cfnatali pushed a change to branch revert-433-fix-SlaveRecoveryTest.PingTimeoutDuringRecovery
in repository https://gitbox.apache.org/repos/asf/mesos.git


      at 7d5928919 Revert "Fixed random SlaveRecoveryTest.PingTimeoutDuringRecovery test failure."

This branch includes the following new commits:

     new 7d5928919 Revert "Fixed random SlaveRecoveryTest.PingTimeoutDuringRecovery test failure."

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[mesos] 01/01: Revert "Fixed random SlaveRecoveryTest.PingTimeoutDuringRecovery test failure."

[cf...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

cfnatali pushed a commit to branch revert-433-fix-SlaveRecoveryTest.PingTimeoutDuringRecovery
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 7d592891956d92591d2bd42a6d71c56c3d7cea36
Author: cf-natali <cf...@gmail.com>
AuthorDate: Tue May 3 19:35:49 2022 +0100

    Revert "Fixed random SlaveRecoveryTest.PingTimeoutDuringRecovery test failure."
    
    This reverts commit 57088b11a328355b7f2a53a1b3fba9928a2fde73.
---
 src/tests/slave_recovery_tests.cpp | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/src/tests/slave_recovery_tests.cpp b/src/tests/slave_recovery_tests.cpp
index 41ecd3b6d..398011437 100644
--- a/src/tests/slave_recovery_tests.cpp
+++ b/src/tests/slave_recovery_tests.cpp
@@ -1010,15 +1010,6 @@ TYPED_TEST(SlaveRecoveryTest, PingTimeoutDuringRecovery)
 
   slave.get()->terminate();
 
-  // Make sure that timers started by the previous slave's process expire,
-  // because we reuse the same PID we don't want them to fire while the
-  // new process is running.
-  Clock::pause();
-  Clock::advance(masterFlags.agent_ping_timeout *
-    masterFlags.max_agent_ping_timeouts);
-  Clock::settle();
-  Clock::resume();
-
   Future<ReregisterExecutorMessage> reregisterExecutor =
     FUTURE_PROTOBUF(ReregisterExecutorMessage(), _, _);