[testing] Add validation for concurrent cache in Conan #18098
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Uilian Ries <[email protected]>
memsharded
reviewed
Apr 8, 2025
| from conan.test.assets.genconanfile import GenConanfile | ||
| from conan.test.utils.tools import TestClient | ||
|
|
||
| _parallel_options = [ |
There was a problem hiding this comment.
The idea was to start first with concurrency for conan config install/install-pkg which hopefully would be a bit easier problem
There was a problem hiding this comment.
Added the test test_parallel_config to validate conan config install
| """ | ||
| num_builds = len(_parallel_options) | ||
| test_client = TestClient() | ||
| test_client.run("new cmake_lib -d name=parallel -d version=0.1.0") |
There was a problem hiding this comment.
The concurrency over test_package would be a different problem. Lets focus first on the cache concurrency, the test_package can have other mechanisms, for example using build_folder_vars using the option will create independent folders.
There was a problem hiding this comment.
Simplified the test to execute without test package. Plus, added a new one only to validate conan export. Please, see 86e341d
Signed-off-by: Uilian Ries <[email protected]>
It's not possible to guarantee the concurrent failure as it may depends on the operate system as well. For instance, in MacOS the failure ocurrs less than in Linux (based on tests executed in the CI) Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello!
This new validation introduces a test for concurrent. This initial validation started in this gist: https://gist.github.com/uilianries/e0438f6ac92f8444a1cb89c6faa3a4a7
So far, we found some conflicts:
/cc @AbrilRBS @memsharded
Changelog: Omit
Docs: Omit
developbranch, documenting this one.