Started by an SCM change Running as SYSTEM [EnvInject] - Loading node environment variables. Building remotely on ha:////4K35RsM8qyf0KD/aCRRqknP1t6WdQCx+G50c+idKYu3rAAAAqx+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAzREgZ5/eT83ILSktQi/fTknPzSFDMT3ZRkAwtjI8OkZFNjQwCyaXcc1AAAAA==gcloud64-dc08321bc531 (bazel-debian mvn bazel-chrome69 docker) in workspace /home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16 Selected Git installation does not exist. Using Default Wiping out workspace first. Cloning the remote Git repository Avoid fetching tags Cloning repository https://gerrit.googlesource.com/a/gerrit > /usr/bin/git init /home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16 # timeout=10 Fetching upstream changes from https://gerrit.googlesource.com/a/gerrit > /usr/bin/git --version # timeout=10 using GIT_ASKPASS to set credentials .netrc credentials for gerrit.googlesource.com > /usr/bin/git fetch --no-tags --force --progress -- https://gerrit.googlesource.com/a/gerrit +refs/heads/*:refs/remotes/gerrit/* # timeout=10 > /usr/bin/git config remote.gerrit.url https://gerrit.googlesource.com/a/gerrit # timeout=10 > /usr/bin/git config --add remote.gerrit.fetch +refs/heads/*:refs/remotes/gerrit/* # timeout=10 > /usr/bin/git config remote.gerrit.url https://gerrit.googlesource.com/a/gerrit # timeout=10 Fetching upstream changes from https://gerrit.googlesource.com/a/gerrit using GIT_ASKPASS to set credentials .netrc credentials for gerrit.googlesource.com > /usr/bin/git fetch --no-tags --force --progress -- https://gerrit.googlesource.com/a/gerrit +refs/heads/*:refs/remotes/gerrit/* # timeout=10 > /usr/bin/git config remote.origin.url https://gerrit.googlesource.com/a/plugins/webhooks # timeout=10 Fetching upstream changes from https://gerrit.googlesource.com/a/plugins/webhooks using GIT_ASKPASS to set credentials .netrc credentials for gerrit.googlesource.com > /usr/bin/git fetch --no-tags --force --progress -- https://gerrit.googlesource.com/a/plugins/webhooks +refs/heads/*:refs/remotes/origin/* # timeout=10 > /usr/bin/git rev-parse origin/stable-2.16^{commit} # timeout=10 > /usr/bin/git rev-parse refs/remotes/gerrit/origin/stable-2.16^{commit} # timeout=10 > /usr/bin/git rev-parse refs/remotes/origin/stable-2.16^{commit} # timeout=10 > /usr/bin/git rev-parse refs/remotes/origin/origin/stable-2.16^{commit} # timeout=10 Checking out Revision f9622383649b111f0fe8b64e91a9210cc94a6f47 (origin/stable-2.16) > /usr/bin/git config core.sparsecheckout # timeout=10 > /usr/bin/git checkout -f f9622383649b111f0fe8b64e91a9210cc94a6f47 # timeout=10 Commit message: "Upgrade bazlets to latest stable-2.16 to build with 2.16.26 API" > /usr/bin/git rev-list --no-walk 8fc0540cf246903994c5ab06ad11401b6ea73e30 # timeout=10 [plugin-webhooks-bazel-stable-2.16] $ /bin/bash -e /tmp/jenkins314054154301993098.sh [plugin-webhooks-bazel-stable-2.16] $ /bin/bash -e /tmp/jenkins3923875168995072942.sh Java set to: /usr/lib/jvm/java-8-openjdk-amd64/bin/java Previous HEAD position was f962238364 Upgrade bazlets to latest stable-2.16 to build with 2.16.26 API Switched to a new branch 'stable-2.16' Branch 'stable-2.16' set up to track remote branch 'stable-2.16' from 'gerrit'. Submodule 'plugins/codemirror-editor' (https://gerrit.googlesource.com/a/plugins/codemirror-editor) registered for path 'plugins/codemirror-editor' Submodule 'plugins/commit-message-length-validator' (https://gerrit.googlesource.com/a/plugins/commit-message-length-validator) registered for path 'plugins/commit-message-length-validator' Submodule 'plugins/download-commands' (https://gerrit.googlesource.com/a/plugins/download-commands) registered for path 'plugins/download-commands' Submodule 'plugins/hooks' (https://gerrit.googlesource.com/a/plugins/hooks) registered for path 'plugins/hooks' Submodule 'plugins/replication' (https://gerrit.googlesource.com/a/plugins/replication) registered for path 'plugins/replication' Submodule 'plugins/reviewnotes' (https://gerrit.googlesource.com/a/plugins/reviewnotes) registered for path 'plugins/reviewnotes' Submodule 'plugins/singleusergroup' (https://gerrit.googlesource.com/a/plugins/singleusergroup) registered for path 'plugins/singleusergroup' Cloning into '/home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16/plugins/codemirror-editor'... Cloning into '/home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16/plugins/commit-message-length-validator'... Cloning into '/home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16/plugins/download-commands'... Cloning into '/home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16/plugins/hooks'... Cloning into '/home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16/plugins/replication'... Cloning into '/home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16/plugins/reviewnotes'... Cloning into '/home/jenkins/workspace/plugin-webhooks-bazel-stable-2.16/plugins/singleusergroup'... Submodule path 'plugins/codemirror-editor': checked out '59942b1adf1c949f3633f60ac42f67fae03b3255' Submodule path 'plugins/commit-message-length-validator': checked out '556e427fd737744ce8a6a37b89fd427ae59bc8ea' Submodule path 'plugins/download-commands': checked out '41c61bf8c1869bff4e0b436f69478c2137d0ca07' Submodule path 'plugins/hooks': checked out 'e7769bd52c2b709d7712a386aa6cb96aec2c11a9' Submodule path 'plugins/replication': checked out '4a44e534128a0415ebca61ad7a6751905f0f4242' Submodule path 'plugins/reviewnotes': checked out 'ba3a422f1909f4970c563212d45cce70d494f60b' Submodule path 'plugins/singleusergroup': checked out '2a38a8413c25fec910fb843932aa869647bdc1e7' From https://gerrit.googlesource.com/a/plugins/webhooks * branch HEAD -> FETCH_HEAD * [new tag] v3.0.0 -> v3.0.0 * [new tag] v3.0.0-rc0 -> v3.0.0-rc0 * [new tag] v3.0.0-rc1 -> v3.0.0-rc1 * [new tag] v3.0.0-rc2 -> v3.0.0-rc2 * [new tag] v3.0.0-rc3 -> v3.0.0-rc3 * [new tag] v3.0.1 -> v3.0.1 * [new tag] v3.0.10 -> v3.0.10 * [new tag] v3.0.11 -> v3.0.11 * [new tag] v3.0.12 -> v3.0.12 * [new tag] v3.0.13 -> v3.0.13 * [new tag] v3.0.16 -> v3.0.16 * [new tag] v3.0.2 -> v3.0.2 * [new tag] v3.0.3 -> v3.0.3 * [new tag] v3.0.4 -> v3.0.4 * [new tag] v3.0.5 -> v3.0.5 * [new tag] v3.0.6 -> v3.0.6 * [new tag] v3.0.7 -> v3.0.7 * [new tag] v3.0.8 -> v3.0.8 * [new tag] v3.0.9 -> v3.0.9 * [new tag] v3.1.0 -> v3.1.0 * [new tag] v3.1.0-rc0 -> v3.1.0-rc0 * [new tag] v3.1.0-rc1 -> v3.1.0-rc1 * [new tag] v3.1.0-rc2 -> v3.1.0-rc2 * [new tag] v3.1.0-rc3 -> v3.1.0-rc3 * [new tag] v3.1.1 -> v3.1.1 * [new tag] v3.1.11 -> v3.1.11 * [new tag] v3.1.12 -> v3.1.12 * [new tag] v3.1.2 -> v3.1.2 * [new tag] v3.1.3 -> v3.1.3 * [new tag] v3.1.4 -> v3.1.4 * [new tag] v3.1.5 -> v3.1.5 * [new tag] v3.1.6 -> v3.1.6 * [new tag] v3.1.7 -> v3.1.7 * [new tag] v3.1.8 -> v3.1.8 * [new tag] v3.2.0 -> v3.2.0 * [new tag] v3.2.0-rc0 -> v3.2.0-rc0 * [new tag] v3.2.0-rc1 -> v3.2.0-rc1 * [new tag] v3.2.0-rc2 -> v3.2.0-rc2 * [new tag] v3.2.0-rc3 -> v3.2.0-rc3 * [new tag] v3.2.0-rc4 -> v3.2.0-rc4 * [new tag] v3.2.1 -> v3.2.1 * [new tag] v3.2.2 -> v3.2.2 * [new tag] v3.2.3 -> v3.2.3 * [new tag] v3.2.5.1 -> v3.2.5.1 * [new tag] v3.2.6 -> v3.2.6 * [new tag] v3.2.7 -> v3.2.7 * [new tag] v3.3.0 -> v3.3.0 * [new tag] v3.3.0-rc0 -> v3.3.0-rc0 * [new tag] v3.3.0-rc1 -> v3.3.0-rc1 * [new tag] v3.3.0-rc2 -> v3.3.0-rc2 * [new tag] v3.3.0-rc3 -> v3.3.0-rc3 * [new tag] v3.3.0-rc4 -> v3.3.0-rc4 * [new tag] v3.3.0-rc5 -> v3.3.0-rc5 * [new tag] v3.3.0-rc6 -> v3.3.0-rc6 * [new tag] v3.3.0-rc7 -> v3.3.0-rc7 * [new tag] v3.3.1 -> v3.3.1 * [new tag] v3.3.2 -> v3.3.2 openjdk full version "1.8.0_275-b01" 2021/02/04 11:24:12 Downloading https://releases.bazel.build/3.1.0/release/bazel-3.1.0-linux-x86_64... Bazelisk version: v1.7.4 Extracting Bazel installation... Starting local Bazel server and connecting to it... Build label: 3.1.0 Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar Build time: Wed Apr 22 10:32:27 2020 (1587551547) Build timestamp: 1587551547 Build timestamp as int: 1587551547 INFO: Invocation ID: 285d8319-5514-4b29-8f09-40d15384fc8c Loading: Loading: 0 packages loaded Loading: 0 packages loaded Loading: 0 packages loaded Loading: 0 packages loaded Loading: 0 packages loaded Loading: 0 packages loaded currently loading: plugins/webhooks Analyzing: target //plugins/webhooks:webhooks (1 packages loaded, 0 targets configured) Analyzing: target //plugins/webhooks:webhooks (6 packages loaded, 7 targets configured) Analyzing: target //plugins/webhooks:webhooks (10 packages loaded, 7 targets configured) Analyzing: target //plugins/webhooks:webhooks (10 packages loaded, 7 targets configured) Analyzing: target //plugins/webhooks:webhooks (10 packages loaded, 7 targets configured) Analyzing: target //plugins/webhooks:webhooks (10 packages loaded, 7 targets configured) Analyzing: target //plugins/webhooks:webhooks (11 packages loaded, 7 targets configured) Analyzing: target //plugins/webhooks:webhooks (29 packages loaded, 254 targets configured) Analyzing: target //plugins/webhooks:webhooks (114 packages loaded, 3007 targets configured) Analyzing: target //plugins/webhooks:webhooks (162 packages loaded, 3108 targets configured) Analyzing: target //plugins/webhooks:webhooks (172 packages loaded, 3130 targets configured) Analyzing: target //plugins/webhooks:webhooks (198 packages loaded, 3794 targets configured) INFO: Analyzed target //plugins/webhooks:webhooks (199 packages loaded, 4210 targets configured). INFO: Found 1 target... [0 / 4] [Prepa] BazelWorkspaceStatusAction stable-status.txt [59 / 341] Compiling external/com_google_protobuf/src/google/protobuf/parse_context.cc [for host]; 5s remote-cache, processwrapper-sandbox ... (29 actions, 28 running) [84 / 341] Compiling external/com_google_protobuf/src/google/protobuf/generated_message_table_driven_lite.cc [for host]; 11s remote-cache, processwrapper-sandbox ... (31 actions, 30 running) [102 / 341] Compiling external/com_google_protobuf/src/google/protobuf/descriptor.cc [for host]; 17s remote-cache, processwrapper-sandbox ... (31 actions, 30 running) [120 / 341] Compiling external/com_google_protobuf/src/google/protobuf/descriptor.cc [for host]; 27s remote-cache, processwrapper-sandbox ... (31 actions, 30 running) [140 / 341] Compiling external/com_google_protobuf/src/google/protobuf/descriptor.cc [for host]; 40s remote-cache, processwrapper-sandbox ... (31 actions, 30 running) [166 / 341] Compiling external/com_google_protobuf/src/google/protobuf/compiler/js/js_generator.cc [for host]; 27s remote-cache, processwrapper-sandbox ... (31 actions, 30 running) [239 / 353] Compiling external/com_google_protobuf/src/google/protobuf/compiler/java/java_message.cc [for host]; 14s remote-cache, processwrapper-sandbox ... (25 actions, 24 running) [315 / 365] Compiling Java headers external/com_google_protobuf/libprotobuf_java-hjar.jar (123 source files, 1 source jar); 4s remote-cache, processwrapper-sandbox ... (21 actions, 20 running) [342 / 365] Executing genrule //prolog:gerrit-prolog-common__pl2j; 9s remote-cache, processwrapper-sandbox ... (6 actions running) [349 / 365] Compiling Java headers java/com/google/gerrit/lucene/liblucene-hjar.jar (12 source files); 1s remote-cache, processwrapper-sandbox ... (18 actions running) [361 / 365] Building plugins/webhooks/libwebhooks__plugin-class.jar (30 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 8s remote-cache, worker Target //plugins/webhooks:webhooks up-to-date: bazel-bin/plugins/webhooks/webhooks.jar INFO: Elapsed time: 218.357s, Critical Path: 113.68s INFO: 348 processes: 342 processwrapper-sandbox, 6 worker. INFO: Build completed successfully, 365 total actions INFO: Build completed successfully, 365 total actions INFO: Invocation ID: d7147370-8471-430b-97e5-985a910175cc Loading: Loading: 0 packages loaded INFO: Build option --test_env has changed, discarding analysis cache. Analyzing: 2 targets (3 packages loaded, 0 targets configured) Analyzing: 2 targets (5 packages loaded, 74 targets configured) Analyzing: 2 targets (6 packages loaded, 594 targets configured) Analyzing: 2 targets (22 packages loaded, 1930 targets configured) Analyzing: 2 targets (47 packages loaded, 4260 targets configured) Analyzing: 2 targets (60 packages loaded, 4626 targets configured) Analyzing: 2 targets (77 packages loaded, 4720 targets configured) INFO: Analyzed 2 targets (80 packages loaded, 4749 targets configured). INFO: Found 2 test targets... [0 / 1] [Prepa] BazelWorkspaceStatusAction stable-status.txt [324 / 501] 1 / 2 tests; [Prepa] BazelWorkspaceStatusAction stable-status.txt ... (28 actions, 19 running) [339 / 501] 1 / 2 tests; Building java/com/google/gerrit/common/libannotations.jar (3 source files); 2s remote-cache, worker ... (32 actions, 20 running) [353 / 502] 1 / 2 tests; Building java/com/google/gerrit/common/libannotations.jar (3 source files); 4s remote-cache, worker ... (32 actions, 20 running) [363 / 502] 1 / 2 tests; Building java/com/google/gerrit/server/libconstants.jar (1 source file); 6s remote-cache, worker ... (32 actions, 20 running) [376 / 502] 1 / 2 tests; Building proto/libcache_proto-speed.jar (1 source jar); 9s remote-cache ... (32 actions, 20 running) [385 / 502] 1 / 2 tests; Building java/com/google/gerrit/extensions/libapi.jar (329 source files); 12s remote-cache ... (32 actions, 21 running) [386 / 502] 1 / 2 tests; Building java/com/google/gerrit/reviewdb/libserver-class.jar (48 source files); 15s remote-cache ... (32 actions, 21 running) INFO: From Building java/com/google/gerrit/util/cli/libcli.jar (7 source files): java/com/google/gerrit/util/cli/CmdLineParser.java:333: 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) java/com/google/gerrit/util/cli/CmdLineParser.java:511: 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) java/com/google/gerrit/util/cli/CmdLineParser.java:512: 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;'? [388 / 502] 1 / 2 tests; Building java/com/google/gerrit/lifecycle/liblifecycle.jar (2 source files); 19s remote-cache ... (32 actions, 22 running) [390 / 502] 1 / 2 tests; Building java/com/google/gerrit/lifecycle/liblifecycle.jar (2 source files); 24s remote-cache ... (32 actions, 20 running) [390 / 502] 1 / 2 tests; Building java/com/google/gerrit/lifecycle/liblifecycle.jar (2 source files); 29s remote-cache ... (32 actions, 20 running) [398 / 502] 1 / 2 tests; Building java/com/google/gerrit/server/util/git/libgit.jar (1 source file); 34s remote-cache ... (32 actions, 22 running) INFO: From Building java/com/google/gerrit/extensions/libapi.jar (329 source files): 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) [412 / 502] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 39s remote-cache, worker ... (32 actions, 20 running) [440 / 502] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 47s remote-cache, worker ... (32 actions, 21 running) INFO: From Building java/com/google/gerrit/pgm/init/api/libapi.jar (14 source files): java/com/google/gerrit/pgm/init/api/Section.java:131: 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());'? [447 / 502] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 57s remote-cache, worker ... (32 actions, 20 running) INFO: From Building java/com/google/gerrit/lucene/liblucene.jar (12 source files): java/com/google/gerrit/lucene/WrappableSearcherManager.java:191: 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)) {'? java/com/google/gerrit/lucene/WrappableSearcherManager.java:202: 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)) {'? [464 / 502] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 68s remote-cache, worker ... (32 actions, 25 running) [484 / 502] 1 / 2 tests; Building java/com/google/gerrit/server/libserver-class.jar (990 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 32s remote-cache, worker ... (24 actions, 17 running) INFO: From Building java/com/google/gerrit/httpd/libhttpd-class.jar (115 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): java/com/google/gerrit/httpd/HttpServletResponseRecorder.java:74: warning: [MissingOverride] getStatus overrides method in HttpServletResponseWrapper; expected @Override public int getStatus() { ^ (see https://errorprone.info/bugpattern/MissingOverride) Did you mean '@Override @SuppressWarnings("all")'? 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")'? INFO: From Building java/com/google/gerrit/sshd/libsshd.jar (101 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): java/com/google/gerrit/sshd/commands/ApproveOption.java:32: 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) java/com/google/gerrit/sshd/commands/ApproveOption.java:35: 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) java/com/google/gerrit/sshd/commands/ApproveOption.java:37: 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;'? java/com/google/gerrit/sshd/commands/StreamEvents.java:284: 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) java/com/google/gerrit/sshd/commands/StreamEvents.java:291: 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) java/com/google/gerrit/sshd/commands/UploadArchive.java:142: 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)) {'? java/com/google/gerrit/sshd/commands/UploadArchive.java:142: 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)) {'? INFO: From Building java/com/google/gerrit/acceptance/libframework-lib.jar (61 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): java/com/google/gerrit/acceptance/GerritServer.java:461: 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) 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) [500 / 502] 1 / 2 tests; Building java/com/google/gerrit/server/libserver-class.jar (990 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 46s remote-cache, worker INFO: From Building java/com/google/gerrit/server/libserver-class.jar (990 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor): 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) java/com/google/gerrit/server/index/change/ChangeIndexRewriter.java:158: 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) {'? java/com/google/gerrit/server/index/change/ChangeIndexRewriter.java:210: 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)) {'? java/com/google/gerrit/server/util/IdGenerator.java:48: 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));'? java/com/google/gerrit/server/util/IdGenerator.java:49: 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));'? java/com/google/gerrit/server/util/IdGenerator.java:57: 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));'? 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));'? 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) java/com/google/gerrit/server/change/ChangeKindCacheImpl.java:386: warning: [ReferenceEquality] Comparison using reference equality instead of value equality && (ps.getId().get() > priorPs.getId().get() || priorPs == patch)) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean '&& (ps.getId().get() > priorPs.getId().get() || priorPs.equals(patch))) {'? java/com/google/gerrit/server/change/ChangeKindCacheImpl.java:396: warning: [ReferenceEquality] Comparison using reference equality instead of value equality if (priorPs != patch) { ^ (see https://errorprone.info/bugpattern/ReferenceEquality) Did you mean 'if (!priorPs.equals(patch)) {'? java/com/google/gerrit/server/index/AbstractIndexModule.java:78: 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 createDummyIndexFactory(Schema schema) { ^ (see https://errorprone.info/bugpattern/TypeParameterUnusedInFormals) 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) java/com/google/gerrit/server/patch/PatchScriptBuilder.java:504: 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))) {'? java/com/google/gerrit/server/patch/PatchScriptBuilder.java:524: 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))) {'? INFO: Elapsed time: 128.468s, Critical Path: 90.73s INFO: 138 processes: 1 local, 72 processwrapper-sandbox, 65 worker. INFO: Build completed successfully, 145 total actions //plugins/webhooks:webhooks_tests PASSED in 12.8s //tools/bzl:always_pass_test PASSED in 0.3s Executed 2 out of 2 tests: 2 tests pass. 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, 145 total actions Feb 04, 2021 11:30:09 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-webhooks-bazel-stable-2.16] $ /bin/bash -e /tmp/jenkins1746200645120220005.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 1619 0 1619 0 0 5239 0 --:--:-- --:--:-- --:--:-- 5239 Archiving artifacts Finished: SUCCESS