-
Notifications
You must be signed in to change notification settings - Fork 725
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
tests/rule: make TestLargeRules more stable #8038
Conversation
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
Is there any error stack? |
This condition will never be met, I printed pd/tests/server/api/rule_test.go Lines 1271 to 1273 in 7e95abf
|
@@ -1261,6 +1261,13 @@ func (suite *ruleTestSuite) postAndCheckRuleBundle(urlPrefix string, bundle []pl | |||
err = tu.CheckGetJSON(testDialClient, urlPrefix+"/config/placement-rule", nil, | |||
tu.StatusOK(re), tu.ExtractJSON(re, &respBundle)) | |||
re.NoError(err) | |||
// skip default rule check |
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.
Will it affect TestDeleteAndUpdate
?
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.
will add default rule after the previous test finished
Why does it happen? Does it share the same environment?
yes, this suite test build env firstly and then using pd/tests/server/api/rule_test.go Lines 51 to 56 in 7e18a69
|
/hold meet other issues need to be fixed |
Root cause is
So, we can close this pr now :) |
What problem does this PR solve?
Issue Number: Ref #7969
What is changed and how does it work?
tested by #7969
Because below code will add default rule after the previous test finished, we need to skip default rule for large rule test to make it stable
pd/tests/server/api/rule_test.go
Lines 62 to 78 in 7e95abf
Check List
Tests
Release note