Started by user ha:////4LhIf0vdZMLFequGTVnN1cMcIypvsThRPBC5Va3yiaavAAAAnh+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAy+EgZh/dLi1CL9nNLkxNxMoEmpxZn5ANPJiavIAAAALuca Milanesio [EnvInject] - Loading node environment variables. Building remotely on ha:////4F6yRHFaENEo4ocVoCvEcGd+RBiwBV3EfiElHnlpp0L9AAAAqx+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAzREgZ5/eT83ILSktQi/fTknPzSFDMzXZM0EzPjpETzRHNLIwCXJN9k1AAAAA==gcloud66-4f463ba7a792 (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 f44f3f09e93d928d75d99b935effdbb158eacaa7 # timeout=10 [plugin-high-availability-gerritforge-stable-2.15] $ /bin/bash -e /tmp/jenkins6077612408281265943.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: 4211ba42-5876-4ed7-88ca-9260c9d85524 Loading: 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 (20 packages loaded, 313 targets configured) Analyzing: target //plugins/high-availability:high-availability (113 packages loaded, 2858 targets configured) Analyzing: target //plugins/high-availability:high-availability (142 packages loaded, 2957 targets configured) Analyzing: target //plugins/high-availability:high-availability (142 packages loaded, 2957 targets configured) Analyzing: target //plugins/high-availability:high-availability (144 packages loaded, 3533 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 / 17] [Prepa] BazelWorkspaceStatusAction stable-status.txt ... (2 actions, 0 running) [44 / 253] Extracting interface @user//jar:jar; 0s local, remote-cache ... (12 actions, 11 running) [260 / 325] Extracting interface //lib/lucene:lucene-core-and-backward-codecs; 1s local, remote-cache ... (31 actions, 24 running) [260 / 325] Extracting interface //lib/lucene:lucene-core-and-backward-codecs; 1s local, remote-cache ... (31 actions, 26 running) [291 / 325] Executing genrule //gerrit-server:prolog-common__pl2j; 2s local, remote-cache ... (17 actions, 16 running) [307 / 325] Compiling Java headers gerrit-common/libserver-hjar.jar (56 files); 0s local, remote-cache ... (3 actions, 2 running) [311 / 325] Compiling Java headers gerrit-lucene/libquery_builder-hjar.jar (1 files); 0s local, remote-cache ... (10 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: 29.638s, Critical Path: 16.14s 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: 94fa88c8-790e-4634-bc8c-5f1e24d007a0 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 (38 packages loaded, 4245 targets configured) INFO: Analyzed target //plugins/high-availability:high_availability_tests (59 packages loaded, 4353 targets configured). INFO: Found 1 test target... [0 / 3] [Prepa] BazelWorkspaceStatusAction stable-status.txt [351 / 408] Building gerrit-patch-jgit/libserver.jar (4 source files); 1s remote-cache, worker ... (47 actions, 31 running) [359 / 411] Building gerrit-patch-jgit/libserver.jar (4 source files); 2s remote-cache, worker ... (48 actions, 30 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) [364 / 416] Building gerrit-launcher/liblauncher.jar (1 source file); 3s remote-cache, worker ... (48 actions, 28 running) [367 / 419] Building gerrit-server/libconstants.jar (1 source file); 4s remote-cache ... (48 actions, 28 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());'? [369 / 420] Building external/com_google_protobuf/libprotobuf_java.jar (122 source files, 1 source jar); 5s remote-cache ... (48 actions, 28 running) [372 / 420] Building gerrit-gwtexpui/libserver.jar (2 source files); 6s remote-cache ... (48 actions, 29 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")'? [377 / 421] Building gerrit-gwtexpui/liblinker_server.jar (1 source file); 7s remote-cache ... (48 actions, 29 running) [385 / 423] Building gerrit-lucene/liblucene.jar (11 source files); 8s remote-cache ... (48 actions, 30 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)) {'? [389 / 424] Building gerrit-util-ssl/libssl.jar (1 source file); 9s remote-cache ... (48 actions, 30 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)) {'? [397 / 426] Building gerrit-pgm/libutil-nodep.jar (12 source files); 9s remote-cache ... (42 actions, 26 running) [402 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 10s remote-cache, worker ... (34 actions, 21 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;'? 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) [409 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 13s remote-cache, worker ... (22 actions, 15 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) [420 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 16s remote-cache, worker ... (8 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); 19s remote-cache, worker [424 / 426] Building gerrit-server/libserver-class.jar (1398 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 23s 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();'? [425 / 426] [Prepa] Building Java resource jar [426 / 427] Testing //plugins/high-availability:high_availability_tests; 9s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 15s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 37s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 48s local [426 / 427] Testing //plugins/high-availability:high_availability_tests; 73s 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: 107.593s, Critical Path: 105.79s 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 73.6s Executed 1 out of 1 test: 1 test passes. INFO: Build completed successfully, 110 total actions Aug 08, 2019 9:12:08 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 9:12:09 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/jenkins2396953299464799111.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 29167 0 --:--:-- --:--:-- --:--:-- 29491 Archiving artifacts Finished: SUCCESS