| // Copyright 2017 The Bazel Authors. All rights reserved. | |
| // | |
| // Licensed under the Apache License, Version 2.0 (the "License"); | |
| // you may not use this file except in compliance with the License. | |
| // You may obtain a copy of the License at | |
| // | |
| // http://www.apache.org/licenses/LICENSE-2.0 | |
| // | |
| // Unless required by applicable law or agreed to in writing, software | |
| // distributed under the License is distributed on an "AS IS" BASIS, | |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | |
| // See the License for the specific language governing permissions and | |
| // limitations under the License. | |
| package com.google.devtools.build.lib.actions; | |
| import static com.google.common.truth.Truth.assertThat; | |
| import static com.google.devtools.build.lib.actions.util.ActionsTestUtil.NULL_ARTIFACT_OWNER; | |
| import static com.google.devtools.build.lib.actions.util.ActionsTestUtil.createArtifact; | |
| import static com.google.devtools.build.lib.actions.util.ActionsTestUtil.createTreeArtifactWithGeneratingAction; | |
| import static com.google.devtools.build.lib.vfs.FileSystemUtils.readContent; | |
| import static com.google.devtools.build.lib.vfs.FileSystemUtils.writeIsoLatin1; | |
| import static java.nio.charset.StandardCharsets.UTF_8; | |
| import static org.mockito.ArgumentMatchers.any; | |
| import static org.mockito.ArgumentMatchers.argThat; | |
| import static org.mockito.Mockito.doReturn; | |
| import static org.mockito.Mockito.mock; | |
| import static org.mockito.Mockito.never; | |
| import static org.mockito.Mockito.verify; | |
| import static org.mockito.Mockito.when; | |
| import com.google.common.collect.ImmutableList; | |
| import com.google.common.collect.ImmutableMap; | |
| import com.google.devtools.build.lib.actions.ActionCacheChecker.Token; | |
| import com.google.devtools.build.lib.actions.Artifact.ArchivedTreeArtifact; | |
| import com.google.devtools.build.lib.actions.Artifact.SpecialArtifact; | |
| import com.google.devtools.build.lib.actions.Artifact.SpecialArtifactType; | |
| import com.google.devtools.build.lib.actions.ArtifactRoot.RootType; | |
| import com.google.devtools.build.lib.actions.FileArtifactValue.RemoteFileArtifactValue; | |
| import com.google.devtools.build.lib.actions.cache.ActionCache; | |
| import com.google.devtools.build.lib.actions.cache.ActionCache.Entry.SerializableTreeArtifactValue; | |
| import com.google.devtools.build.lib.actions.cache.CompactPersistentActionCache; | |
| import com.google.devtools.build.lib.actions.cache.OutputMetadataStore; | |
| import com.google.devtools.build.lib.actions.cache.Protos.ActionCacheStatistics; | |
| import com.google.devtools.build.lib.actions.cache.Protos.ActionCacheStatistics.MissDetail; | |
| import com.google.devtools.build.lib.actions.cache.Protos.ActionCacheStatistics.MissReason; | |
| import com.google.devtools.build.lib.actions.util.ActionsTestUtil; | |
| import com.google.devtools.build.lib.actions.util.ActionsTestUtil.FakeArtifactResolverBase; | |
| import com.google.devtools.build.lib.actions.util.ActionsTestUtil.FakeInputMetadataHandlerBase; | |
| import com.google.devtools.build.lib.actions.util.ActionsTestUtil.MissDetailsBuilder; | |
| import com.google.devtools.build.lib.actions.util.ActionsTestUtil.NullAction; | |
| import com.google.devtools.build.lib.clock.Clock; | |
| import com.google.devtools.build.lib.collect.nestedset.NestedSet; | |
| import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder; | |
| import com.google.devtools.build.lib.collect.nestedset.Order; | |
| import com.google.devtools.build.lib.events.NullEventHandler; | |
| import com.google.devtools.build.lib.skyframe.TreeArtifactValue; | |
| import com.google.devtools.build.lib.testutil.ManualClock; | |
| import com.google.devtools.build.lib.testutil.Scratch; | |
| import com.google.devtools.build.lib.util.Fingerprint; | |
| import com.google.devtools.build.lib.vfs.DigestHashFunction; | |
| import com.google.devtools.build.lib.vfs.Dirent; | |
| import com.google.devtools.build.lib.vfs.FileSystem; | |
| import com.google.devtools.build.lib.vfs.FileSystemUtils; | |
| import com.google.devtools.build.lib.vfs.OutputPermissions; | |
| import com.google.devtools.build.lib.vfs.Path; | |
| import com.google.devtools.build.lib.vfs.PathFragment; | |
| import com.google.devtools.build.lib.vfs.Root; | |
| import com.google.devtools.build.lib.vfs.inmemoryfs.InMemoryFileSystem; | |
| import com.google.testing.junit.testparameterinjector.TestParameter; | |
| import com.google.testing.junit.testparameterinjector.TestParameterInjector; | |
| import java.io.IOException; | |
| import java.io.PrintStream; | |
| import java.time.Instant; | |
| import java.util.ArrayDeque; | |
| import java.util.Deque; | |
| import java.util.HashMap; | |
| import java.util.HashSet; | |
| import java.util.Map; | |
| import java.util.Optional; | |
| import java.util.Set; | |
| import java.util.function.Predicate; | |
| import javax.annotation.Nullable; | |
| import org.junit.After; | |
| import org.junit.Before; | |
| import org.junit.Test; | |
| import org.junit.runner.RunWith; | |
| @RunWith(TestParameterInjector.class) | |
| public class ActionCacheCheckerTest { | |
| private static final RemoteArtifactChecker CHECK_TTL = | |
| (file, metadata) -> metadata.isAlive(Instant.now()); | |
| private CorruptibleActionCache cache; | |
| private ActionCacheChecker cacheChecker; | |
| private Set<Path> filesToDelete; | |
| private DigestHashFunction digestHashFunction; | |
| private FileSystem fileSystem; | |
| private ArtifactRoot artifactRoot; | |
| @Before | |
| public void setupCache() throws Exception { | |
| Scratch scratch = new Scratch(); | |
| Clock clock = new ManualClock(); | |
| cache = new CorruptibleActionCache(scratch.resolve("/cache/test.dat"), clock); | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ false); | |
| digestHashFunction = DigestHashFunction.SHA256; | |
| fileSystem = new InMemoryFileSystem(digestHashFunction); | |
| Path execRoot = fileSystem.getPath("/output"); | |
| artifactRoot = ArtifactRoot.asDerivedRoot(execRoot, RootType.Output, "bin"); | |
| } | |
| private byte[] digest(byte[] content) { | |
| return digestHashFunction.getHashFunction().hashBytes(content).asBytes(); | |
| } | |
| private ActionCacheChecker createActionCacheChecker(boolean storeOutputMetadata) { | |
| return new ActionCacheChecker( | |
| cache, | |
| new FakeArtifactResolverBase(), | |
| new ActionKeyContext(), | |
| action -> true, | |
| ActionCacheChecker.CacheConfig.builder() | |
| .setEnabled(true) | |
| .setVerboseExplanations(false) | |
| .setStoreOutputMetadata(storeOutputMetadata) | |
| .build()); | |
| } | |
| @Before | |
| public void clearFilesToDeleteAfterTest() throws Exception { | |
| filesToDelete = new HashSet<>(); | |
| } | |
| @After | |
| public void deleteFilesCreatedDuringTest() throws Exception { | |
| for (Path path : filesToDelete) { | |
| if (path.isDirectory()) { | |
| path.deleteTree(); | |
| } else { | |
| path.delete(); | |
| } | |
| } | |
| } | |
| /** "Executes" the given action from the point of view of the cache's lifecycle. */ | |
| private void runAction(Action action) throws Exception { | |
| runAction(action, new HashMap<>()); | |
| } | |
| private void runAction( | |
| Action action, | |
| InputMetadataProvider inputMetadataProvider, | |
| OutputMetadataStore outputMetadataStore) | |
| throws Exception { | |
| runAction( | |
| action, new HashMap<>(), ImmutableMap.of(), inputMetadataProvider, outputMetadataStore); | |
| } | |
| /** | |
| * "Executes" the given action from the point of view of the cache's lifecycle with a custom | |
| * client environment. | |
| */ | |
| private void runAction(Action action, Map<String, String> clientEnv) throws Exception { | |
| runAction(action, clientEnv, ImmutableMap.of()); | |
| } | |
| private void runAction(Action action, Map<String, String> clientEnv, Map<String, String> platform) | |
| throws Exception { | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action, clientEnv, platform, metadataHandler, metadataHandler); | |
| } | |
| private void runAction( | |
| Action action, | |
| Map<String, String> clientEnv, | |
| Map<String, String> platform, | |
| InputMetadataProvider inputMetadataProvider, | |
| OutputMetadataStore outputMetadataStore) | |
| throws Exception { | |
| runAction( | |
| action, | |
| clientEnv, | |
| platform, | |
| inputMetadataProvider, | |
| outputMetadataStore, | |
| RemoteArtifactChecker.TRUST_ALL); | |
| } | |
| private void runAction( | |
| Action action, | |
| Map<String, String> clientEnv, | |
| Map<String, String> platform, | |
| InputMetadataProvider inputMetadataProvider, | |
| OutputMetadataStore outputMetadataStore, | |
| RemoteArtifactChecker remoteArtifactChecker) | |
| throws Exception { | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| clientEnv, | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| inputMetadataProvider, | |
| outputMetadataStore, | |
| /* artifactExpander= */ null, | |
| platform, | |
| remoteArtifactChecker); | |
| runAction(action, clientEnv, platform, inputMetadataProvider, outputMetadataStore, token); | |
| } | |
| private void runAction( | |
| Action action, | |
| Map<String, String> clientEnv, | |
| Map<String, String> platform, | |
| InputMetadataProvider inputMetadataProvider, | |
| OutputMetadataStore outputMetadataStore, | |
| @Nullable Token token) | |
| throws Exception { | |
| if (token != null) { | |
| for (Artifact artifact : action.getOutputs()) { | |
| Path path = artifact.getPath(); | |
| // Record all action outputs as files to be deleted across tests to prevent cross-test | |
| // pollution. We need to do this on a path basis because we don't know upfront which file | |
| // system they live in so we cannot just recreate the file system. (E.g. all NullActions | |
| // share an in-memory file system to hold dummy outputs.) | |
| filesToDelete.add(path); | |
| Path parent = path.getParentDirectory(); | |
| if (parent != null) { | |
| parent.createDirectoryAndParents(); | |
| } | |
| } | |
| // Real action execution would happen here. | |
| ActionExecutionContext context = mock(ActionExecutionContext.class); | |
| when(context.getOutputMetadataStore()).thenReturn(outputMetadataStore); | |
| action.execute(context); | |
| cacheChecker.updateActionCache( | |
| action, | |
| token, | |
| inputMetadataProvider, | |
| outputMetadataStore, | |
| /* artifactExpander= */ null, | |
| clientEnv, | |
| OutputPermissions.READONLY, | |
| platform); | |
| } | |
| } | |
| /** Ensures that the cache statistics match exactly the given values. */ | |
| private void assertStatistics(int hits, Iterable<MissDetail> misses) { | |
| ActionCacheStatistics.Builder builder = ActionCacheStatistics.newBuilder(); | |
| cache.mergeIntoActionCacheStatistics(builder); | |
| ActionCacheStatistics stats = builder.build(); | |
| assertThat(stats.getHits()).isEqualTo(hits); | |
| assertThat(stats.getMissDetailsList()).containsExactlyElementsIn(misses); | |
| } | |
| private void doTestNotCached(Action action, MissReason missReason) throws Exception { | |
| runAction(action); | |
| assertStatistics(0, new MissDetailsBuilder().set(missReason, 1).build()); | |
| } | |
| private void doTestCached(Action action, MissReason missReason) throws Exception { | |
| int runs = 5; | |
| for (int i = 0; i < runs; i++) { | |
| runAction(action); | |
| } | |
| assertStatistics(runs - 1, new MissDetailsBuilder().set(missReason, 1).build()); | |
| } | |
| private void doTestCorruptedCacheEntry(Action action) throws Exception { | |
| cache.corruptAllEntries(); | |
| runAction(action); | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder().set(MissReason.CORRUPTED_CACHE_ENTRY, 1).build()); | |
| } | |
| @Test | |
| public void testNoActivity() throws Exception { | |
| assertStatistics(0, new MissDetailsBuilder().build()); | |
| } | |
| @Test | |
| public void testNotCached() throws Exception { | |
| doTestNotCached(new WriteEmptyOutputAction(), MissReason.NOT_CACHED); | |
| } | |
| @Test | |
| public void testCached() throws Exception { | |
| doTestCached(new WriteEmptyOutputAction(), MissReason.NOT_CACHED); | |
| } | |
| @Test | |
| public void testCorruptedCacheEntry() throws Exception { | |
| doTestCorruptedCacheEntry(new WriteEmptyOutputAction()); | |
| } | |
| @Test | |
| public void testDifferentActionKey() throws Exception { | |
| Action action = | |
| new WriteEmptyOutputAction() { | |
| @Override | |
| protected void computeKey( | |
| ActionKeyContext actionKeyContext, | |
| @Nullable ArtifactExpander artifactExpander, | |
| Fingerprint fp) { | |
| fp.addString("key1"); | |
| } | |
| }; | |
| runAction(action); | |
| action = | |
| new NullAction() { | |
| @Override | |
| protected void computeKey( | |
| ActionKeyContext actionKeyContext, | |
| @Nullable ArtifactExpander artifactExpander, | |
| Fingerprint fp) { | |
| fp.addString("key2"); | |
| } | |
| }; | |
| runAction(action); | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder() | |
| .set(MissReason.DIFFERENT_ACTION_KEY, 1) | |
| .set(MissReason.NOT_CACHED, 1) | |
| .build()); | |
| } | |
| @Test | |
| public void testDifferentEnvironment() throws Exception { | |
| Action action = | |
| new WriteEmptyOutputAction() { | |
| @Override | |
| public ImmutableList<String> getClientEnvironmentVariables() { | |
| return ImmutableList.of("used-var"); | |
| } | |
| }; | |
| Map<String, String> clientEnv = new HashMap<>(); | |
| clientEnv.put("unused-var", "1"); | |
| runAction(action, clientEnv); // Not cached. | |
| clientEnv.remove("unused-var"); | |
| runAction(action, clientEnv); // Cache hit because we only modified uninteresting variables. | |
| clientEnv.put("used-var", "2"); | |
| runAction(action, clientEnv); // Cache miss because of different environment. | |
| runAction(action, clientEnv); // Cache hit because we did not change anything. | |
| assertStatistics( | |
| 2, | |
| new MissDetailsBuilder() | |
| .set(MissReason.DIFFERENT_ENVIRONMENT, 1) | |
| .set(MissReason.NOT_CACHED, 1) | |
| .build()); | |
| } | |
| @Test | |
| public void testDifferentRemoteDefaultPlatform() throws Exception { | |
| Action action = new WriteEmptyOutputAction(); | |
| Map<String, String> env = new HashMap<>(); | |
| env.put("unused-var", "1"); | |
| Map<String, String> platform = new HashMap<>(); | |
| platform.put("used-var", "1"); | |
| // Not cached. | |
| runAction(action, env, platform); | |
| // Cache hit because nothing changed. | |
| runAction(action, env, platform); | |
| // Cache miss because platform changed to an empty from a previous value. | |
| runAction(action, env, ImmutableMap.of()); | |
| // Cache hit with an empty platform. | |
| runAction(action, env, ImmutableMap.of()); | |
| // Cache miss because platform changed to a value from an empty one. | |
| runAction(action, env, ImmutableMap.copyOf(platform)); | |
| platform.put("another-var", "1234"); | |
| // Cache miss because platform value changed. | |
| runAction(action, env, ImmutableMap.copyOf(platform)); | |
| assertStatistics( | |
| 2, | |
| new MissDetailsBuilder() | |
| .set(MissReason.DIFFERENT_ENVIRONMENT, 3) | |
| .set(MissReason.NOT_CACHED, 1) | |
| .build()); | |
| } | |
| @Test | |
| public void testDifferentFiles() throws Exception { | |
| Action action = new WriteEmptyOutputAction(); | |
| runAction(action); // Not cached. | |
| assertThat(readContent(action.getPrimaryOutput().getPath(), UTF_8)).isEmpty(); | |
| writeContentAsLatin1(action.getPrimaryOutput().getPath(), "modified"); | |
| runAction(action); // Cache miss because output files were modified. | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder() | |
| .set(MissReason.DIFFERENT_FILES, 1) | |
| .set(MissReason.NOT_CACHED, 1) | |
| .build()); | |
| } | |
| @Test | |
| public void testUnconditionalExecution() throws Exception { | |
| Action action = | |
| new WriteEmptyOutputAction() { | |
| @Override | |
| public boolean executeUnconditionally() { | |
| return true; | |
| } | |
| @Override | |
| public boolean isVolatile() { | |
| return true; | |
| } | |
| }; | |
| int runs = 5; | |
| for (int i = 0; i < runs; i++) { | |
| runAction(action); | |
| } | |
| assertStatistics( | |
| 0, new MissDetailsBuilder().set(MissReason.UNCONDITIONAL_EXECUTION, runs).build()); | |
| } | |
| @Test | |
| public void testMiddleman_notCached() throws Exception { | |
| doTestNotCached(new NullMiddlemanAction(), MissReason.DIFFERENT_DEPS); | |
| } | |
| @Test | |
| public void testMiddleman_cached() throws Exception { | |
| doTestCached(new NullMiddlemanAction(), MissReason.DIFFERENT_DEPS); | |
| } | |
| @Test | |
| public void testMiddleman_corruptedCacheEntry() throws Exception { | |
| doTestCorruptedCacheEntry(new NullMiddlemanAction()); | |
| } | |
| @Test | |
| public void testMiddleman_differentFiles() throws Exception { | |
| Action action = | |
| new NullMiddlemanAction() { | |
| @Override | |
| public synchronized NestedSet<Artifact> getInputs() { | |
| FileSystem fileSystem = getPrimaryOutput().getPath().getFileSystem(); | |
| Path path = fileSystem.getPath("/input"); | |
| ArtifactRoot root = ArtifactRoot.asSourceRoot(Root.fromPath(fileSystem.getPath("/"))); | |
| return NestedSetBuilder.create( | |
| Order.STABLE_ORDER, ActionsTestUtil.createArtifact(root, path)); | |
| } | |
| }; | |
| // Manually register outputs of middleman action in `filesToDelete` because it won't get a cache | |
| // token to execute and register in `runAction`. Failing to delete outputs may populate the | |
| // filesystem for other test cases. b/308017721 | |
| for (var output : action.getOutputs()) { | |
| filesToDelete.add(output.getPath()); | |
| } | |
| runAction(action); // Not cached so recorded as different deps. | |
| writeContentAsLatin1(action.getPrimaryInput().getPath(), "modified"); | |
| runAction(action); // Cache miss because input files were modified. | |
| writeContentAsLatin1(action.getPrimaryOutput().getPath(), "modified"); | |
| runAction(action); // Outputs are not considered for middleman actions, so this is a cache hit. | |
| runAction(action); // Outputs are not considered for middleman actions, so this is a cache hit. | |
| assertStatistics( | |
| 2, | |
| new MissDetailsBuilder() | |
| .set(MissReason.DIFFERENT_DEPS, 1) | |
| .set(MissReason.DIFFERENT_FILES, 1) | |
| .build()); | |
| } | |
| @Test | |
| public void testDeletedConstantMetadataOutputCausesReexecution() throws Exception { | |
| SpecialArtifact output = | |
| SpecialArtifact.create( | |
| artifactRoot, | |
| PathFragment.create("bin/dummy"), | |
| NULL_ARTIFACT_OWNER, | |
| SpecialArtifactType.CONSTANT_METADATA); | |
| output.getPath().getParentDirectory().createDirectoryAndParents(); | |
| Action action = new WriteEmptyOutputAction(output); | |
| runAction(action); | |
| output.getPath().delete(); | |
| FakeInputMetadataHandler fakeMetadataHandler = new FakeInputMetadataHandler(); | |
| assertThat( | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| fakeMetadataHandler, | |
| fakeMetadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| RemoteArtifactChecker.TRUST_ALL)) | |
| .isNotNull(); | |
| } | |
| private RemoteFileArtifactValue createRemoteFileMetadata(String content) { | |
| return createRemoteFileMetadata(content, /* materializationExecPath= */ null); | |
| } | |
| private RemoteFileArtifactValue createRemoteFileMetadata( | |
| String content, @Nullable PathFragment materializationExecPath) { | |
| byte[] bytes = content.getBytes(UTF_8); | |
| return RemoteFileArtifactValue.create( | |
| digest(bytes), bytes.length, 1, /* expireAtEpochMilli= */ -1, materializationExecPath); | |
| } | |
| private RemoteFileArtifactValue createRemoteFileMetadata( | |
| String content, long expireAtEpochMilli, @Nullable PathFragment materializationExecPath) { | |
| byte[] bytes = content.getBytes(UTF_8); | |
| return RemoteFileArtifactValue.create( | |
| digest(bytes), bytes.length, 1, expireAtEpochMilli, materializationExecPath); | |
| } | |
| private static TreeArtifactValue createTreeMetadata( | |
| SpecialArtifact parent, | |
| ImmutableMap<String, ? extends FileArtifactValue> children, | |
| Optional<FileArtifactValue> archivedArtifactValue, | |
| Optional<PathFragment> materializationExecPath) { | |
| TreeArtifactValue.Builder builder = TreeArtifactValue.newBuilder(parent); | |
| for (Map.Entry<String, ? extends FileArtifactValue> entry : children.entrySet()) { | |
| builder.putChild( | |
| Artifact.TreeFileArtifact.createTreeOutput(parent, entry.getKey()), entry.getValue()); | |
| } | |
| archivedArtifactValue.ifPresent( | |
| metadata -> { | |
| ArchivedTreeArtifact artifact = ArchivedTreeArtifact.createForTree(parent); | |
| builder.setArchivedRepresentation( | |
| TreeArtifactValue.ArchivedRepresentation.create(artifact, metadata)); | |
| }); | |
| materializationExecPath.ifPresent(builder::setMaterializationExecPath); | |
| return builder.build(); | |
| } | |
| @Test | |
| public void saveOutputMetadata_remoteFileMetadataSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| Action action = new InjectOutputFileMetadataAction(output, createRemoteFileMetadata(content)); | |
| // Not cached. | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isEqualTo(createRemoteFileMetadata(content)); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_localFileMetadataNotSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| Action action = new WriteEmptyOutputAction(output); | |
| output.getPath().delete(); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isTrue(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isNull(); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_remoteMetadataInjectedAndLocalFilesStored() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| Action action = | |
| new WriteEmptyOutputAction(output) { | |
| @Override | |
| public ActionResult execute(ActionExecutionContext actionExecutionContext) { | |
| actionExecutionContext | |
| .getOutputMetadataStore() | |
| .injectFile(output, createRemoteFileMetadata("")); | |
| return super.execute(actionExecutionContext); | |
| } | |
| }; | |
| output.getPath().delete(); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isTrue(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isEqualTo(createRemoteFileMetadata("")); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_notSavedIfDisabled() throws Exception { | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| Action action = new InjectOutputFileMetadataAction(output, createRemoteFileMetadata(content)); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isNull(); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_remoteFileMetadataLoaded() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| Action action = new InjectOutputFileMetadataAction(output, createRemoteFileMetadata(content)); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| RemoteArtifactChecker.TRUST_ALL); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| assertThat(token).isNull(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isEqualTo(createRemoteFileMetadata(content)); | |
| assertThat(metadataHandler.getOutputMetadata(output)) | |
| .isEqualTo(createRemoteFileMetadata(content)); | |
| } | |
| @Test | |
| public void saveOutputMetadata_remoteFileExpired_remoteFileMetadataNotLoaded() throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| Action action = | |
| new InjectOutputFileMetadataAction( | |
| output, | |
| createRemoteFileMetadata( | |
| content, /* expireAtEpochMilli= */ 0, /* materializationExecPath= */ null)); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| CHECK_TTL); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| assertThat(token).isNotNull(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNull(); | |
| } | |
| @Test | |
| public void saveOutputMetadata_storeOutputMetadataDisabled_remoteFileMetadataNotLoaded() | |
| throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ false); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| Action action = new InjectOutputFileMetadataAction(output, createRemoteFileMetadata(content)); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| /* remoteArtifactChecker= */ null); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| assertThat(token).isNotNull(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNull(); | |
| } | |
| @Test | |
| public void saveOutputMetadata_localMetadataIsSameAsRemoteMetadata_cached( | |
| @TestParameter({"", "/target/path"}) String materializationExecPathParam) throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| PathFragment materializationExecPath = | |
| materializationExecPathParam.isEmpty() ? null : PathFragment.create("/target/path"); | |
| Action action = | |
| new InjectOutputFileMetadataAction( | |
| output, createRemoteFileMetadata(content, materializationExecPath)); | |
| runAction(action); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| writeContentAsLatin1(output.getPath(), content); | |
| // Cached since local metadata is same as remote metadata | |
| runAction(action); | |
| assertStatistics(1, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)) | |
| .isEqualTo(createRemoteFileMetadata(content, materializationExecPath)); | |
| } | |
| @Test | |
| public void saveOutputMetadata_localMetadataIsDifferentFromRemoteMetadata_notCached() | |
| throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content1 = "content1"; | |
| String content2 = "content2"; | |
| Action action = | |
| new InjectOutputFileMetadataAction( | |
| output, createRemoteFileMetadata(content1), createRemoteFileMetadata(content2)); | |
| runAction(action); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| writeContentAsLatin1(output.getPath(), content2); | |
| // Assert that if local file exists, shouldTrustRemoteArtifact is not called for the remote | |
| // metadata. | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| var mockedRemoteArtifactChecker = mock(RemoteArtifactChecker.class); | |
| var token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| mockedRemoteArtifactChecker); | |
| verify(mockedRemoteArtifactChecker, never()).shouldTrustRemoteArtifact(any(), any()); | |
| // Not cached since local file changed | |
| runAction( | |
| action, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| /* platform= */ ImmutableMap.of(), | |
| metadataHandler, | |
| metadataHandler, | |
| token); | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder() | |
| .set(MissReason.NOT_CACHED, 1) | |
| .set(MissReason.DIFFERENT_FILES, 1) | |
| .build()); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isEqualTo(createRemoteFileMetadata(content2)); | |
| } | |
| @Test | |
| public void saveOutputMetadata_trustedRemoteMetadataFromOutputStore_cached() throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| RemoteFileArtifactValue metadata = createRemoteFileMetadata(content); | |
| Action action = new InjectOutputFileMetadataAction(output, metadata, metadata); | |
| runAction(action); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| FakeInputMetadataHandler fakeOutputMetadataStore = new FakeInputMetadataHandler(); | |
| fakeOutputMetadataStore.injectFile(output, metadata); | |
| runAction( | |
| action, | |
| ImmutableMap.of(), | |
| ImmutableMap.of(), | |
| new FakeInputMetadataHandler(), | |
| fakeOutputMetadataStore); | |
| assertStatistics(1, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isEqualTo(metadata); | |
| } | |
| @Test | |
| public void saveOutputMetadata_untrustedRemoteMetadataFromOutputStore_notCached() | |
| throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ true); | |
| Artifact output = createArtifact(artifactRoot, "bin/dummy"); | |
| String content = "content"; | |
| RemoteFileArtifactValue metadata = createRemoteFileMetadata(content); | |
| Action action = new InjectOutputFileMetadataAction(output, metadata, metadata); | |
| runAction(action); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| FakeInputMetadataHandler fakeOutputMetadataStore = new FakeInputMetadataHandler(); | |
| fakeOutputMetadataStore.injectFile(output, metadata); | |
| RemoteArtifactChecker remoteArtifactChecker = mock(RemoteArtifactChecker.class); | |
| when(remoteArtifactChecker.shouldTrustRemoteArtifact(any(), any())).thenReturn(false); | |
| runAction( | |
| action, | |
| ImmutableMap.of(), | |
| ImmutableMap.of(), | |
| new FakeInputMetadataHandler(), | |
| fakeOutputMetadataStore, | |
| remoteArtifactChecker); | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder() | |
| .set(MissReason.NOT_CACHED, 1) | |
| .set(MissReason.DIFFERENT_FILES, 1) | |
| .build()); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputFile(output)).isEqualTo(metadata); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_remoteFileMetadataSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, RemoteFileArtifactValue> children = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", createRemoteFileMetadata("content2")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| children, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| children, | |
| /* archivedFileValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_remoteArchivedArtifactSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| Optional.of(createRemoteFileMetadata("content")), | |
| /* materializationExecPath= */ Optional.empty())); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| ImmutableMap.of(), | |
| Optional.of(createRemoteFileMetadata("content")), | |
| /* materializationExecPath= */ Optional.empty())); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_materializationExecPathSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| Optional.of(PathFragment.create("/target/path")))); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| ImmutableMap.of(), | |
| /* archivedFileValue= */ Optional.empty(), | |
| Optional.of(PathFragment.create("/target/path")))); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_emptyTreeMetadata_notSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| RemoteArtifactChecker.TRUST_ALL); | |
| assertThat(token).isNull(); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)).isNull(); | |
| assertStatistics(1, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_localFileMetadataNotSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| writeIsoLatin1(fileSystem.getPath("/file2"), ""); | |
| ImmutableMap<String, FileArtifactValue> children = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", FileArtifactValue.createForTesting(fileSystem.getPath("/file2"))); | |
| fileSystem.getPath("/file2").delete(); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| children, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| ImmutableMap.of("file1", createRemoteFileMetadata("content1")), | |
| /* archivedFileValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_localArchivedArtifactNotSaved() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| writeIsoLatin1(fileSystem.getPath("/archive"), ""); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| Optional.of(FileArtifactValue.createForTesting(fileSystem.getPath("/archive"))), | |
| /* materializationExecPath= */ Optional.empty())); | |
| fileSystem.getPath("/archive").delete(); | |
| runAction(action); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)).isNull(); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_remoteFileMetadataLoaded() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, FileArtifactValue> children = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", createRemoteFileMetadata("content2")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| children, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| RemoteArtifactChecker.TRUST_ALL); | |
| TreeArtifactValue expectedMetadata = | |
| createTreeMetadata( | |
| output, | |
| children, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty()); | |
| assertThat(token).isNull(); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo(SerializableTreeArtifactValue.createSerializable(expectedMetadata).get()); | |
| assertThat(metadataHandler.getTreeArtifactValue(output)).isEqualTo(expectedMetadata); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_localFileMetadataLoaded() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, FileArtifactValue> children1 = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", createRemoteFileMetadata("content2")); | |
| ImmutableMap<String, FileArtifactValue> children2 = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", createRemoteFileMetadata("modified_remote")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| children1, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty()), | |
| createTreeMetadata( | |
| output, | |
| children2, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| writeIsoLatin1(output.getPath().getRelative("file2"), "modified_local"); | |
| var mockedRemoteArtifactChecker = mock(RemoteArtifactChecker.class); | |
| doReturn(true).when(mockedRemoteArtifactChecker).shouldTrustRemoteArtifact(any(), any()); | |
| var token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| mockedRemoteArtifactChecker); | |
| verify(mockedRemoteArtifactChecker) | |
| .shouldTrustRemoteArtifact( | |
| argThat(arg -> arg.getExecPathString().endsWith("file1")), any()); | |
| verify(mockedRemoteArtifactChecker, never()) | |
| .shouldTrustRemoteArtifact( | |
| argThat(arg -> arg.getExecPathString().endsWith("file2")), any()); | |
| // Not cached since local file changed | |
| runAction( | |
| action, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| /* platform= */ ImmutableMap.of(), | |
| metadataHandler, | |
| metadataHandler, | |
| token); | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder() | |
| .set(MissReason.NOT_CACHED, 1) | |
| .set(MissReason.DIFFERENT_FILES, 1) | |
| .build()); | |
| assertThat(output.getPath().exists()).isTrue(); | |
| TreeArtifactValue expectedMetadata = | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", createRemoteFileMetadata("modified_remote")), | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty()); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo(SerializableTreeArtifactValue.createSerializable(expectedMetadata).get()); | |
| assertThat(metadataHandler.getTreeArtifactValue(output)).isEqualTo(expectedMetadata); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadata_localArchivedArtifactLoaded() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| Optional.of(createRemoteFileMetadata("content")), | |
| /* materializationExecPath= */ Optional.empty()), | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| Optional.of(createRemoteFileMetadata("modified")), | |
| /* materializationExecPath= */ Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| writeIsoLatin1(ArchivedTreeArtifact.createForTree(output).getPath(), "modified"); | |
| var mockedRemoteArtifactChecker = mock(RemoteArtifactChecker.class); | |
| var token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| mockedRemoteArtifactChecker); | |
| verify(mockedRemoteArtifactChecker, never()).shouldTrustRemoteArtifact(any(), any()); | |
| // Not cached since local file changed | |
| runAction( | |
| action, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| /* platform= */ ImmutableMap.of(), | |
| metadataHandler, | |
| metadataHandler, | |
| token); | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder() | |
| .set(MissReason.NOT_CACHED, 1) | |
| .set(MissReason.DIFFERENT_FILES, 1) | |
| .build()); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| TreeArtifactValue expectedMetadata = | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| Optional.of(createRemoteFileMetadata("modified")), | |
| /* materializationExecPath= */ Optional.empty()); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo(SerializableTreeArtifactValue.createSerializable(expectedMetadata).get()); | |
| assertThat(metadataHandler.getTreeArtifactValue(output)).isEqualTo(expectedMetadata); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeFileExpired_treeMetadataNotLoaded() throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, RemoteFileArtifactValue> children = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", | |
| createRemoteFileMetadata( | |
| "content2", /* expireAtEpochMilli= */ 0, /* materializationExecPath= */ null)); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| children, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| CHECK_TTL); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| assertThat(token).isNotNull(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNull(); | |
| } | |
| @Test | |
| public void saveOutputMetadata_archivedRepresentationExpired_treeMetadataNotLoaded() | |
| throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, RemoteFileArtifactValue> children = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", createRemoteFileMetadata("content2")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| children, | |
| /* archivedArtifactValue= */ Optional.of( | |
| createRemoteFileMetadata( | |
| "archived", | |
| /* expireAtEpochMilli= */ 0, | |
| /* materializationExecPath= */ null)), | |
| /* materializationExecPath= */ Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| CHECK_TTL); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| assertThat(token).isNotNull(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNull(); | |
| } | |
| private static void writeContentAsLatin1(Path path, String content) throws IOException { | |
| Path parent = path.getParentDirectory(); | |
| if (parent != null) { | |
| parent.createDirectoryAndParents(); | |
| } | |
| FileSystemUtils.writeContentAsLatin1(path, content); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadataWithSameLocalFileMetadata_cached() throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, RemoteFileArtifactValue> children = | |
| ImmutableMap.of( | |
| "file1", createRemoteFileMetadata("content1"), | |
| "file2", createRemoteFileMetadata("content2")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, children, /* archivedArtifactValue= */ Optional.empty(), Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| writeContentAsLatin1(output.getPath().getRelative("file1"), "content1"); | |
| // Cache hit | |
| Token token = | |
| cacheChecker.getTokenIfNeedToExecute( | |
| action, | |
| /* resolvedCacheArtifacts= */ null, | |
| /* clientEnv= */ ImmutableMap.of(), | |
| OutputPermissions.READONLY, | |
| /* handler= */ null, | |
| metadataHandler, | |
| metadataHandler, | |
| /* artifactExpander= */ null, | |
| /* remoteDefaultPlatformProperties= */ ImmutableMap.of(), | |
| RemoteArtifactChecker.TRUST_ALL); | |
| assertThat(token).isNull(); | |
| assertStatistics(1, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| assertThat(output.getPath().exists()).isTrue(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| children, /* archivedFileValue= */ Optional.empty(), Optional.empty())); | |
| assertThat(metadataHandler.getTreeArtifactValue(output)) | |
| .isEqualTo( | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of( | |
| "file1", | |
| FileArtifactValue.createForTesting(output.getPath().getRelative("file1")), | |
| "file2", | |
| createRemoteFileMetadata("content2")), | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| } | |
| @Test | |
| public void saveOutputMetadata_treeMetadataWithSameLocalArchivedArtifact_cached() | |
| throws Exception { | |
| cacheChecker = createActionCacheChecker(/*storeOutputMetadata=*/ true); | |
| SpecialArtifact output = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| Action action = | |
| new InjectOutputTreeMetadataAction( | |
| output, | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| Optional.of(createRemoteFileMetadata("content")), | |
| /* materializationExecPath= */ Optional.empty())); | |
| FakeInputMetadataHandler metadataHandler = new FakeInputMetadataHandler(); | |
| runAction(action); | |
| writeContentAsLatin1(ArchivedTreeArtifact.createForTree(output).getPath(), "content"); | |
| // Cache hit | |
| runAction(action, metadataHandler, metadataHandler); | |
| assertStatistics(1, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| assertThat(output.getPath().exists()).isFalse(); | |
| ActionCache.Entry entry = cache.get(output.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(output)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| ImmutableMap.of(), | |
| /* archivedFileValue= */ Optional.of(createRemoteFileMetadata("content")), | |
| /* materializationExecPath= */ Optional.empty())); | |
| assertThat(metadataHandler.getTreeArtifactValue(output)) | |
| .isEqualTo( | |
| createTreeMetadata( | |
| output, | |
| ImmutableMap.of(), | |
| Optional.of( | |
| FileArtifactValue.createForTesting(ArchivedTreeArtifact.createForTree(output))), | |
| /* materializationExecPath= */ Optional.empty())); | |
| } | |
| @Test | |
| public void saveOutputMetadata_trustedRemoteTreeMetadataFromOutputStore_cached() | |
| throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ true); | |
| SpecialArtifact tree = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, RemoteFileArtifactValue> children = | |
| ImmutableMap.of("file", createRemoteFileMetadata("content")); | |
| TreeArtifactValue treeMetadata = | |
| createTreeMetadata( | |
| tree, | |
| children, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty()); | |
| Action action = new InjectOutputTreeMetadataAction(tree, treeMetadata, treeMetadata); | |
| runAction(action); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| FakeInputMetadataHandler fakeOutputMetadataStore = new FakeInputMetadataHandler(); | |
| fakeOutputMetadataStore.injectTree(tree, treeMetadata); | |
| runAction( | |
| action, | |
| ImmutableMap.of(), | |
| ImmutableMap.of(), | |
| new FakeInputMetadataHandler(), | |
| fakeOutputMetadataStore); | |
| assertStatistics(1, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| ActionCache.Entry entry = cache.get(tree.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(tree)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| children, | |
| /* archivedFileValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| } | |
| @Test | |
| public void saveOutputMetadata_untrustedRemoteTreeMetadataFromOutputStore_notCached() | |
| throws Exception { | |
| cacheChecker = createActionCacheChecker(/* storeOutputMetadata= */ true); | |
| SpecialArtifact tree = | |
| createTreeArtifactWithGeneratingAction(artifactRoot, PathFragment.create("bin/dummy")); | |
| ImmutableMap<String, RemoteFileArtifactValue> children = | |
| ImmutableMap.of("file", createRemoteFileMetadata("content")); | |
| TreeArtifactValue treeMetadata = | |
| createTreeMetadata( | |
| tree, | |
| children, | |
| /* archivedArtifactValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty()); | |
| Action action = new InjectOutputTreeMetadataAction(tree, treeMetadata, treeMetadata); | |
| runAction(action); | |
| assertStatistics(0, new MissDetailsBuilder().set(MissReason.NOT_CACHED, 1).build()); | |
| FakeInputMetadataHandler fakeOutputMetadataStore = new FakeInputMetadataHandler(); | |
| fakeOutputMetadataStore.injectTree(tree, treeMetadata); | |
| RemoteArtifactChecker remoteArtifactChecker = mock(RemoteArtifactChecker.class); | |
| when(remoteArtifactChecker.shouldTrustRemoteArtifact(any(), any())).thenReturn(false); | |
| runAction( | |
| action, | |
| ImmutableMap.of(), | |
| ImmutableMap.of(), | |
| new FakeInputMetadataHandler(), | |
| fakeOutputMetadataStore, | |
| remoteArtifactChecker); | |
| assertStatistics( | |
| 0, | |
| new MissDetailsBuilder() | |
| .set(MissReason.NOT_CACHED, 1) | |
| .set(MissReason.DIFFERENT_FILES, 1) | |
| .build()); | |
| ActionCache.Entry entry = cache.get(tree.getExecPathString()); | |
| assertThat(entry).isNotNull(); | |
| assertThat(entry.getOutputTree(tree)) | |
| .isEqualTo( | |
| SerializableTreeArtifactValue.create( | |
| children, | |
| /* archivedFileValue= */ Optional.empty(), | |
| /* materializationExecPath= */ Optional.empty())); | |
| } | |
| // TODO(tjgq): Add tests for cached tree artifacts with a materialization path. They should take | |
| // into account every combination of entirely/partially remote metadata and symlink present/not | |
| // present in the filesystem. | |
| /** An {@link ActionCache} that allows injecting corruption for testing. */ | |
| private static final class CorruptibleActionCache implements ActionCache { | |
| private final CompactPersistentActionCache delegate; | |
| private boolean corrupted = false; | |
| CorruptibleActionCache(Path cacheRoot, Clock clock) throws IOException { | |
| this.delegate = | |
| CompactPersistentActionCache.create(cacheRoot, clock, NullEventHandler.INSTANCE); | |
| } | |
| void corruptAllEntries() { | |
| corrupted = true; | |
| } | |
| @Override | |
| public Entry get(String key) { | |
| return corrupted ? ActionCache.Entry.CORRUPTED : delegate.get(key); | |
| } | |
| @Override | |
| public void put(String key, Entry entry) { | |
| delegate.put(key, entry); | |
| } | |
| @Override | |
| public void remove(String key) { | |
| delegate.remove(key); | |
| } | |
| @Override | |
| public void removeIf(Predicate<Entry> predicate) { | |
| delegate.removeIf(predicate); | |
| } | |
| @Override | |
| public long save() throws IOException { | |
| return delegate.save(); | |
| } | |
| @Override | |
| public void clear() { | |
| delegate.clear(); | |
| } | |
| @Override | |
| public void dump(PrintStream out) { | |
| delegate.dump(out); | |
| } | |
| @Override | |
| public int size() { | |
| return delegate.size(); | |
| } | |
| @Override | |
| public void accountHit() { | |
| delegate.accountHit(); | |
| } | |
| @Override | |
| public void accountMiss(MissReason reason) { | |
| delegate.accountMiss(reason); | |
| } | |
| @Override | |
| public void mergeIntoActionCacheStatistics(ActionCacheStatistics.Builder builder) { | |
| delegate.mergeIntoActionCacheStatistics(builder); | |
| } | |
| @Override | |
| public void resetStatistics() { | |
| delegate.resetStatistics(); | |
| } | |
| } | |
| /** A null middleman action. */ | |
| private static class NullMiddlemanAction extends NullAction { | |
| @Override | |
| public MiddlemanType getActionType() { | |
| return MiddlemanType.RUNFILES_MIDDLEMAN; | |
| } | |
| } | |
| /** A fake metadata handler that is able to obtain metadata from the file system. */ | |
| private static final class FakeInputMetadataHandler extends FakeInputMetadataHandlerBase { | |
| private final Map<Artifact, FileArtifactValue> fileMetadata = new HashMap<>(); | |
| private final Map<SpecialArtifact, TreeArtifactValue> treeMetadata = new HashMap<>(); | |
| @Override | |
| public void injectFile(Artifact output, FileArtifactValue metadata) { | |
| fileMetadata.put(output, metadata); | |
| } | |
| @Override | |
| public void injectTree(SpecialArtifact treeArtifact, TreeArtifactValue tree) { | |
| treeMetadata.put(treeArtifact, tree); | |
| } | |
| @Override | |
| public FileArtifactValue getInputMetadata(ActionInput input) throws IOException { | |
| if (!(input instanceof Artifact)) { | |
| return null; | |
| } | |
| return FileArtifactValue.createForTesting((Artifact) input); | |
| } | |
| @Override | |
| public FileArtifactValue getOutputMetadata(ActionInput input) | |
| throws IOException, InterruptedException { | |
| if (!(input instanceof Artifact output)) { | |
| return null; | |
| } | |
| if (output.isTreeArtifact()) { | |
| TreeArtifactValue treeArtifactValue = getTreeArtifactValue((SpecialArtifact) output); | |
| if (treeArtifactValue != null) { | |
| return treeArtifactValue.getMetadata(); | |
| } else { | |
| return null; | |
| } | |
| } | |
| if (fileMetadata.containsKey(output)) { | |
| return fileMetadata.get(output); | |
| } | |
| return FileArtifactValue.createForTesting(output); | |
| } | |
| @Override | |
| public TreeArtifactValue getTreeArtifactValue(SpecialArtifact output) | |
| throws IOException, InterruptedException { | |
| if (treeMetadata.containsKey(output)) { | |
| return treeMetadata.get(output); | |
| } | |
| TreeArtifactValue.Builder tree = TreeArtifactValue.newBuilder(output); | |
| Path treeDir = output.getPath(); | |
| if (treeDir.exists()) { | |
| TreeArtifactValue.visitTree( | |
| treeDir, | |
| (parentRelativePath, type, traversedSymlink) -> { | |
| if (type == Dirent.Type.DIRECTORY) { | |
| return; | |
| } | |
| Artifact.TreeFileArtifact child = | |
| Artifact.TreeFileArtifact.createTreeOutput(output, parentRelativePath); | |
| FileArtifactValue metadata = | |
| FileArtifactValue.createForTesting(treeDir.getRelative(parentRelativePath)); | |
| synchronized (tree) { | |
| tree.putChild(child, metadata); | |
| } | |
| }); | |
| } | |
| ArchivedTreeArtifact archivedTreeArtifact = ArchivedTreeArtifact.createForTree(output); | |
| if (archivedTreeArtifact.getPath().exists()) { | |
| tree.setArchivedRepresentation( | |
| archivedTreeArtifact, | |
| FileArtifactValue.createForTesting(archivedTreeArtifact.getPath())); | |
| } | |
| return tree.build(); | |
| } | |
| @Override | |
| public void setDigestForVirtualArtifact(Artifact artifact, byte[] digest) {} | |
| } | |
| private static class WriteEmptyOutputAction extends NullAction { | |
| WriteEmptyOutputAction() {} | |
| WriteEmptyOutputAction(Artifact... outputs) { | |
| super(outputs); | |
| } | |
| @Override | |
| public ActionResult execute(ActionExecutionContext actionExecutionContext) { | |
| for (Artifact output : getOutputs()) { | |
| Path path = output.getPath(); | |
| if (!path.exists()) { | |
| try { | |
| writeContentAsLatin1(path, ""); | |
| } catch (IOException e) { | |
| throw new IllegalStateException("Failed to create output", e); | |
| } | |
| } | |
| } | |
| return super.execute(actionExecutionContext); | |
| } | |
| } | |
| private static class InjectOutputFileMetadataAction extends NullAction { | |
| private final Artifact output; | |
| private final Deque<FileArtifactValue> metadataDeque; | |
| InjectOutputFileMetadataAction(Artifact output, FileArtifactValue... metadata) { | |
| super(output); | |
| this.output = output; | |
| this.metadataDeque = new ArrayDeque<>(ImmutableList.copyOf(metadata)); | |
| } | |
| @Override | |
| public ActionResult execute(ActionExecutionContext actionExecutionContext) { | |
| actionExecutionContext.getOutputMetadataStore().injectFile(output, metadataDeque.pop()); | |
| return super.execute(actionExecutionContext); | |
| } | |
| } | |
| private static final class InjectOutputTreeMetadataAction extends NullAction { | |
| private final SpecialArtifact output; | |
| private final Deque<TreeArtifactValue> metadataDeque; | |
| InjectOutputTreeMetadataAction(SpecialArtifact output, TreeArtifactValue... metadata) { | |
| super(output); | |
| this.output = output; | |
| this.metadataDeque = new ArrayDeque<>(ImmutableList.copyOf(metadata)); | |
| } | |
| @Override | |
| public ActionResult execute(ActionExecutionContext actionExecutionContext) { | |
| actionExecutionContext.getOutputMetadataStore().injectTree(output, metadataDeque.pop()); | |
| return super.execute(actionExecutionContext); | |
| } | |
| } | |
| } |