-
Notifications
You must be signed in to change notification settings - Fork 305
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
[Feature]: Support multi-instance AMS REST service #2810
Comments
Are you still working on this? I'm interesting in this too. @link3280 |
I'm reconsidering the proposal since @majin1102 plans to support multi-instance AMS. We could align the efforts to avoid duplicate work. What do you think? @baiyangtx |
I started a discussion and provided a design. #2997 You are welcome to participate in the discussion. @link3280 @majin1102 |
+1 We also need this part, design can be completed together will reduce duplication of work |
The implementation of this issue can be divided into several steps:
|
Description
Currently, only the leader AMS would provide REST service. When an AMS leader transition happens, the standby AMS would take 30+ seconds to initialize optimizing, after which the REST service is available. That affects the availability of the REST service.
I proposed to make the REST service independent of the leadership. That's to say, the stand-by AMS instances need to initialize and run REST service on start. Their only difference to a leader instance is no optimizing service.
Use case/motivation
No response
Describe the solution
I drew a diagram of the main modules in AMS.
Subtasks
No response
Related issues
No response
Are you willing to submit a PR?
Code of Conduct
The text was updated successfully, but these errors were encountered: