----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/ -----------------------------------------------------------
(Updated April 11, 2017, 5:18 p.m.) Review request for geode, Jinmei Liao, Jared Stewart, Kirk Lund, and Patrick Rhomberg. Changes ------- Inn reponse to Kirk's comment, I renamed parameter port to jmxPort in geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/Server.java to clarify it's usage. Repository: geode Description ------- GEODE-2737: Change Pulse UI tests to use random port for JMX connections Diffs (updated) ----- geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/Server.java ebf291b481c78533adab9006e32a9dcb5d1d8d39 geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/rules/ServerRule.java af3d64bd5118ff870fadc16dcdeb3cde858611ce Diff: https://reviews.apache.org/r/58325/diff/2/ Changes: https://reviews.apache.org/r/58325/diff/1-2/ Testing (updated) ------- ==> Concourse had a problem on the prechecking while running flaky tests ==> precheckin has been restarted All Pulse UI tests run locally pass. Precheckin is running Thanks, Ken Howe