-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Refactor mock tests in different files #113930
Conversation
@gnufied: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/kind cleanup |
/release-note-none |
32ce77b
to
a095c76
Compare
I checked the same CSI Mock tests are running in this PR as it was before + the tests look roughly the same ( |
/assign @pohly |
/assign @msau42 |
/hold Can we please merge #112923 first? That PR was pushed back to go in as soon as the branch reopens, but it will conflict with almost any other PR that touches E2E tests. |
yeah that seems fine. Do we have necessary lgtms and approvals on other PR though? |
Not yet, but there was a general agreement that it should go in. We just need to formalize that and then hit a time period where there no merge conflicts that would require a rebase and thus invalidate the lgtm. |
56443d4
to
a095c76
Compare
If you still need this PR then please rebase, if not, please close the PR |
a095c76
to
40dc14f
Compare
40dc14f
to
98114f5
Compare
Change code to use pointer for mock driver setup
98114f5
to
9e5d082
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
/approve
/hold cancel |
/assign @jsafrane For approval. |
/retest |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gnufied, jsafrane, msau42, pohly The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
CSI mock tests are becoming this bag of functions which are ever expanding in same file.
Lets split them according to features they are testing.
/sig storage