-
Notifications
You must be signed in to change notification settings - Fork 426
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
Confusing (wrong) description on supported replica of metrics server deployment #861
Comments
There is though, only 1 replica will be active and used so it does not provide HA |
But the KEDA metrics server is an aggregated apiserver (which naturally support HA) right? Say I have two replicas of it, and there's one down, the other one can still handle the request from HPA controller. I haven't look through the code so I'm not sure if I missed sth. |
Yes but it's not going to actively load balance across multiple instances |
But @zroubalik might correct me, but I'm pretty sure that's correct |
Yeah, that the usual default way it works, ie. only one instance handles the load. For the requests to be balanced between all the instances of the metrics-apiserver, you need to set |
The issue is, that once we introduce metrics values caching on the metric server side, then we will have to somehow deal with sharing the cache between multiple replicas |
Yeah that might be a problem. So the question can become: if I setup KEDA metrics server with HA (more than 1 replica), what possible issue would I meet? |
That the only "HA" that you get, is that it will reduce failing over ones the primary fails |
@zqzten no issues at the moment when specifying multiple replicas. |
Thanks! So shall we correct the doc to say that KEDA metrics server does support HA with multiple relicas? |
Yeah, that would be great. But we should be careful with the wording, so people don't have the impression that they could have mutliple installations per cluster. |
I would prefer not to change it given it's not really HA IMO but if everything things it's best then that's that. What about this?
|
LGTM |
That might not be accurate enough? According to what @zroubalik mentioned, we can achive load balanced HA of KEDA metrics server by setting |
Just found that the official metrics server also has this recommandation, FYI: https://github.com/kubernetes-sigs/metrics-server#high-availability |
Yeah, we can also mention this property there. Are you willing to open a PR to fix this? |
Glad to. I'll open one in these days. |
In High Availability section of the operate doc, it said that KEDA metrics server does not support HA according to limitation in k8s custom metrics server.
However, the limitation mentioned is that k8s does not support more than one type of custom metrics server but not more than one replica of the same custom metrics server. So actually there's no limitation on the replica of KEDA metrics server and it does support HA. Seems the current doc is wrong there.
The text was updated successfully, but these errors were encountered: