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

Label distribution has residual labels #4825

Closed
rleungx opened this issue Apr 20, 2022 · 0 comments · Fixed by #4824
Closed

Label distribution has residual labels #4825

rleungx opened this issue Apr 20, 2022 · 0 comments · Fixed by #4824
Labels
affects-5.0 affects-5.1 affects-5.2 affects-5.3 affects-5.4 affects-6.0 severity/minor type/bug The issue is confirmed as a bug.

Comments

@rleungx
Copy link
Member

rleungx commented Apr 20, 2022

Bug Report

What did you do?

  1. use TiUP to start a cluster
  2. execute config set location-labels zone
  3. execute store label 1 zone z1
  4. after we can see z1 in the monitor
  5. execute store label 1 zone z2

What did you expect to see?

z1 disappears and the monitor only has z2

What did you see instead?

z1 leaves until transfer PD leader or restart PD.

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

master
@rleungx rleungx added the type/bug The issue is confirmed as a bug. label Apr 20, 2022
@rleungx rleungx mentioned this issue Apr 20, 2022
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 21, 2022
@rleungx
metrics: fix the residual label (#4824)
6c5d61c 
close #4825

Signed-off-by: Ryan Leung <[email protected]>
This was referenced Apr 21, 2022
Merged
Closed
Merged
Merged
Merged
Merged
ti-chi-bot added a commit that referenced this issue Apr 26, 2022
@ti-chi-bot @rleungx
metrics: fix the residual label (#4824) (#4826)
1583c0f 
ref #4824, close #4825

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
ti-chi-bot added a commit that referenced this issue May 27, 2022
@ti-chi-bot @rleungx
metrics: fix the residual label (#4824) (#4831)
b73b12d 
ref #4824, close #4825

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
ti-chi-bot added a commit that referenced this issue May 27, 2022
@ti-chi-bot @rleungx
metrics: fix the residual label (#4824) (#4830)
c7baa24 
ref #4824, close #4825

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
ti-chi-bot added a commit that referenced this issue Jun 13, 2022
@ti-chi-bot @rleungx
metrics: fix the residual label (#4824) (#4828)
bcf3615 
ref #4824, close #4825

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
ti-chi-bot added a commit that referenced this issue Jun 13, 2022
@ti-chi-bot @rleungx
metrics: fix the residual label (#4824) (#4829)
f88ec89 
ref #4824, close #4825

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
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.0 affects-5.1 affects-5.2 affects-5.3 affects-5.4 affects-6.0 severity/minor type/bug The issue is confirmed as a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

metrics: fix the residual label rleungx/pd
2 participants
@nolouch @rleungx