charts icon indicating copy to clipboard operation
charts copied to clipboard

feat: add the option to disable api server registration for metrics-server

Open raphapr opened this issue 1 year ago • 3 comments

Add the metricsServer.registerAPIService option to disable external metrics API registration.

This is useful when you already have an API registration (e.g. Datadog Cluster Agent) and want to deploy KEDA without impacting the current external metrics API.

Checklist

  • [x] I have verified that my change is according to the deprecations & breaking changes policy
  • [x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • [x] README is updated with new configuration values (if applicable) learn more
  • [x] A PR is opened to update KEDA core (repo) (if applicable, ie. when deployment manifests are modified)

raphapr avatar Feb 15 '24 19:02 raphapr

@zroubalik I understand that is mandatory. My rationale is there are potential scenarios where deploying KEDA without registering its metrics server becomes useful for migration purposes.

For example, in my case, I manage a cluster in which 1beta1.external.metrics.k8s.io is pointed to datadog; so deploying KEDA without deregistering the actual metrics provider is desired to facilitate the creation of the KEDA objects - I want to migrate the datadogmetric to scaledobject first. And subsequently, I can switch the API by using metricsServer.registerAPIService.

Does that make sense?

raphapr avatar Feb 15 '24 21:02 raphapr

Does that make sense?

~Yes, it can make sense IMO.~

~I guess that we could accept that feature now because I'll give another try to the proxy for supporting multiple metric servers (there have been some poeple reluctant to the subdomain idea in control plane side), so probably we will have to integrate it sooner or later and it also allows using KEDA only for ScaledJobs~

~Of course, this is something that requires a good explanation in docs, but I'd say that we could continue~

~WDYT?~

JorTurFer avatar Feb 15 '24 22:02 JorTurFer

I've been thinking about this and I'm not totally sure. Even though I understand the use case, I'm not sure if we are in the moment for supporting this feature as the proxy is still an on-going idea. I won't block it either but it has to be really well documented with the side effects of this. @tomkerkhove ?

JorTurFer avatar Feb 16 '24 10:02 JorTurFer