-
Notifications
You must be signed in to change notification settings - Fork 54
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
Add test case test_drain_with_block_for_eviction_success #1733
Add test case test_drain_with_block_for_eviction_success #1733
Conversation
ref: 7521 Signed-off-by: Chris <[email protected]>
Add test case 10 times test result I will fix the CodeFacor issue later, thank you. |
84de17d
to
f1ec914
Compare
4eb6a4b
to
8feda8b
Compare
…lica_success ref: 7521 Signed-off-by: Chris <[email protected]>
8feda8b
to
b489201
Compare
Test case have potential fail rate if the drain node have backup store located, need invetigate after longhorn/longhorn#8004
|
ref: 7521 Signed-off-by: Chris <[email protected]>
3d527e3
to
2f3ad3e
Compare
10 times test of below test cases all passed
|
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
@mergify backport v1.6.x |
✅ Backports have been created
|
Add test case
test_drain_with_block_for_eviction_success
ref: 7521
Which issue(s) this PR fixes:
Issue #7521
What this PR does / why we need it:
Test case only have skeleton
Special notes for your reviewer:
10 times test result
Additional documentation or context
N/A