Age | Commit message (Collapse) | Author | |
---|---|---|---|
6 days | Allow colons in remote-execution properties | Bin Yu | |
When user setting `remote-execution-properties` with commandline, we assume that input value must have format with `key:val`, in that case, if `val` includes colon(s), the real value after parsing won't be the same as user intends to have, for example, considering commandline flag: `--remote-execution-properties image:test.registry:8080/test`, will be parsed into a pair `<image, test.registry>` in existing implementation. Since it's reasonable to allow colon(s) in execution property value, we should allow this case, and still keep the restriction on key format. Signed-off-by: Bin Yu <yubin14@huawei.com> | |||
2025-06-24 | Clarify use of a clang-tidy check | Paul Cristian Sarbu | |
The google-default-arguments check normally imposes that virtual methods have no default arguments. For our use-cases, all implementations of such methods are expected to use the same default arguments, and thus this check is manually disabled via NOLINT comments. However, this is not done consistently. This commit cleans this up and clarifies our intent by: - removing the default values (and the NOLINT statement) for all implementations of virtual methods with default argument values, matching the desired intended behaviour, but - keeping the clang-tidy check for future cases where derived classes would want to provide each different defaults. | |||
2025-06-24 | Test: Add test for execution server APIs | Oliver Reiche | |
2025-06-24 | Test: Add test for API output path modes | Oliver Reiche | |
2025-06-24 | ExecutionAPI: Support output_paths in requests | Oliver Reiche | |
... and prepare local execution for clients using only RBEv2.1 (setting only output_paths). | |||
2025-06-24 | ExecutionApi: Drop DirectorySymlinks from common api | Oliver Reiche | |
... as it is only needed for local execution, there is no need to provide it in the common api interface. | |||
2025-06-16 | Avoid unnecessary work in accessing container entries | Paul Cristian Sarbu | |
- in sequence containers, use operator[] instead of .at() when accessing indices guaranteed to be in bound; - in associative containers, prefer .find() and reusing the returned const iterator to using .contains() and .at(); while there, make any so obtained iterators const if they are read-only. | |||
2025-06-13 | Test: Fix flaky LargeObjectCAS test. | Maksim Denisov | |
Due to a random nature of the LargeObjectUtils generator, it may generate 2 identical files in a row. To prevent the test from failing, check that a newly generated file doesn't collide with any already added to the CAS. | |||
2025-06-12 | GarbageCollector: Support removal of all generations at once | Maksim Denisov | |
...that ignores compactification. | |||
2025-06-04 | directory_map: Correctly report fatal on failures to read from workspace roots | Paul Cristian Sarbu | |
2025-06-04 | Tests: Verify symlink handling in APIs | Oliver Reiche | |
... which should not do any symlink checks in compatible mode. | |||
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 | FileRoot: Give git-based roots access to storage config | Paul Cristian Sarbu | |
2025-06-04 | GitTree: Ensure all read entries are valid | Paul Cristian Sarbu | |
Match behaviour of reading trees, which always checks for invalid entries, also for reading blobs. | |||
2025-05-12 | Tests: Assume only parents of out_dirs exist | Oliver Reiche | |
... instead of the full out_dir path, which is not guaranteed by the RBE protocol. | |||
2025-05-12 | LocalAction: Fix collection of directory symlink | Oliver Reiche | |
2025-05-05 | TreeOperationsUtils: add tree-overlay deduplication test | Sascha Roloff | |
This test ensures that the AsyncMap implementation of the tree-overlay computation works as expected and properly prevents duplicated work when it comes to the repeated computation of the same trees. | |||
2025-04-22 | FileSystemManager: When copying, take into account equivalent files. | Maksim Denisov | |
2025-04-22 | FileSystemManager: Always copy directories recursively | Maksim Denisov | |
2025-04-22 | FileSystemManager: Always remove directories recursively | Maksim Denisov | |
2025-04-07 | Add built-in rules {disjoint_,}tree_overlay | Klaus Aehlig | |
2025-04-07 | Defined rules: add TREE_OVERLAY and DISJOINT_TREE_OVERLAY function | Klaus Aehlig | |
2025-04-07 | analysed_target: include tree overlays | Klaus Aehlig | |
2025-04-07 | Action: support tree-overlay actions | Klaus Aehlig | |
2025-03-24 | Test 'zip_map' expression | Paul Cristian Sarbu | |
2025-03-24 | Test 'zip_with' expression | Paul Cristian Sarbu | |
2025-03-24 | BazelNetworkReader: Remove IncrementalReader | Maksim Denisov | |
2025-03-24 | BytestreamClient: Read to temporary files. | 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-17 | execution_api: support reading off stdout/stderr digests | Klaus Aehlig | |
2025-03-10 | execution response interface: include execution duration | Klaus Aehlig | |
2025-03-10 | Profiling: start collecting informations per actions | Klaus Aehlig | |
... so far, which actions where considered, and which of those were cached. | |||
2025-03-10 | Make graph-dumping options cummulative | Klaus Aehlig | |
If --dump-graph or --dump-plain-graph is given several times, the action graph wil also be written several times. In this way, regular use of those options will not be affected by adding them implicitly through invocation-logging options in the rc file. | |||
2025-02-28 | Make statistics a separate library | Maksim Denisov | |
2025-02-28 | Include ArtifactDigestFactory into "common" library | Maksim Denisov | |
2025-02-27 | BazelNetwork: Adapt the test that reads objects of unknown size | Maksim Denisov | |
2025-02-27 | Remove BytestreamClient test. | Maksim Denisov | |
ArtifactBlob rehashes content, there's no way to construct it with a wrong digest. | |||
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: Convert to a class | Maksim Denisov | |
2025-02-27 | ArtifactBlob: Move from execution_api/common to common | Maksim Denisov | |
2025-02-21 | ByteStreamClient: Return ArtifactBlob from Read. | Maksim Denisov | |
2025-02-21 | ByteStreamUtils: Simplify ReadRequest and WriteRequest | Maksim Denisov | |
2025-02-21 | ByteStreamClient: Use ByteStreamUtils::ReadRequest internally only. | Maksim Denisov | |
2025-02-21 | ByteStreamClient: Use ArtifactBlob in Write | Maksim Denisov | |
2025-02-20 | Separate off id generation to a separate library | Klaus Aehlig | |
... and rename appropriately to reflect contents more precisely than the generic "common". This separation also disentangles dependencies a bit. | |||
2025-02-19 | ApiBundle: Remove HashFunction. | Maksim Denisov | |
And ensure every user obtains HashFunction from corresponding IExecutionApi | |||
2025-02-19 | IExecutionApi: Return supported HashFunction::Type | Maksim Denisov | |