Repository: incubator-brooklyn
Updated Branches:
  refs/heads/master b50e17645 -> 81f552b82


mark ConnectionFailureDetectorTest.java as Integration because it requires 
ports accessible

the failure log is recorded at:
https://gist.github.com/davidx/1a21649a5e637b7b2a6a


Project: http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/repo
Commit: 
http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/commit/0482cbd9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/tree/0482cbd9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/diff/0482cbd9

Branch: refs/heads/master
Commit: 0482cbd9c3f9197c032f9188837fc872d94aebc7
Parents: 5118860
Author: Alex Heneveld <alex.henev...@cloudsoftcorp.com>
Authored: Thu Jan 21 09:41:44 2016 +0000
Committer: Alex Heneveld <alex.henev...@cloudsoftcorp.com>
Committed: Thu Jan 21 09:42:29 2016 +0000

----------------------------------------------------------------------
 .../brooklyn/policy/ha/ConnectionFailureDetectorTest.java     | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/blob/0482cbd9/brooklyn-server/policy/src/test/java/org/apache/brooklyn/policy/ha/ConnectionFailureDetectorTest.java
----------------------------------------------------------------------
diff --git 
a/brooklyn-server/policy/src/test/java/org/apache/brooklyn/policy/ha/ConnectionFailureDetectorTest.java
 
b/brooklyn-server/policy/src/test/java/org/apache/brooklyn/policy/ha/ConnectionFailureDetectorTest.java
index 1b34c11..25f8d54 100644
--- 
a/brooklyn-server/policy/src/test/java/org/apache/brooklyn/policy/ha/ConnectionFailureDetectorTest.java
+++ 
b/brooklyn-server/policy/src/test/java/org/apache/brooklyn/policy/ha/ConnectionFailureDetectorTest.java
@@ -37,19 +37,24 @@ import org.apache.brooklyn.core.entity.Entities;
 import org.apache.brooklyn.core.entity.factory.ApplicationBuilder;
 import org.apache.brooklyn.core.test.entity.LocalManagementContextForTests;
 import org.apache.brooklyn.core.test.entity.TestApplication;
+import org.apache.brooklyn.policy.ha.HASensors.FailureDescriptor;
 import org.apache.brooklyn.test.Asserts;
 import org.apache.brooklyn.util.collections.MutableMap;
 import org.apache.brooklyn.util.time.Duration;
 import org.testng.annotations.AfterMethod;
 import org.testng.annotations.BeforeMethod;
 import org.testng.annotations.Test;
-import org.apache.brooklyn.policy.ha.HASensors.FailureDescriptor;
 
 import com.google.common.base.Predicate;
 import com.google.common.base.Predicates;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.net.HostAndPort;
 
+@Test(groups="Integration")
+// can fail if a firewall is in place on high-numbered ephemeral ports;
+// at least one report of *test* failures, as in no events coming in;
+// log unavailable but the test has been passing everywhere else so we
+// suspect some environments don't support this test
 public class ConnectionFailureDetectorTest {
 
     private static final int TIMEOUT_MS = 30*1000;

Reply via email to