Closed rleungx closed 3 days ago
/test pull-integration-realcluster-test
--- FAIL: TestServerTestSuite (49.63s)
--- FAIL: TestServerTestSuite/TestStoreLimit (0.24s)
server_test.go:596:
Error Trace: /home/runner/work/pd/pd/tests/integrations/mcs/scheduling/server_test.go:596
/home/runner/work/pd/pd/tests/integrations/mcs/scheduling/server_test.go:550
Error: Should be true
Test: TestServerTestSuite/TestStoreLimit
ci failed, because API cluster didn't clear the environment (the suite is internally serialized), and the InitRegions called by TestOnlineProgress generated region 2 which is different from TestStoreLimit needed, causing it to fail.
@okJiang: adding LGTM is restricted to approvers and reviewers in OWNERS files.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: HuSharp, lhy1024, okJiang
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/merge
@rleungx: We have migrated to builtin LGTM
and approve
plugins for reviewing.
👉 Please use /approve
when you want approve this pull request.
The changes announcement: Proposal: Strengthen configuration change approval.
@rleungx: Your PR was out of date, I have automatically updated it for you.
If the CI test fails, you just re-trigger the test that failed and the bot will merge the PR for you after the CI passes.
What problem does this PR solve?
Issue Number: Close #8331.
What is changed and how does it work?
When enabling scheduling service, use an individual function to check if the cluster is prepared.
Check List
Tests
Release note