cluster: remove created directories after fio test in tiup cluster check
#2511
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.
What problem does this PR solve?
Close #2159, #2497
In
tiup cluster check topology.yaml --enable-disk=true
, the parent directories oftiup-fio-test
are created with root but are not removed after the test. The subsequent operations will fail because the directories exist but the user doesn't have permission.What is changed and how it works?
Before creating
tiup-fio-test
, check which parent directory doesn't exist and should be removed.After testing, remove the parent directory.
Check List
Tests
/home/xxx/data
and executetiup cluster check --enable-disk=true
. After the test, check that/home/xxx/data
is removed.and execute
tiup cluster check --enable-disk=true`. After the test, check that `/home/xxx/data` still exists but it is empty.Code changes
Side effects
Related changes
Release notes: