Age | Commit message (Collapse) | Author | |
---|---|---|---|
2024-07-16 | Remove the LocalExecutionConfig singleton | Paul Cristian Sarbu | |
...and replace it with passed instances created early via a builder pattern. | |||
2024-07-16 | Pass LocalExecutionConfig to TargetService | Paul Cristian Sarbu | |
2024-07-16 | Pass LocalExecutionConfig to ApiBundle | Paul Cristian Sarbu | |
2024-07-16 | Pass LocalExecutionConfig to LocalAction and LocalApi | Paul Cristian Sarbu | |
2024-07-16 | LocalAction, BazelAction: Extend constness in fields and methods | Paul Cristian Sarbu | |
2024-07-16 | Remove Data struct from LocalExecutionConfig | Paul Cristian Sarbu | |
...to simplify the class and clarify the singleton pattern in preparation for its removal. | |||
2024-07-16 | Add missing RemoteExecutionConfig includes | Paul Cristian Sarbu | |
2024-07-16 | Ensure config builders always have a valid state | Paul Cristian Sarbu | |
...by making the respective Build methods const. It should be perfectly valid for multiple Build calls to happen for the same builder instance, so its internal state should never be invalidated by, e.g., moving from internal fields. | |||
2024-07-16 | test_auth_config: Fix wrong deps | Paul Cristian Sarbu | |
2024-07-16 | test: Reduce verbosity in config methods | Paul Cristian Sarbu | |
As the classes describe the config type, their static creators can have generic names, which reduces unnecessary verbosity. | |||
2024-07-16 | Set compatibility in tests | Maksim Denisov | |
2024-07-16 | Check compatibility in the test of large_object_cas | Maksim Denisov | |
2024-07-16 | Check compatibility in the test of cas_server | Maksim Denisov | |
2024-07-16 | Check compatibility in the test of file_root | Maksim Denisov | |
2024-07-16 | Check compatibility in the test of source_map | Maksim Denisov | |
2024-07-16 | Set compatibility in tests for Serve | Maksim Denisov | |
2024-07-16 | Add a test verifying that reconstructed executables are usable | Klaus Aehlig | |
... ensuring we do not regress on the recently fixed race that allowed file descriptors to reconstructed executables to be kept alive. | |||
2024-07-15 | Update CHANGELOG with latest fixes | Klaus Aehlig | |
2024-07-12 | Make end-to-end tests independent of the user's rc file | Klaus Aehlig | |
Our end-to-end tests involve calling just-mr. When run locally, in order to make then self-contained we need to make sure this test call to just-mr does not pick up the user's .just-mrrc that might contain different setting not overridden at the command line. | |||
2024-07-12 | Remove redundant ArtifactFactory class used in tests only | Maksim Denisov | |
...and move the related tests to artifact_description.test | |||
2024-07-12 | ArtifactFactory: remove FromDescription method | Maksim Denisov | |
...since it is used in tests only. | |||
2024-07-12 | ArtifactFactory: remove Identifier method | Maksim Denisov | |
...since it is used in tests only. It also duplicated serialization-deserialization of ArtifactDescription. | |||
2024-07-12 | Use precalculated hash for comparison of ArtifactDescriptions | Maksim Denisov | |
...instead of comparing raw data. | |||
2024-07-12 | Use static Create functions to construct ArtifactDescription | Maksim Denisov | |
...instead of unobvious ctors relying on overload resolution. | |||
2024-07-12 | Move implementation details of ArtifactDescription to the cpp file | Maksim Denisov | |
2024-07-12 | DAG: remove methods used in tests only | Maksim Denisov | |
2024-07-12 | DAG: remove IsValid since it is used in tests only. | Maksim Denisov | |
2024-07-12 | DAG: remove unused methods | Maksim Denisov | |
2024-07-12 | Capture in TreeReaders by pointer to avoid temporaries | Maksim Denisov | |
2024-07-12 | Replace IBundle classes family with BazelBlob | Maksim Denisov | |
2024-07-12 | Return std::nullopt if creation of an action digest fails | Maksim Denisov | |
...instead of dereferencing nullptr. | |||
2024-07-12 | Pack arguments of CreateActionDigestFromCommandLine to a struct | Maksim Denisov | |
2024-07-12 | Remove unused parameters in CreateDirectoryDigestFromTree | Maksim Denisov | |
2024-07-12 | Move BazelMsgFactory reading functions to a separate class | Maksim Denisov | |
2024-07-12 | Store newly spliced executables as copies. | Maksim Denisov | |
During multithreaded splicing, the main process can be forked (inheriting open file descriptors). In this case, the executable file saved using hardlinking becomes inaccessible. To prevent this, executables must be stored as copies made in a child process. | |||
2024-07-12 | test summary defaults: use times | Klaus Aehlig | |
... as the default summarizer can make good use of that, if provided. | |||
2024-07-12 | Internal test rules: only use in summary action what is necessary | Klaus Aehlig | |
2024-07-12 | Internal test rules with infastructure: also record pwd | Klaus Aehlig | |
2024-07-12 | Test summarizer: also report on average time | Klaus Aehlig | |
... of passed tests (as only for those, the time is meaningful). Given that we read the timing information anyway, if available, we can as well report more useful information. | |||
2024-07-12 | Test rules: only provide to the summary action what is needed | Klaus Aehlig | |
... and add a rule allowing the summarizer to specify what it needs. | |||
2024-07-12 | Tests: also record `pwd` | Klaus Aehlig | |
... as test meta data. Tests are executed in an unspecified directory, assuming pass or fail is independent of the location where the test is run. While this generally is true, test logs often contain the working directory. So, in order to more easily compare different execution orders of a potential race condition, it can be desirable to compare logs "up to the execution directory". This, however, requires that this directory is recored in the first place. Do so. For consistency of the output format, also have a (fixed) artifact pwd in the summary report. | |||
2024-07-11 | Add test to check correct target-cache sharding | Paul Cristian Sarbu | |
...between local, remote, and served builds. | |||
2024-07-11 | just: Fix storage instantiation in main | Paul Cristian Sarbu | |
As the storage instance also instantiates the target cache, we need to ensure that each such instance has at that point all the correct remote endpoint information to ensure the target cache sharding works as expected. In particular, the server-side and client-side operations of just have a slightly different setup for the remote endpoint address, therefore they require the storage instantiation to be done separately. | |||
2024-07-10 | ParallelRetrieveToCas: process prerequisites in parallel | Klaus Aehlig | |
2024-07-10 | ParallelRetrieveToCas: avoid duplicated requests | Klaus Aehlig | |
While no additional blob will be transferred, doing a request for missing blobs is still a request and, in particular, an unnecessary round trip. Therefore avoid this, by remembering what we synchronized already. | |||
2024-07-10 | Artifact::ObjectInfo: add order | Klaus Aehlig | |
2024-07-10 | ArtifactDigest: add order | Klaus Aehlig | |
2024-07-09 | Test rules with infrastructure: also honor TEST_SUMMARY_EXECUTION_PROPERTIES | Klaus Aehlig | |
Also for tests that provide infrastructure, it can be desirable to run those tests a large number of times. Therefore, support additional remote-execution properties for the summary action so that a suitable remote-execution endpoint can be chosen. | |||
2024-07-08 | tutorial on tests: set shell defaults | Klaus Aehlig | |
... as they will be used in newer versions of rules-cc to set the path for the test-summary action. | |||
2024-07-08 | Remove hardlink design as implemented | Klaus Aehlig | |
... and mention the fix in CHANGELOG. |