Skip to content
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

mcs: inconsistency in watch placement rule #7418

Closed
lhy1024 opened this issue Nov 23, 2023 · 0 comments · Fixed by #7550
Closed

mcs: inconsistency in watch placement rule #7418

lhy1024 opened this issue Nov 23, 2023 · 0 comments · Fixed by #7550
Assignees
Labels
severity/major The issue's severity is major. type/bug The issue is confirmed as a bug.

Comments

@lhy1024
Copy link
Contributor

lhy1024 commented Nov 23, 2023

Bug Report

What did you do?

run SetPlacementRules test in api mode and pd mode

What did you expect to see?

it is consistency

What did you see instead?

Before SetPlacementRules
[2023/11/23 11:59:59.383 +08:00] [INFO] [rule.go:433] ["get placement rules"] [bundles="[{\"group_id\":\"pd\",\"group_index\":0,\"group_override\":false,\"rules\":[{\"group_id\":\"pd\",\"id\":\"default\",\"start_key\":\"\",\"end_key\":\"\",\"role\":\"voter\",\"is_witness\":false,\"count\":3}]}]"]

After SetPlacementRules in PD mode
[2023/11/23 11:57:14.503 +08:00] [INFO] [rule.go:433] ["get placement rules"] [bundles="[{\"group_id\":\"5\",\"group_index\":5,\"group_override\":false,\"rules\":[{\"group_id\":\"5\",\"id\":\"foo\",\"index\":1,\"start_key\":\"\",\"end_key\":\"\",\"role\":\"voter\",\"is_witness\":false,\"count\":1,\"create_timestamp\":1700711834}]}]"]

After SetPlacementRules in API mode
[2023/11/23 11:54:05.863 +08:00] [INFO] [api.go:959] ["get placement rules"] [bundles="[{\"group_id\":\"pd\",\"group_index\":0,\"group_override\":false,\"rules\":[{\"group_id\":\"pd\",\"id\":\"default\",\"start_key\":\"\",\"end_key\":\"\",\"role\":\"voter\",\"is_witness\":false,\"count\":3}]},{\"group_id\":\"5\",\"group_index\":5,\"group_override\":false,\"rules\":[{\"group_id\":\"5\",\"id\":\"foo\",\"index\":1,\"start_key\":\"\",\"end_key\":\"\",\"role\":\"voter\",\"is_witness\":false,\"count\":1,\"create_timestamp\":1700711616}]}]"]

What version of PD are you using (pd-server -V)?

@lhy1024 lhy1024 added the type/bug The issue is confirmed as a bug. label Nov 23, 2023
@github-actions github-actions bot added this to Need Triage in Questions and Bug Reports Nov 23, 2023
@lhy1024 lhy1024 self-assigned this Nov 29, 2023
@lhy1024 lhy1024 added the severity/major The issue's severity is major. label Nov 29, 2023
ti-chi-bot bot added a commit that referenced this issue Dec 18, 2023
ref #7418

Signed-off-by: lhy1024 <admin@liudos.us>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Questions and Bug Reports automation moved this from Need Triage to Closed Dec 20, 2023
ti-chi-bot bot added a commit that referenced this issue Dec 20, 2023
close #7418

Signed-off-by: lhy1024 <admin@liudos.us>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/major The issue's severity is major. type/bug The issue is confirmed as a bug.
Projects
1 participant