From c77790f55e90a4f66d3c439b583ad7fe0b9a34fd Mon Sep 17 00:00:00 2001 From: Googler Date: Tue, 14 Feb 2023 05:33:40 -0800 Subject: [PATCH] Flip flag --incompatible_remote_use_new_exit_code_for_lost_inputs so that when remote cache evicts blobs, Bazel will exit with code 39. RELNOTES[INC]: When remote cache evicts blobs, Bazel will exit with code 39. PiperOrigin-RevId: 509499179 Change-Id: Ic218cd5b91cfaa44cf96742325af6badd87abce4 --- .../google/devtools/build/lib/remote/options/RemoteOptions.java | 2 +- .../build/lib/remote/BuildWithoutTheBytesIntegrationTest.java | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/main/java/com/google/devtools/build/lib/remote/options/RemoteOptions.java b/src/main/java/com/google/devtools/build/lib/remote/options/RemoteOptions.java index ba8488382cb3b1..e82b230dbc0188 100644 --- a/src/main/java/com/google/devtools/build/lib/remote/options/RemoteOptions.java +++ b/src/main/java/com/google/devtools/build/lib/remote/options/RemoteOptions.java @@ -660,7 +660,7 @@ public RemoteOutputsStrategyConverter() { @Option( name = "incompatible_remote_use_new_exit_code_for_lost_inputs", - defaultValue = "false", + defaultValue = "true", documentationCategory = OptionDocumentationCategory.REMOTE, effectTags = {OptionEffectTag.UNKNOWN}, metadataTags = {OptionMetadataTag.INCOMPATIBLE_CHANGE}, diff --git a/src/test/java/com/google/devtools/build/lib/remote/BuildWithoutTheBytesIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/remote/BuildWithoutTheBytesIntegrationTest.java index 3709b2c9003300..957ec4d68f8c5f 100644 --- a/src/test/java/com/google/devtools/build/lib/remote/BuildWithoutTheBytesIntegrationTest.java +++ b/src/test/java/com/google/devtools/build/lib/remote/BuildWithoutTheBytesIntegrationTest.java @@ -446,7 +446,6 @@ public void remoteCacheEvictBlobs_exitWithCode39() throws Exception { getOutputPath("a/bar.out").delete(); getOutputBase().getRelative("action_cache").deleteTreesBelow(); restartServer(); - addOptions("--incompatible_remote_use_new_exit_code_for_lost_inputs"); // Clean build, foo.out isn't downloaded buildTarget("//a:bar");