Age | Commit message (Collapse) | Author | |
---|---|---|---|
2025-06-04 | Remove unneeded extra checks for upwards symlinks | Paul Cristian Sarbu | |
2025-06-04 | Executor: Check validity of action outputs in compatible mode | Paul Cristian Sarbu | |
This ensures that any entries that the standard remote execution protocol accepts but are invalid in justbuild, i.e., upwards symlinks, are rejected. For this purpose, do not fail in the action response instances, just perform the check there, as all required information is available, and set a flag that the executor can check as needed. | |||
2025-06-04 | {Local,Bazel}Response: Fix logic for marking succesful population | Paul Cristian Sarbu | |
While in practice a failure to populate the fields of a response happens once per invocation, as it will trigger a failure of the execution, from an algorithmic standpoint the flag to mark a successful population of the response fields should only be set on actual success. Fix this. | |||
2025-06-04 | LocalCasReader: Add validity check for bazel directories | Paul Cristian Sarbu | |
This will check if directories contain upwards symlinks. | |||
2025-06-04 | Execution server: Let the underlying API handle invalid entries | Paul Cristian Sarbu | |
The execution server itself should not consider anything special in setting the response message to the client, instead let the underlying API fail or not during collection. | |||
2025-06-04 | RepositoryConfig: Ensure consistency in reading blobs and trees | Paul Cristian Sarbu | |
...with respect to rejecting invalid entries such as upwards symlinks. Also ensure that valid trees are only checked once by remebering known valid tress though marker files in local storage. | |||
2025-06-02 | ParallelRetrieveToCasWithCache: abort early on failure | Klaus Aehlig | |
... and do mark artifacts internally as synchronized. First all all, we will abort anyway, to the entry won't even be read and, secondly it is not necessarily true that the artifact is synchronized. | |||
2025-05-12 | LocalAction: Fix collection of directory symlink | Oliver Reiche | |
2025-04-22 | FileSystemManager: Always remove directories recursively | Maksim Denisov | |
2025-04-17 | bazel_msg_factory: avoid copies in lambdas... | Alberto Sartori | |
and while there, replace `auto` with explicit signatures. | |||
2025-04-17 | bazel_execution_client: eliminate one un-needed copy | Alberto Sartori | |
2025-04-11 | Remove unused function GetTree from BazelCasClient | Klaus Aehlig | |
2025-04-11 | Drop unused directory map | Klaus Aehlig | |
The BazelNetworkReader contains an optimization for reading directories in case the remote execution (in compatible mode) supports the GetTree request. This is, however not the case for many remote exeuciton services, including our own single-node execution service. So the code is basically untested and rarely used, if at all. Moreover, justbuild is usually used in native mode and using compatibility mode is expected to handle tree operations less efficient. Therefore, remove this basically dead code and decrease complexity this way. | |||
2025-04-10 | cas_utils: Fix typo in log message | Paul Cristian Sarbu | |
2025-04-10 | BazelAcClient: log cache misses at trace level | Klaus Aehlig | |
... instead of at debug. We expect actions to be not in cache, so the fact that we experience cache misses is not surprising. Given the information available at this point, a useful logging indicating (in terms meaningful to the user) is not possible. Therefore, keep the debug-level log clean. | |||
2025-04-10 | Remove unnecessary targets files | Klaus Aehlig | |
2025-04-08 | BazelCasClient: only warn on failure of GetTree | Klaus Aehlig | |
... as that failure does not necessarily abort the build. | |||
2025-03-24 | BazelNetworkReader: Remove IncrementalReader | Maksim Denisov | |
2025-03-24 | BazelResponse: Use non-incremental reading | Maksim Denisov | |
to avoid downloading the same blobs. | |||
2025-03-24 | BazelApi: Use non-incremental reading | Maksim Denisov | |
...to avoid downloading the same blobs. | |||
2025-03-24 | BazelNetworkReader: Implement non-incremental reading. | Maksim Denisov | |
2025-03-24 | BazelCasClient: BatchRead to temporary files. | Maksim Denisov | |
2025-03-24 | BytestreamClient: Read to temporary files. | Maksim Denisov | |
2025-03-24 | BazelApi: Use file sources of ArtifactBlobs. | Maksim Denisov | |
2025-03-24 | LocalApi: Use file sources of ArtifactBlobs. | Maksim Denisov | |
2025-03-24 | ExecutionApi: Return TmpDir | Maksim Denisov | |
2025-03-24 | Pass TmpDir to BazelNetwork and BazelCasClient | Maksim Denisov | |
2025-03-24 | TmpDir: minor refactoring | Maksim Denisov | |
2025-03-24 | Remove redundant UpdateContainerAndUpload. | Maksim Denisov | |
2025-03-17 | execution_api: support reading off stdout/stderr digests | Klaus Aehlig | |
2025-03-11 | just execute: write pid and info files atomically | Klaus Aehlig | |
2025-03-10 | just execute: set basic timing meta data | Klaus Aehlig | |
While there, increase the time precision to nanoseconds, which is the resolution of the underlying timestamp proto. | |||
2025-03-10 | remote actions: include duration | Klaus Aehlig | |
2025-03-10 | local actions: return duration | Klaus Aehlig | |
2025-03-10 | execution response interface: include execution duration | Klaus Aehlig | |
2025-03-10 | ids: add generic interface function for creating a UUID | Klaus Aehlig | |
2025-03-05 | Move obtain capabilities failures to debug log | Oliver Reiche | |
... instead of reporting an error, although not being fatal. | |||
2025-02-28 | Avoid rehashing content when using GitRepo::ReadTreeData | Maksim Denisov | |
2025-02-28 | Include ArtifactDigestFactory into "common" library | Maksim Denisov | |
2025-02-27 | LocalApi: Create ArtifactBlobs from existing files | Maksim Denisov | |
2025-02-27 | BazelNetworkReader: Remove additional validation | Maksim Denisov | |
...since ArtifactBlob rehashes data on construction and guarantees that the digest matches the content. | |||
2025-02-27 | ArtifactBlob: Use static function for construction | Maksim Denisov | |
2025-02-27 | ArtifactBlob: Check access to the content | Maksim Denisov | |
2025-02-27 | ArtifactBlob: Support incremental reading | Maksim Denisov | |
2025-02-27 | ArtifactBlob: Convert to a class | Maksim Denisov | |
2025-02-27 | ArtifactBlob: Move from execution_api/common to common | Maksim Denisov | |
2025-02-27 | GitApi: remove intermediate ArtifactBlob container | Maksim Denisov | |
2025-02-27 | BazelCasClient: Support request creation failure in InitRequest. | Maksim Denisov | |
2025-02-21 | ByteStreamClient: Return ArtifactBlob from Read. | Maksim Denisov | |
2025-02-21 | ByteStreamUtils: Simplify ReadRequest and WriteRequest | Maksim Denisov | |