-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
kola: add raid0 tests for root and data devices #36
Conversation
aabada2
to
b748b0c
Compare
opened "Dec 13, 2019" ahah - @dongsupark are you still interested by contributing to this PR ? I think it would be nice to have this new tests ! |
@tormath1 Thanks for the reminder. 👍 |
b748b0c
to
b509b5f
Compare
Rebased. Locally tested.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me, happy to have more tests to cover this part. 💪 I added some suggestions to reduce code duplication. :)
To be able to parametrize raid levels for RAID tests, we should create a helper function `ExecTemplate()` that reads template and parses it into an actual output string. Also make the existing root raid test use `ExecTemplate()`.
The existing data raid test should also make use of `ExecTemplate()` helper. Convert the input config for raid1 data also into the ignition format, to make it consistent with the raid1 root config.
Instead of manually checking if a type is raid1, we should create a helper `isValidRaidType()` that checks if it is one of the pre-defined raid types.
b509b5f
to
2e4481c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last thing and we'll good to go: we can just add an entry into the changelog :)
We should test root & data devices, not only with raid1 but also with raid0, to catch issues with raid0 devices. To reduce amount of code, we iterate through the map of raidTypes.
Add an entry for new kola raid0 tests. Fix a typo.
2e4481c
to
95aa844
Compare
Done, added changelog. |
We should test root and data devices not only with raid1 but also with raid0, to catch issues with raid0 devices.
To be able to parametrize raid levels for RAID tests, we should create a helper function
ExecTemplate()
that reads template and parses it into an actual output string. Also make the existing raid tests useExecTemplate()
.