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 2016/01/22 09:28:27 UTC
[09/10] mesos git commit: Removed misleading note from allocator
tests.
Removed misleading note from allocator tests.
Review: https://reviews.apache.org/r/42635/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/8b5dbd8b
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/8b5dbd8b
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/8b5dbd8b
Branch: refs/heads/master
Commit: 8b5dbd8b0f7c516c82d7b7fb11b54ee3295af290
Parents: cebbdab
Author: Alexander Rukletsov <ru...@gmail.com>
Authored: Fri Jan 22 00:17:26 2016 -0800
Committer: Benjamin Mahler <be...@gmail.com>
Committed: Fri Jan 22 00:17:26 2016 -0800
----------------------------------------------------------------------
src/tests/hierarchical_allocator_tests.cpp | 6 ------
1 file changed, 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/8b5dbd8b/src/tests/hierarchical_allocator_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/hierarchical_allocator_tests.cpp b/src/tests/hierarchical_allocator_tests.cpp
index d11e1c7..f7c82a4 100644
--- a/src/tests/hierarchical_allocator_tests.cpp
+++ b/src/tests/hierarchical_allocator_tests.cpp
@@ -1569,8 +1569,6 @@ TEST_F(HierarchicalAllocatorTest, RemoveQuota)
// Notify allocator of agents, frameworks, quota and current allocations.
allocator->setQuota(QUOTA_ROLE, quota1);
- // NOTE: We do not report about allocated resources to `framework1`
- // here to avoid double accounting.
allocator->addFramework(
framework1.id(),
framework1,
@@ -1837,8 +1835,6 @@ TEST_F(HierarchicalAllocatorTest, DRFWithQuota)
// Notify allocator of agents, frameworks, quota and current allocations.
allocator->setQuota(QUOTA_ROLE, quota1);
- // NOTE: We do not report about allocated resources to `framework1`
- // here to avoid double accounting.
allocator->addFramework(
framework1.id(),
framework1,
@@ -1933,8 +1929,6 @@ TEST_F(HierarchicalAllocatorTest, QuotaAgainstStarvation)
SlaveInfo agent2 = createSlaveInfo("cpus:1;mem:512;disk:0");
// Notify allocator of agents, frameworks, and current allocations.
- // NOTE: We do not report about allocated resources to `framework1`
- // here to avoid double accounting.
allocator->addFramework(
framework1.id(),
framework1,