-
Notifications
You must be signed in to change notification settings - Fork 33
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
PWX-32648: Adding Install/Uninstall test cases #1198
Conversation
This PR is stale because it has been in review for 3 days with no activity. |
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.
Love the testing methodology overall! A few nits as far as the case generation but should be easy to fix
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.
Looks good overall. And much needed I feel.
Just some nits and questions.
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.
lgtm
Install/Uninstall tests for PX-StoreV2 https://portworx.testrail.net/index.php?/runs/view/6624&group_by=cases:custom_automated&group_order=asc The above link has a bunch of tests where we try different combinations of the below to make sure dmthin is installed. - Journal device - KVDB device - Auto Journal - Metadata Device - -T px-storev2 option Two operator integration tests are added to enforce the above combinations and have 32 tests Signed-off-by: Naveen Revanna <nrevanna@purestorage.com>
5b26bdc
to
511efd7
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.
LGTM!
Install/Uninstall tests for PX-StoreV2
https://portworx.testrail.net/index.php?/runs/view/6624&group_by=cases:custom_automated&group_order=asc
The above link has a bunch of tests where we try different
combinations of the below to make sure dmthin is installed.
Two operator integration tests are added to enforce the above
combinations and have 32 tests
What this PR does / why we need it:
Which issue(s) this PR fixes (optional)
Closes #
Special notes for your reviewer:
Testing notes:
Two jenkins jobs were created to run the two tests added
https://jenkins.pwx.dev.purestorage.com/job/3.1.0-dev-jobs/job/vSphere-withDmthinOption-opint/
https://jenkins.pwx.dev.purestorage.com/job/3.1.0-dev-jobs/job/vSphere-withoutDmthinOption-opint/
The other run