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: async evict-leader-scheduler config in api server #7388

Closed
lhy1024 opened this issue Nov 17, 2023 · 3 comments · Fixed by #7389
Closed

mcs: async evict-leader-scheduler config in api server #7388

lhy1024 opened this issue Nov 17, 2023 · 3 comments · Fixed by #7389
Labels
severity/major type/bug The issue is confirmed as a bug.

Comments

@lhy1024
Copy link
Contributor

lhy1024 commented Nov 17, 2023

Bug Report

What did you do?

add evict leader scheduler config and check it

What did you expect to see?

It is synced

What did you see instead?

It is async between memory and persist config in api server.

Of course, because scheduling server is sync with persist config, so the memory data in api server is different from scheduling server
image

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

@lhy1024
Copy link
Contributor Author

lhy1024 commented Nov 17, 2023

It appears that there is a synchronization discrepancy between the API server and the scheduling server regarding the paused state of the store. During testing, we observed that the scheduling server reflects store 2 as being paused, whereas the API server indicates that store 3 is paused.

This misalignment may be attributed to a lack of initial configuration and cleanup, which has led to an inconsistency in the in-memory representation of the store's state on the API server when compared to the persisted configuration.

@rleungx
Copy link
Member

rleungx commented Nov 20, 2023

Is it duplicated with #7257?

@lhy1024
Copy link
Contributor Author

lhy1024 commented Nov 20, 2023

Is it duplicated with #7257?

#7388 is a part of #7257. There still be four tests that cannot run in API mode.

ti-chi-bot bot added a commit that referenced this issue Nov 20, 2023
close #7388

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
rleungx pushed a commit to rleungx/pd that referenced this issue Dec 1, 2023
close tikv#7388

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 type/bug The issue is confirmed as a bug.
Projects
Development

Successfully merging a pull request may close this issue.

2 participants