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.

Sign up for GitHub

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

Jump to bottom

Isolation level is not synchronize to default rule when setting with pd-ctl #7121

Closed
rleungx opened this issue Sep 20, 2023 · 0 comments · Fixed by #7122
Closed

Isolation level is not synchronize to default rule when setting with pd-ctl #7121

rleungx opened this issue Sep 20, 2023 · 0 comments · Fixed by #7122
Labels
affects-5.3 affects-5.4 affects-6.1 affects-6.5 affects-7.1 severity/major type/bug The issue is confirmed as a bug.

Comments

@rleungx
Copy link
Member

rleungx commented Sep 20, 2023

Bug Report

What did you do?

Use pd-ctl to set location labels and isolation level

What did you expect to see?

Both two configurations in the placement rule are updated

What did you see instead?

Only location label is updated

What version of PD are you using (pd-server -V)?
@rleungx rleungx added the type/bug The issue is confirmed as a bug. label Sep 20, 2023
@rleungx rleungx mentioned this issue Sep 20, 2023
Merged
ti-chi-bot bot added a commit that referenced this issue Sep 21, 2023
@rleungx @ti-chi-bot
*: fix sync isolation level to default placement rule (#7122)
5b3d017 
close #7121

Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@ti-chi-bot ti-chi-bot mentioned this issue Sep 21, 2023
Closed
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Sep 21, 2023
@rleungx @ti-chi-bot
This is an automated cherry-pick of tikv#7122
44ff3f7 
close tikv#7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this issue Sep 21, 2023
Merged
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Sep 21, 2023
@rleungx @ti-chi-bot
This is an automated cherry-pick of tikv#7122
cc7a825 
close tikv#7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Sep 21, 2023
@rleungx @ti-chi-bot
This is an automated cherry-pick of tikv#7122
e2d33e0 
close tikv#7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this issue Sep 21, 2023
Merged
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Sep 21, 2023
@rleungx @ti-chi-bot
This is an automated cherry-pick of tikv#7122
804b1fb 
close tikv#7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this issue Sep 21, 2023
Merged
@ti-chi-bot ti-chi-bot mentioned this issue Sep 21, 2023
Merged
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Sep 21, 2023
@rleungx @ti-chi-bot
This is an automated cherry-pick of tikv#7122
74b1b42 
close tikv#7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
ti-chi-bot bot added a commit that referenced this issue Sep 28, 2023
@ti-chi-bot @rleungx @ti-chi-bot
*: fix sync isolation level to default placement rule (#7122) (#7127)
fb6139c 
close #7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: Ryan Leung <rleungx@gmail.com>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@niubell niubell mentioned this issue Oct 18, 2023
Merged
18 tasks
ti-chi-bot bot added a commit that referenced this issue Nov 22, 2023
@ti-chi-bot @rleungx @ti-chi-bot
*: fix sync isolation level to default placement rule (#7122) (#7126)
7d0708f 
close #7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: Ryan Leung <rleungx@gmail.com>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx added a commit to ti-chi-bot/pd that referenced this issue May 11, 2024
@rleungx
This is an automated cherry-pick of tikv#7122
5e56ccc 
close tikv#7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
ti-chi-bot bot added a commit that referenced this issue Sep 10, 2024
@ti-chi-bot @rleungx @ti-chi-bot
*: fix sync isolation level to default placement rule (#7122) (#7125)
5c8d3f3 
close #7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: Ryan Leung <rleungx@gmail.com>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ti-chi-bot bot pushed a commit that referenced this issue Sep 11, 2024
@ti-chi-bot @rleungx
*: fix sync isolation level to default placement rule (#7122) (#7124)
1747c2f 
close #7121

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>

Co-authored-by: Ryan Leung <rleungx@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects-5.3 affects-5.4 affects-6.1 affects-6.5 affects-7.1 severity/major type/bug The issue is confirmed as a bug.
Projects
Questions and Bug Reports
Status: Closed
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

*: fix sync isolation level to default placement rule rleungx/pd
1 participant
@rleungx