Started by user ha:////4LhIf0vdZMLFequGTVnN1cMcIypvsThRPBC5Va3yiaavAAAAnh+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAy+EgZh/dLi1CL9nNLkxNxMoEmpxZn5ANPJiavIAAAALuca Milanesio [EnvInject] - Loading node environment variables. Building remotely on ha:////4KMXceo6sujqEJ61FnhtvzoKQbTkONE5fFqSuXr6Pdj/AAAAqx+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAzREgZ5/eT83ILSktQi/fTknPzSFDMzXaMUg5QU06QkC8NEcwCG8bf51AAAAA==gcloud66-2d0dd5bb81a7 (bazel-debian mvn) in workspace /home/jenkins/workspace/plugin-high-availability-gerritforge-stable-2.15 Wiping out workspace first. Cloning the remote Git repository Cloning repository https://gerrit.googlesource.com/gerrit > git init /home/jenkins/workspace/plugin-high-availability-gerritforge-stable-2.15 # timeout=10 Fetching upstream changes from https://gerrit.googlesource.com/gerrit > git --version # timeout=10 > git fetch --tags --progress https://gerrit.googlesource.com/gerrit +refs/heads/*:refs/remotes/gerrit/* > git config remote.gerrit.url https://gerrit.googlesource.com/gerrit # timeout=10 > git config --add remote.gerrit.fetch +refs/heads/*:refs/remotes/gerrit/* # timeout=10 > git config remote.gerrit.url https://gerrit.googlesource.com/gerrit # timeout=10 Fetching upstream changes from https://gerrit.googlesource.com/gerrit > git fetch --tags --progress https://gerrit.googlesource.com/gerrit +refs/heads/*:refs/remotes/gerrit/* > git config remote.origin.url https://github.com/GerritForge/plugins_high-availability # timeout=10 Fetching upstream changes from https://github.com/GerritForge/plugins_high-availability > git fetch --tags --progress https://github.com/GerritForge/plugins_high-availability +refs/heads/*:refs/remotes/origin/* > git rev-parse origin/stable-2.15^{commit} # timeout=10 > git rev-parse refs/remotes/gerrit/origin/stable-2.15^{commit} # timeout=10 > git rev-parse refs/remotes/origin/stable-2.15^{commit} # timeout=10 > git rev-parse refs/remotes/origin/origin/stable-2.15^{commit} # timeout=10 Checking out Revision f44f3f09e93d928d75d99b935effdbb158eacaa7 (origin/stable-2.15) > git config core.sparsecheckout # timeout=10 > git checkout -f f44f3f09e93d928d75d99b935effdbb158eacaa7 Commit message: "Merge branch 'stable-2.14' into stable-2.15" > git rev-list --no-walk 03df232c0e88ce3edc8c81e5d88133123bad4f6d # timeout=10 [plugin-high-availability-gerritforge-stable-2.15] $ /bin/bash -e /tmp/jenkins5076890894927920846.sh Previous HEAD position was f44f3f09e9... Merge branch 'stable-2.14' into stable-2.15 HEAD is now at 31298e635e... detach -> detached Java set to: /usr/lib/jvm/java-8-openjdk-amd64/bin/java Buck set to: /opt/buck-java/bin/buck Bazelisk version: development Starting local Bazel server and connecting to it... Build label: 0.26.1 Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar Build time: Thu Jun 6 11:05:05 2019 (1559819105) Build timestamp: 1559819105 Build timestamp as int: 1559819105 INFO: Invocation ID: a94fdcd2-72c4-42cf-a4d8-c341f12f1619 Loading: Loading: 0 packages loaded Loading: 0 packages loaded Loading: 0 packages loaded Analyzing: target //plugins/high-availability:high-availability (1 packages loaded, 0 targets configured) Analyzing: target //plugins/high-availability:high-availability (6 packages loaded, 17 targets configured) Analyzing: target //plugins/high-availability:high-availability (20 packages loaded, 298 targets configured) Analyzing: target //plugins/high-availability:high-availability (51 packages loaded, 555 targets configured) Analyzing: target //plugins/high-availability:high-availability (81 packages loaded, 2782 targets configured) Analyzing: target //plugins/high-availability:high-availability (104 packages loaded, 2844 targets configured) Analyzing: target //plugins/high-availability:high-availability (111 packages loaded, 2892 targets configured) Analyzing: target //plugins/high-availability:high-availability (143 packages loaded, 3181 targets configured) Analyzing: target //plugins/high-availability:high-availability (144 packages loaded, 3533 targets configured) INFO: Analyzed target //plugins/high-availability:high-availability (145 packages loaded, 3889 targets configured). INFO: Found 1 target... [0 / 11] [Prepa] Expanding template plugins/high-availability/high-availability__non_stamped [91 / 287] SkylarkAction external/bazel_tools/tools/jdk/platformclasspath.jar [for host]; 0s remote-cache ... (157 actions, 31 running) [276 / 325] Executing genrule //gerrit-server:prolog-common__pl2j; 1s local, remote-cache ... (6 actions, 2 running) [276 / 325] Executing genrule //gerrit-server:prolog-common__pl2j; 1s local, remote-cache ... (8 actions, 5 running) [295 / 325] Executing genrule //gerrit-server:prolog-common__pl2j; 5s local, remote-cache ... (10 actions running) [310 / 325] Compiling Java headers gerrit-server/libserver-hjar.jar (1398 files); 0s remote-cache [313 / 325] Compiling Java headers gerrit-server/libprolog-common-hjar.jar (1 files); 1s local, remote-cache ... (8 actions running) [321 / 325] Building plugins/high-availability/libhigh-availability__plugin-class.jar (80 source files); 1s remote-cache, worker Target //plugins/high-availability:high-availability up-to-date: bazel-bin/plugins/high-availability/high-availability.jar INFO: Elapsed time: 41.638s, Critical Path: 20.70s INFO: 308 processes: 187 remote cache hit, 119 local, 2 worker. INFO: Build completed successfully, 325 total actions INFO: Build completed successfully, 325 total actions INFO: Invocation ID: fe878cff-b54d-4874-b255-6b86afbdb720 Loading: Loading: 0 packages loaded INFO: Build option --test_env has changed, discarding analysis cache. Analyzing: target //plugins/high-availability:high_availability_tests (0 packages loaded, 0 targets configured) Analyzing: target //plugins/high-availability:high_availability_tests (12 packages loaded, 1576 targets configured) Analyzing: target //plugins/high-availability:high_availability_tests (52 packages loaded, 4302 targets configured) INFO: Analyzed target //plugins/high-availability:high_availability_tests (59 packages loaded, 4354 targets configured). INFO: Found 1 test target... [0 / 3] [Prepa] BazelWorkspaceStatusAction stable-status.txt [346 / 402] Building gerrit-util-cli/libcli.jar (6 source files); 0s remote-cache, worker ... (47 actions, 33 running) [351 / 406] Building gerrit-util-cli/libcli.jar (6 source files); 1s remote-cache, worker ... (47 actions, 31 running) [361 / 415] Building gerrit-util-cli/libcli.jar (6 source files); 3s remote-cache, worker ... (47 actions, 27 running) [361 / 415] Building gerrit-util-cli/libcli.jar (6 source files); 4s remote-cache, worker ... (48 actions, 29 running) INFO: From Building gerrit-server/libprolog-common.jar (1 source jar): /gerrit/PRED_commit_message_matches_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_split_commit_delta_5.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_commit_delta_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_commit_delta_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_commit_committer_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_commit_author_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_subtract1_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_remove_label_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_find_label2_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_find_label_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_locate_submit_type_filter_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_noop_filter_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_locate_submit_filter_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_filter_submit_type_results_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_call_submit_filter_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_filter_submit_results_4.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_$dummy_3_gerrit$002Dserver$002Fsrc$002Fmain$002Fprolog$002Fgerrit_common$002Epl_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_filter_submit_results_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_check_label_range_permission_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_max_no_block_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_any_with_block_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_$dummy_2_gerrit$002Dserver$002Fsrc$002Fmain$002Fprolog$002Fgerrit_common$002Epl_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_max_with_block_4.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_legacy_submit_rule_5.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_default_submit_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_default_submit_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_default_submit_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_locate_submit_type_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_get_submit_type_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_locate_submit_rule_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_listN_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_locate_helper_4.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_is_all_ok_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_call_rule_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_can_submit_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_$dummy_1_gerrit$002Dserver$002Fsrc$002Fmain$002Fprolog$002Fgerrit_common$002Epl_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_not_same_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_assume_range_from_label_4.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_assume_range_from_label_0.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_user_label_range_4.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_check_user_label_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_index_commit_labels_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_set_commit_labels_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_get_commit_labels_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_scan_commit_labels_3.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_commit_label_2.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_$dummy_0_gerrit$002Dserver$002Fsrc$002Fmain$002Fprolog$002Fgerrit_common$002Epl_4.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_define_hash_1.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_init_0.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) /gerrit/PRED_$init_0.java:8: warning: [WildcardImport] Wildcard imports, static or otherwise, should not be used import com.googlecode.prolog_cafe.builtin.*; ^ (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports) [363 / 416] Building gerrit-util-cli/libcli.jar (6 source files); 5s remote-cache, worker ... (48 actions, 28 running) INFO: From Building gerrit-util-cli/libcli.jar (6 source files): gerrit-util-cli/src/main/java/com/google/gerrit/util/cli/CmdLineParser.java:326: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand. private static class PrefixedOption implements Option { ^ (see https://errorprone.info/bugpattern/BadAnnotationImplementation) gerrit-util-cli/src/main/java/com/google/gerrit/util/cli/CmdLineParser.java:327: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'PrefixedOption' has non-final field 'prefix' String prefix; ^ (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker) Did you mean 'final String prefix;'? gerrit-util-cli/src/main/java/com/google/gerrit/util/cli/CmdLineParser.java:328: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'PrefixedOption' has non-final field 'o' Option o; ^ (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker) Did you mean 'final Option o;'? gerrit-util-cli/src/main/java/com/google/gerrit/util/cli/CmdLineParser.java:494: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand. private static class HelpOption implements Option, Setter { ^ (see https://errorprone.info/bugpattern/BadAnnotationImplementation) gerrit-util-cli/src/main/java/com/google/gerrit/util/cli/CmdLineParser.java:495: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'HelpOption' has non-final field 'value' private boolean value; ^ (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker) Did you mean 'private final boolean value;'? [364 / 417] Building gerrit-reviewdb/libclient-class.jar (32 source files); 6s remote-cache, worker ... (48 actions, 28 running) [364 / 417] Building gerrit-reviewdb/libclient-class.jar (32 source files); 8s remote-cache, worker ... (48 actions, 28 running) [367 / 421] Building gerrit-reviewdb/libserver-class.jar (49 source files); 10s remote-cache, worker ... (48 actions, 28 running) INFO: From Building gerrit-httpd/libhttpd-class.jar (111 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): gerrit-httpd/src/main/java/com/google/gerrit/httpd/HttpServletResponseRecorder.java:75: warning: [MissingOverride] getStatus overrides method in HttpServletResponseWrapper; expected @Override public int getStatus() { ^ (see https://errorprone.info/bugpattern/MissingOverride) Did you mean '@Override @SuppressWarnings("all")'? gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/AuditedHttpServletResponse.java:29: warning: [MissingOverride] getStatus overrides method in HttpServletResponseWrapper; expected @Override public int getStatus() { ^ (see https://errorprone.info/bugpattern/MissingOverride) Did you mean '@Override @SuppressWarnings("all")'? [372 / 421] Building gerrit-lucene/liblucene.jar (11 source files); 11s remote-cache ... (48 actions, 29 running) INFO: From Building gerrit-pgm/libinit-api.jar (14 source files): gerrit-pgm/src/main/java/com/google/gerrit/pgm/init/api/Section.java:121: warning: [GetClassOnEnum] Calling getClass() on an enum may return a subclass of the enum type E allowedValues = (E) EnumSet.allOf(defValue.getClass()); ^ (see https://errorprone.info/bugpattern/GetClassOnEnum) Did you mean 'E allowedValues = (E) EnumSet.allOf(defValue.getDeclaringClass());'? [383 / 422] Building gerrit-lucene/liblucene.jar (11 source files); 14s remote-cache ... (48 actions, 30 running) INFO: From Building gerrit-extension-api/libapi.jar (295 source files): gerrit-extension-api/src/main/java/com/google/gerrit/extensions/restapi/RestApiException.java:37: warning: [TypeParameterUnusedInFormals] Declaring a type parameter that is only used in the return type is a misuse of generics: operations on the type parameter are unchecked, it hides unsafe casts at invocations of the method, and it interacts badly with method overload resolution. public T caching(CacheControl c) { ^ (see https://errorprone.info/bugpattern/TypeParameterUnusedInFormals) [390 / 426] Building gerrit-lucene/liblucene.jar (11 source files); 16s remote-cache ... (47 actions, 30 running) [398 / 426] Building gerrit-prettify/libserver.jar (2 source files); 17s remote-cache ... (38 actions, 24 running) INFO: From Building gerrit-lucene/liblucene.jar (11 source files): gerrit-lucene/src/main/java/com/google/gerrit/lucene/WrappableSearcherManager.java:185: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (unwrapped == reader) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (Objects.equals(unwrapped, reader)) {' or 'if (unwrapped.equals(reader)) {'? gerrit-lucene/src/main/java/com/google/gerrit/lucene/WrappableSearcherManager.java:196: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (unwrapped != reader) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (!Objects.equals(unwrapped, reader)) {' or 'if (!unwrapped.equals(reader)) {'? [411 / 426] Building external/com_google_protobuf/libprotobuf_java.jar (122 source files, 1 source jar); 9s remote-cache, worker ... (20 actions, 14 running) INFO: From Building gerrit-pgm/libinit-class.jar (60 source files): gerrit-pgm/src/main/java/com/google/gerrit/pgm/init/api/Section.java:121: warning: [GetClassOnEnum] Calling getClass() on an enum may return a subclass of the enum type E allowedValues = (E) EnumSet.allOf(defValue.getClass()); ^ (see https://errorprone.info/bugpattern/GetClassOnEnum) Did you mean 'E allowedValues = (E) EnumSet.allOf(defValue.getDeclaringClass());'? INFO: From Building gerrit-server/libtestutil.jar (24 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): gerrit-server/src/test/java/com/google/gerrit/testutil/InMemoryDatabase.java:68: warning: [AmbiguousMethodReference] This method's reference is ambiguous, its name and functional interface type are the same as: drop() public static void drop(InMemoryDatabase db) { ^ (see https://errorprone.info/bugpattern/AmbiguousMethodReference) INFO: From Building gerrit-sshd/libsshd.jar (98 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ApproveOption.java:30: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand. final class ApproveOption implements Option, Setter { ^ (see https://errorprone.info/bugpattern/BadAnnotationImplementation) gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ApproveOption.java:33: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'ApproveOption' has field 'type' of type 'com.google.gerrit.common.data.LabelType', the declaration of type 'com.google.gerrit.common.data.LabelType' is not annotated with @com.google.errorprone.annotations.Immutable private final LabelType type; ^ (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker) gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ApproveOption.java:35: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'ApproveOption' has non-final field 'value' private Short value; ^ (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker) Did you mean 'private final Short value;'? gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ShowConnections.java:110: warning: [BadComparable] Possible sign flip from narrowing conversion return (int) (arg0.getId() - arg1.getId()); ^ (see https://errorprone.info/bugpattern/BadComparable) Did you mean 'return Long.compare(arg0.getId(), arg1.getId());'? gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/StreamEvents.java:279: warning: [SynchronizeOnNonFinalField] Synchronizing on non-final fields is not safe: if the field is ever updated, different threads may end up locking on different objects. synchronized (stdout) { ^ (see https://errorprone.info/bugpattern/SynchronizeOnNonFinalField) gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/StreamEvents.java:286: warning: [SynchronizeOnNonFinalField] Synchronizing on non-final fields is not safe: if the field is ever updated, different threads may end up locking on different objects. synchronized (stdout) { ^ (see https://errorprone.info/bugpattern/SynchronizeOnNonFinalField) gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/UploadArchive.java:138: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (s == PacketLineIn.END) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (Objects.equals(s, PacketLineIn.END)) {' or 'if (s.equals(PacketLineIn.END)) {'? gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/UploadArchive.java:138: warning: [StringEquality] String comparison using reference equality instead of value equality if (s == PacketLineIn.END) { ^ (see https://errorprone.info/bugpattern/StringEquality) Did you mean 'if (Objects.equals(s, PacketLineIn.END)) {' or 'if (s.equals(PacketLineIn.END)) {'? [418 / 426] Building external/com_google_protobuf/libprotobuf_java.jar (122 source files, 1 source jar); 12s remote-cache, worker ... (10 actions, 9 running) INFO: From Building gerrit-acceptance-framework/liblib.jar (35 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): gerrit-acceptance-framework/src/test/java/com/google/gerrit/acceptance/GerritServer.java:413: warning: [TypeParameterUnusedInFormals] Declaring a type parameter that is only used in the return type is a misuse of generics: operations on the type parameter are unchecked, it hides unsafe casts at invocations of the method, and it interacts badly with method overload resolution. private static T get(Object obj, String field) ^ (see https://errorprone.info/bugpattern/TypeParameterUnusedInFormals) gerrit-acceptance-framework/src/test/java/com/google/gerrit/acceptance/ConfigAnnotationParser.java:48: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand. static class GlobalPluginConfigToGerritConfig implements GerritConfig { ^ (see https://errorprone.info/bugpattern/BadAnnotationImplementation) [423 / 426] Building external/com_google_protobuf/libprotobuf_java.jar (122 source files, 1 source jar); 17s remote-cache, worker ... (4 actions running) [424 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 26s remote-cache, worker [424 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 32s remote-cache, worker INFO: From Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): gerrit-server/src/main/java/com/google/gerrit/server/notedb/ReviewerStateInternal.java:53: warning: [ImmutableEnumChecker] enums should be immutable: 'ReviewerStateInternal' has field 'footerKey' of type 'org.eclipse.jgit.revwalk.FooterKey', the declaration of type 'org.eclipse.jgit.revwalk.FooterKey' is not annotated with @com.google.errorprone.annotations.Immutable private final FooterKey footerKey; ^ (see https://errorprone.info/bugpattern/ImmutableEnumChecker) gerrit-server/src/main/java/com/google/gerrit/server/index/change/ChangeIndexRewriter.java:157: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (in == out || out instanceof IndexPredicate) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (Objects.equals(in, out) || out instanceof IndexPredicate) {' or 'if (in.equals(out) || out instanceof IndexPredicate) {'? gerrit-server/src/main/java/com/google/gerrit/server/index/change/ChangeIndexRewriter.java:209: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (nc == c) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (Objects.equals(nc, c)) {' or 'if (nc.equals(c)) {'? gerrit-server/src/main/java/com/google/gerrit/server/util/IdGenerator.java:58: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts v0 += ((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1; ^ (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment) Did you mean 'v0 = (short) (v0 + (((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1));'? gerrit-server/src/main/java/com/google/gerrit/server/util/IdGenerator.java:59: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts v1 += ((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3; ^ (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment) Did you mean 'v1 = (short) (v1 + (((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3));'? gerrit-server/src/main/java/com/google/gerrit/server/util/IdGenerator.java:67: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts v1 -= ((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3; ^ (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment) Did you mean 'v1 = (short) (v1 - (((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3));'? gerrit-server/src/main/java/com/google/gerrit/server/util/IdGenerator.java:68: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts v0 -= ((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1; ^ (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment) Did you mean 'v0 = (short) (v0 - (((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1));'? gerrit-server/src/main/java/com/google/gerrit/server/account/AccountDeactivator.java:74: warning: [FutureReturnValueIgnored] Return value of methods returning Future must be checked. Ignoring returned Futures suppresses exceptions thrown from the code that completes the Future. .scheduleAtFixedRate(deactivator, delay, interval, TimeUnit.MILLISECONDS); ^ (see https://errorprone.info/bugpattern/FutureReturnValueIgnored) Did you mean to remove this line? gerrit-server/src/main/java/com/google/gerrit/server/patch/PatchScriptBuilder.java:452: warning: [ReferenceEquality] Comparison using reference equality instead of value equality && (aId == within || within.equals(aId))) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean '&& (Objects.equals(aId, within) || within.equals(aId))) {' or '&& (aId.equals(within) || within.equals(aId))) {'? gerrit-server/src/main/java/com/google/gerrit/server/patch/PatchScriptBuilder.java:472: warning: [ReferenceEquality] Comparison using reference equality instead of value equality && (aId == within || within.equals(aId))) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean '&& (Objects.equals(aId, within) || within.equals(aId))) {' or '&& (aId.equals(within) || within.equals(aId))) {'? gerrit-server/src/main/java/com/google/gerrit/server/change/ArchiveFormat.java:38: warning: [ImmutableEnumChecker] enums should be immutable: 'ArchiveFormat' has field 'format' of type 'org.eclipse.jgit.api.ArchiveCommand.Format', the declaration of type 'org.eclipse.jgit.api.ArchiveCommand.Format' is not annotated with @com.google.errorprone.annotations.Immutable private final ArchiveCommand.Format format; ^ (see https://errorprone.info/bugpattern/ImmutableEnumChecker) gerrit-server/src/main/java/com/google/gerrit/server/git/SubmoduleOp.java:594: warning: [OperatorPrecedence] Use grouping parenthesis to make the operator precedence explicit || iter.hasNext() && (newSize + ellipsis.length()) > maxCombinedCommitMessageSize) { ^ (see https://errorprone.info/bugpattern/OperatorPrecedence) Did you mean '|| (iter.hasNext() && (newSize + ellipsis.length()) > maxCombinedCommitMessageSize)) {'? gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java:305: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (next.getTree() != prior.getTree()) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (!Objects.equals(next.getTree(), prior.getTree())) {' or 'if (!next.getTree().equals(prior.getTree())) {'? gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java:318: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (next.getParent(i).getTree() != prior.getParent(i).getTree()) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (!Objects.equals(next.getParent(i).getTree(), prior.getParent(i).getTree())) {' or 'if (!next.getParent(i).getTree().equals(prior.getParent(i).getTree())) {'? gerrit-server/src/main/java/com/google/gerrit/server/index/change/AllChangesIndexer.java:118: warning: [NarrowingCompoundAssignment] Compound assignments from long to int hide lossy casts changeCount += size; ^ (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment) Did you mean 'changeCount = (int) (changeCount + size);'? gerrit-server/src/main/java/com/google/gerrit/server/notedb/NotesMigrationState.java:60: warning: [ImmutableEnumChecker] enums should be immutable: 'NotesMigrationState' has field 'snapshot' of type 'com.google.gerrit.server.notedb.NotesMigration.Snapshot', the declaration of type 'com.google.gerrit.server.notedb.NotesMigration.Snapshot' is not annotated with @com.google.errorprone.annotations.Immutable private final Snapshot snapshot; ^ (see https://errorprone.info/bugpattern/ImmutableEnumChecker) gerrit-server/src/main/java/com/google/gerrit/server/schema/Schema_146.java:107: warning: [FutureReturnValueIgnored] Return value of methods returning Future must be checked. Ignoring returned Futures suppresses exceptions thrown from the code that completes the Future. batches.stream().forEach(batch -> pool.submit(() -> processBatch(batch, ui))); ^ (see https://errorprone.info/bugpattern/FutureReturnValueIgnored) Did you mean 'batches.stream().forEach();'? [426 / 427] Testing //plugins/high-availability:high_availability_tests; 7s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 16s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 36s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 63s local [426 / 427] 1 / 1 tests; Testing //plugins/high-availability:high_availability_tests; 84s local Target //plugins/high-availability:high_availability_tests up-to-date: bazel-bin/plugins/high-availability/high_availability_tests.jar bazel-bin/plugins/high-availability/high_availability_tests INFO: Elapsed time: 135.714s, Critical Path: 131.50s INFO: 105 processes: 7 remote cache hit, 1 local, 54 processwrapper-sandbox, 43 worker. INFO: Build completed successfully, 110 total actions //plugins/high-availability:high_availability_tests PASSED in 84.1s Executed 1 out of 1 test: 1 test passes. INFO: Build completed successfully, 110 total actions Aug 08, 2019 8:50:38 PM java.util.jar.Attributes read WARNING: Duplicate name in Manifest: Implementation-Version. Ensure that the manifest does not have duplicate entries, and that blank lines separate individual sections in both your manifest and in the META-INF/MANIFEST.MF entry in the jar file. Aug 08, 2019 8:50:38 PM java.util.jar.Attributes read WARNING: Duplicate name in Manifest: Implementation-Version. Ensure that the manifest does not have duplicate entries, and that blank lines separate individual sections in both your manifest and in the META-INF/MANIFEST.MF entry in the jar file. [plugin-high-availability-gerritforge-stable-2.15] $ /bin/bash -e /tmp/jenkins7922590581846620070.sh % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0 100 1799 0 1799 0 0 36347 0 --:--:-- --:--:-- --:--:-- 36714 Archiving artifacts Finished: SUCCESS