-----------------------------------------------------------
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

Reply via email to