Merge pull request #950 from akka/wip-2793-max-is-not-max-patriknw

Remove check of heap max value, see #2793

akka-cluster/src/test/scala/akka/cluster/MetricValuesSpec.scala

@@ -37,17 +37,14 @@ class MetricValuesSpec extends AkkaSpec(MetricsEnabledSpec.config) with MetricsC
     "extract expected MetricValue types for load balancing" in {
       nodes foreach { node ⇒
         node match {
-          case HeapMemory(address, _, used, committed, Some(max)) ⇒
+          case HeapMemory(address, _, used, committed, _) ⇒
+            used must be > (0L)
             committed must be >= (used)
-            used must be <= (max)
-            committed must be <= (max)
+            // Documentation java.lang.management.MemoryUsage says that committed <= max,
+            // but in practice that is not always true (we have seen it happen). Therefore
+            // we don't check the heap max value in this test.
             // extract is the java api
             StandardMetrics.extractHeapMemory(node) must not be (null)
-          case HeapMemory(address, _, used, committed, None) ⇒
-            used must be > (0L)
-            committed must be > (0L)
-            // extract is the java api
-            StandardMetrics.extractCpu(node) must not be (null)
         }
 
         node match {