Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12042 )
Change subject: java: add support for flaky test reporting ...................................................................... java: add support for flaky test reporting This patch hooks into the existing RetryRule to report test results to the flaky test server inline as the tests are executed. All of the actual reporting logic is factored out into a separate ResultReporter class. The interface for the test reporter to pass relevant information about the build environment to the flaky test server is based on environment variables. This includes configuration and build metadata such as flaky test server address, git revision, build id, build host, and build type. This patch also includes a simple integration test for the reporter using a mocked-up flaky test server HTTP endpoint. This patch does not integrate the above functionality into the build. That will happen in a follow-up patch. Change-Id: I34f88363dbf52c2f7bba50fbfb5de059f88e7f74 Reviewed-on: http://gerrit.cloudera.org:8080/12042 Tested-by: Adar Dembo <a...@cloudera.com> Reviewed-by: Grant Henke <granthe...@apache.org> --- M java/gradle/dependencies.gradle M java/kudu-test-utils/build.gradle M java/kudu-test-utils/src/main/java/org/apache/kudu/test/CapturingLogAppender.java A java/kudu-test-utils/src/main/java/org/apache/kudu/test/CapturingToFileLogAppender.java A java/kudu-test-utils/src/main/java/org/apache/kudu/test/junit/ResultReporter.java M java/kudu-test-utils/src/main/java/org/apache/kudu/test/junit/RetryRule.java A java/kudu-test-utils/src/test/java/org/apache/kudu/test/junit/TestResultReporter.java M java/kudu-test-utils/src/test/java/org/apache/kudu/test/junit/TestRetryRule.java 8 files changed, 721 insertions(+), 18 deletions(-) Approvals: Adar Dembo: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12042 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I34f88363dbf52c2f7bba50fbfb5de059f88e7f74 Gerrit-Change-Number: 12042 Gerrit-PatchSet: 10 Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>