-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Provide schema for scaler configuration #2359
Comments
Provide json schema for use with kubeval or kubeconform? |
For example, yes. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
Any thoughts on how to do it @kedacore/keda-maintainers? |
Interested in this. |
Dapr does a similar thing: https://github.com/dapr/components-contrib/blob/master/bindings/azure/servicebusqueues/metadata.yaml We should really do this rather sooner than later as well to allow the community to build tooling for our scalers. Let me figure out how this is being generated. |
Hey @tomkerkhove , It would really help us validating KEDA resource definitions next to basic Kubernetes resources as well as flux definitions based on their schemas |
We need to find a way to auto-generate the schema rather than hand-crafted. @zroubalik @JorTurFer can we do this now after doing #5797? |
I think that we will be able somehow, yes. @wozniakjan ? |
I think that we can start with the tool indeed. I mean, we have already some scalers using this declarative approach, so we can do it right now and once there are more scalers supporting this approach, the docs should be updated "automatically" |
Totally agree!
+1, is this something you can help with @SpiritZhou? Maybe you can collaborate with @wozniakjan on this? |
Provide schema for scaler configuration so that we can do diffs on PRs to detect (unwanted) breaking changes, allow community to build tooling (see #249), allow platform providers to integrate and more.
This should be automatically generated based on our Go-code, or our Go-code should be linted against it.
The text was updated successfully, but these errors were encountered: