You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2017/07/18 03:25:32 UTC

[2/2] mesos git commit: Removed a misleading comment in the reservation endpoint tests.

Removed a misleading comment in the reservation endpoint tests.

This comment seemed to suggest that the reason for exposing
the reserved resources in the agent endpoint was that because
it is only *eventually* consistent with the master's state.
However, we expose agent state in both agent and master
endpoints because of the differing availability and performance
of the two components, as well as because the state is distributed
across the two (and we want to show the state at each distribution
point).


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

Branch: refs/heads/master
Commit: 53fc25dd0c83d4d5995a23e6fd383f3d3ae86f41
Parents: 197fc04
Author: Benjamin Mahler <bm...@apache.org>
Authored: Mon Jul 17 20:09:04 2017 -0700
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Mon Jul 17 20:09:04 2017 -0700

----------------------------------------------------------------------
 src/tests/reservation_endpoints_tests.cpp | 2 --
 1 file changed, 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/53fc25dd/src/tests/reservation_endpoints_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/reservation_endpoints_tests.cpp b/src/tests/reservation_endpoints_tests.cpp
index 34ad48e..3278732 100644
--- a/src/tests/reservation_endpoints_tests.cpp
+++ b/src/tests/reservation_endpoints_tests.cpp
@@ -1530,8 +1530,6 @@ TEST_F(ReservationEndpointsTest, DifferentPrincipalsSameRole)
 
 // This test verifies that unreserved resources, dynamic reservations, allocated
 // resources per each role are reflected in the agent's "/state" endpoint.
-// Separately exposing reservations from the agent's endpoint is necessary
-// because it's not a guarantee that it matches the master's versions.
 TEST_F(ReservationEndpointsTest, AgentStateEndpointResources)
 {
   Try<Owned<cluster::Master>> master = StartMaster();