You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2016/10/31 23:03:29 UTC

mesos git commit: Fixed flaky MasterTest.OrphanTasksMultipleAgents test.

Repository: mesos
Updated Branches:
  refs/heads/master 15d67d4a4 -> 81b3eef35


Fixed flaky MasterTest.OrphanTasksMultipleAgents test.

See the attached bug for details.

Review: https://reviews.apache.org/r/53318


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/81b3eef3
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/81b3eef3
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/81b3eef3

Branch: refs/heads/master
Commit: 81b3eef354f0cc013e9c21dfab2b2d7b3a77d9a4
Parents: 15d67d4
Author: Vinod Kone <vi...@gmail.com>
Authored: Mon Oct 31 14:24:35 2016 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Mon Oct 31 14:25:02 2016 -0700

----------------------------------------------------------------------
 src/tests/master_tests.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/81b3eef3/src/tests/master_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/master_tests.cpp b/src/tests/master_tests.cpp
index e1e264e..ce42a10 100644
--- a/src/tests/master_tests.cpp
+++ b/src/tests/master_tests.cpp
@@ -3030,10 +3030,10 @@ TEST_F(MasterTest, OrphanTasksMultipleAgents)
   EXPECT_EQ(TASK_RUNNING, status2.get().state());
 
   Future<SlaveReregisteredMessage> slaveReregisteredMessage1 =
-    FUTURE_PROTOBUF(SlaveReregisteredMessage(), _, _);
+    FUTURE_PROTOBUF(SlaveReregisteredMessage(), _, slave1.get()->pid);
 
   Future<SlaveReregisteredMessage> slaveReregisteredMessage2 =
-    FUTURE_PROTOBUF(SlaveReregisteredMessage(), _, _);
+    FUTURE_PROTOBUF(SlaveReregisteredMessage(), _, slave2.get()->pid);
 
   // Failover the master.
   master->reset();