You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by gr...@apache.org on 2018/04/02 18:46:50 UTC

[2/6] mesos git commit: Removed unnecessary code and a misleading comment from a test.

Removed unnecessary code and a misleading comment from a test.

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


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

Branch: refs/heads/master
Commit: b7cc83140561816444842968d6c4d3b13d4c49a0
Parents: 1f2da7b
Author: Gaston Kleiman <ga...@mesosphere.io>
Authored: Mon Apr 2 10:44:04 2018 -0700
Committer: Greg Mann <gr...@gmail.com>
Committed: Mon Apr 2 11:43:15 2018 -0700

----------------------------------------------------------------------
 src/tests/reconciliation_tests.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/b7cc8314/src/tests/reconciliation_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/reconciliation_tests.cpp b/src/tests/reconciliation_tests.cpp
index ad6c993..b06244b 100644
--- a/src/tests/reconciliation_tests.cpp
+++ b/src/tests/reconciliation_tests.cpp
@@ -448,14 +448,11 @@ TEST_F(ReconciliationTest, RecoveredAgent)
   Try<Owned<cluster::Master>> master = StartMaster(masterFlags);
   ASSERT_SOME(master);
 
-  // Reuse slaveFlags so both StartSlave() use the same work_dir.
-  slave::Flags slaveFlags = CreateSlaveFlags();
-
   Future<SlaveRegisteredMessage> slaveRegisteredMessage =
     FUTURE_PROTOBUF(SlaveRegisteredMessage(), _, _);
 
   Owned<MasterDetector> detector = master.get()->createDetector();
-  Try<Owned<cluster::Slave>> slave = StartSlave(detector.get(), slaveFlags);
+  Try<Owned<cluster::Slave>> slave = StartSlave(detector.get());
   ASSERT_SOME(slave);
 
   // Wait for the slave to register and get the slave id.