Make the CCM tests work #1215
Closed
Make the CCM tests work #1215
Conversation
This file contains 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
Since there will be more folders in scylla/tests, not just integration tests, then we'll need to extract common logic need by all tests folders into seperate module (`common`) before moving forward. Signed-off-by: Dusan Malusev <dusan.malusev@scylladb.com>
feature(structre): extract utils into seperate common module
Cleanup unused dependancies
Implement simple abstraction that allows to run commands logging command, return status, stderr and stdout to a file and optionally to stderr/stdout
In order to programatically manipulate by scylla.yaml and cassandra.yaml We need to have some struct that provides API for that. It also needs to represent it in the way that ccm updateconf understand
CCM is cli tool to ease scylla/cassandra/dse cluster management. Using it in integration suit will help to expand test coverage.
Will be removed later. Added it so the file compiles.
This is an initial version (without any spans etc.).
It's not going to be used, since we log ccm commands using tracing.
Async closures are not yet supported.
It was release over 6 months ago so it is ok. It allows us to use LazyCell.
Tests were not passing because tests tried to use the same dir concurrently.
|
|
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.
Pre-review checklist
./docs/source/.Fixes:annotations to PR description.