[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mesos-commits
Subject:    git commit: Disabled System.Metrics test as per MESOS-1433.
From:       benh () apache ! org
Date:       2014-05-29 1:34:09
Message-ID: 719981b5fecc4195a7fab0cd02f8fdd2 () git ! apache ! org
[Download RAW message or body]

Repository: mesos
Updated Branches:
  refs/heads/master 9370001a4 -> d5eef825c


Disabled System.Metrics test as per MESOS-1433.

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


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

Branch: refs/heads/master
Commit: d5eef825c5908f935961e500cdb8b682289d534e
Parents: 9370001
Author: Dominic Hamon <dhamon@twopensource.com>
Authored: Wed May 28 18:32:59 2014 -0700
Committer: Benjamin Hindman <benjamin.hindman@gmail.com>
Committed: Wed May 28 18:33:23 2014 -0700

----------------------------------------------------------------------
 3rdparty/libprocess/src/tests/system_tests.cpp | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/d5eef825/3rdparty/libprocess/src/tests/system_tests.cpp
                
----------------------------------------------------------------------
diff --git a/3rdparty/libprocess/src/tests/system_tests.cpp \
b/3rdparty/libprocess/src/tests/system_tests.cpp index e07314f..a917790 100644
--- a/3rdparty/libprocess/src/tests/system_tests.cpp
+++ b/3rdparty/libprocess/src/tests/system_tests.cpp
@@ -31,7 +31,14 @@ using namespace process;
 
 using process::metrics::internal::MetricsProcess;
 
-TEST(System, Metrics)
+// MESOS-1433
+// This test is disabled as the Gauges that are used for these metrics
+// may return Failures. In this case we do not put the metric into the
+// endpoint. This has been observed specifically for the memory
+// metrics. If in the future we put the error message from the Failure
+// in the endpoint, or the memory metric is always available, we
+// should reenable this test.
+TEST(System, DISABLED_Metrics)
 {
   Future<http::Response> response =
     http::get(MetricsProcess::instance()->self(), "snapshot");


[prev in list] [next in list] [prev in thread] [next in thread]