Started by user ha:////4LhIf0vdZMLFequGTVnN1cMcIypvsThRPBC5Va3yiaavAAAAnh+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAy+EgZh/dLi1CL9nNLkxNxMoEmpxZn5ANPJiavIAAAALuca Milanesio [EnvInject] - Loading node environment variables. Building remotely on ha:////4OFrka/NqMDEqvLy8dddrE0SUiajqdJ4FzrhlcHCzEUcAAAAqx+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAzREgZ5/eT83ILSktQi/fTknPzSFDNjXXPDZDOjZLO0VKMUIwAnbjZl1AAAAA==gcloud63-71c62c6fe2d2 (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 03df232c0e88ce3edc8c81e5d88133123bad4f6d (origin/stable-2.15) > git config core.sparsecheckout # timeout=10 > git checkout -f 03df232c0e88ce3edc8c81e5d88133123bad4f6d 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/jenkins1540862689056967895.sh Previous HEAD position was 03df232c0e... Merge branch 'stable-2.14' into stable-2.15 HEAD is now at b708081d99... Set version to 2.15.15-SNAPSHOT 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.27.0rc3 Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar Build time: Thu Jun 6 16:23:06 2019 (1559838186) Build timestamp: 1559838186 Build timestamp as int: 1559838186 INFO: Invocation ID: c70b3785-e936-491f-abfa-0f2e6e5db276 Loading: Loading: 0 packages loaded 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 (10 packages loaded, 30 targets configured) Analyzing: target //plugins/high-availability:high-availability (10 packages loaded, 30 targets configured) Analyzing: target //plugins/high-availability:high-availability (21 packages loaded, 280 targets configured) Analyzing: target //plugins/high-availability:high-availability (35 packages loaded, 387 targets configured) Analyzing: target //plugins/high-availability:high-availability (47 packages loaded, 799 targets configured) Analyzing: target //plugins/high-availability:high-availability (83 packages loaded, 1865 targets configured) Analyzing: target //plugins/high-availability:high-availability (102 packages loaded, 2828 targets configured) Analyzing: target //plugins/high-availability:high-availability (135 packages loaded, 2942 targets configured) Analyzing: target //plugins/high-availability:high-availability (145 packages loaded, 3536 targets configured) Analyzing: target //plugins/high-availability:high-availability (145 packages loaded, 3536 targets configured) INFO: Analyzed target //plugins/high-availability:high-availability (146 packages loaded, 3892 targets configured). INFO: Found 1 target... [0 / 3] [Prepa] BazelWorkspaceStatusAction stable-status.txt [59 / 309] SkylarkAction external/bazel_tools/tools/jdk/platformclasspath_classes/DumpPlatformClassPath.class; 1s local, remote-cache ... (38 actions, 28 running) [82 / 309] SkylarkAction external/bazel_tools/tools/jdk/platformclasspath_classes/DumpPlatformClassPath.class [for host]; 5s local, remote-cache ... (194 actions, 46 running) [95 / 309] [Scann] Compiling external/zlib/infback.c [for host]; 9s ... (183 actions, 48 running) [106 / 309] [Scann] Compiling external/zlib/infback.c [for host]; 15s ... (172 actions, 48 running) [126 / 309] [Scann] Compiling external/zlib/infback.c [for host]; 21s ... (152 actions, 47 running) [146 / 309] [Scann] Compiling external/zlib/infback.c [for host]; 28s ... (135 actions, 48 running) [171 / 309] [Scann] Compiling external/zlib/infback.c [for host]; 36s ... (111 actions, 46 running) [212 / 309] [Scann] Compiling external/com_google_protobuf/src/google/protobuf/api.pb.cc [for host]; 46s ... (71 actions, 46 running) [275 / 325] Compiling external/com_google_protobuf/src/google/protobuf/descriptor.cc [for host]; 14s local, remote-cache ... (31 actions, 30 running) [321 / 325] [Prepa] Building plugins/high-availability/libhigh-availability__plugin-class.jar (80 source files) Target //plugins/high-availability:high-availability up-to-date: bazel-bin/plugins/high-availability/high-availability.jar INFO: Elapsed time: 102.339s, Critical Path: 57.01s INFO: 307 processes: 307 local. INFO: Build completed successfully, 325 total actions INFO: Build completed successfully, 325 total actions INFO: Invocation ID: cbe2e597-42ac-4461-8f86-b758f5f1899a 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 (20 packages loaded, 4151 targets configured) INFO: Analyzed target //plugins/high-availability:high_availability_tests (59 packages loaded, 4352 targets configured). INFO: Found 1 test target... [0 / 4] [Prepa] BazelWorkspaceStatusAction stable-status.txt [347 / 400] Building gerrit-common/libannotations.jar (3 source files); 1s remote-cache, worker ... (48 actions, 33 running) [352 / 403] Building gerrit-common/libannotations.jar (3 source files); 2s remote-cache, worker ... (48 actions, 31 running) [356 / 407] Building gerrit-common/libannotations.jar (3 source files); 3s remote-cache, worker ... (48 actions, 28 running) [357 / 408] Building gerrit-launcher/liblauncher.jar (1 source file); 4s remote-cache, worker ... (48 actions, 28 running) [361 / 410] Building gerrit-launcher/liblauncher.jar (1 source file); 5s remote-cache, worker ... (48 actions, 29 running) [364 / 411] Building gerrit-sshd/libsshd.jar (98 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 6s remote-cache ... (48 actions, 29 running) [367 / 415] Building gerrit-pgm/libutil-nodep.jar (12 source files); 7s remote-cache ... (48 actions, 29 running) [371 / 418] Building gerrit-oauth/liboauth-class.jar (4 source files); 8s remote-cache ... (48 actions, 28 running) [376 / 419] Building gerrit-cache-h2/libcache-h2.jar (4 source files); 10s remote-cache ... (48 actions, 29 running) [378 / 420] Building gerrit-gpg/libgpg.jar (18 source files); 12s remote-cache ... (48 actions, 29 running) [380 / 420] Building gerrit-reviewdb/libserver-class.jar (49 source files); 14s remote-cache ... (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) [384 / 422] Building gerrit-reviewdb/libclient-class.jar (32 source files); 16s remote-cache ... (48 actions, 29 running) 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)) {'? [390 / 424] Building gerrit-pgm/libinit-api.jar (14 source files); 18s remote-cache ... (48 actions, 30 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());'? 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;'? [398 / 426] Building external/com_google_protobuf/libprotobuf_java.jar (122 source files, 1 source jar); 19s remote-cache, worker ... (44 actions, 26 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) 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-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)) {'? [407 / 426] Building external/com_google_protobuf/libprotobuf_java.jar (122 source files, 1 source jar); 23s remote-cache, worker ... (28 actions, 18 running) INFO: From Building external/com_google_protobuf/libprotobuf_java.jar (122 source files, 1 source jar): warning: -parameters is not supported for target value 1.7. Use 1.8 or later. INFO: From Building gerrit-server/libtestutil.jar (22 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-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")'? [421 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 12s remote-cache, worker ... (6 actions 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) [424 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 17s remote-cache, worker [424 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 24s 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_154.java:54: warning: [MutableConstantField] Constant field declarations should use the immutable type (such as ImmutableList) instead of the general collection interface type (such as List) private static final Map ACCOUNT_FIELDS_MAP = ^ (see https://errorprone.info/bugpattern/MutableConstantField) Did you mean 'private static final ImmutableMap ACCOUNT_FIELDS_MAP ='? 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; 9s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 18s 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: 75.316s, Critical Path: 71.46s INFO: 105 processes: 1 local, 61 processwrapper-sandbox, 43 worker. INFO: Build completed successfully, 110 total actions //plugins/high-availability:high_availability_tests PASSED in 26.5s Executed 1 out of 1 test: 1 test passes. There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are. INFO: Build completed successfully, 110 total actions Jun 13, 2019 7:03:39 AM 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. Jun 13, 2019 7:03:40 AM 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/jenkins6158875145746448484.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 43590 0 --:--:-- --:--:-- --:--:-- 43878 Archiving artifacts Finished: SUCCESS