Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: refactor lvm tests and add lvm tests to lvm testplan #69

Merged
merged 1 commit into from
Sep 27, 2024
Merged

fix: refactor lvm tests and add lvm tests to lvm testplan #69

merged 1 commit into from
Sep 27, 2024

Conversation

rohan2794
Copy link
Collaborator

@rohan2794 rohan2794 commented Sep 23, 2024 •
edited
Loading

  • create loop device over /mnt otherwise if it's created over /tmp, it might get deleted over node reboot
  • verify lvm snapshot and content to be ready and it also verify that snapshot and content restore size should be zero
  • refactor lvm test cases
  • add remaining lvm tests to lvm test profile

@r1jt r1jt self-requested a review September 27, 2024 16:46
@rohan2794 rohan2794 merged commit f138c3c into develop Sep 27, 2024
2 checks passed
@rohan2794 rohan2794 deleted the lvm-tests-refactor branch September 27, 2024 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@r1jt r1jt r1jt approved these changes

@blaisedias blaisedias blaisedias approved these changes

@w3aman w3aman Awaiting requested review from w3aman

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rohan2794 @blaisedias @r1jt