----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56042/#review163354 -----------------------------------------------------------
geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/ui/PulseAutomatedTest.java (line 89) <https://reviews.apache.org/r/56042/#comment234761> I just realized this is an "UITest.class" -- I think that means we can't add "FlakyTest.class" to anything in this test. I guess all "UITests" are already isolated in their own test job and aren't executed during precheckin anyway. I'd just close this out and not apply the change. Sorry. - Kirk Lund On Jan. 27, 2017, 10:27 p.m., Kevin Duling wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56042/ > ----------------------------------------------------------- > > (Updated Jan. 27, 2017, 10:27 p.m.) > > > Review request for geode, Jinmei Liao, Jared Stewart, and Kirk Lund. > > > Repository: geode > > > Description > ------- > > GEODE-2378: CI Failure: PulseAutomatedTest > clickHostShowsMemberTest > > Simply added `@Category(FlakyTest.class)` to the test. > > > Diffs > ----- > > > geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/ui/PulseAutomatedTest.java > b19416c3b2e917f591d4b4e85d98a411fa9f4c95 > > Diff: https://reviews.apache.org/r/56042/diff/ > > > Testing > ------- > > unable to run precheckin currently > > > Thanks, > > Kevin Duling > >
