-
Notifications
You must be signed in to change notification settings - Fork 235
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
Golang Plugins for kube-linter #642
Comments
I think this is halfly baked feature :( |
Why do you think so? I`m just curious, as don`t program in Go a lot, mostly Python and Typescript. On first glance sounds like a cool feature. For example we can define each custom linting rule as separate file, then compile it in docker build and load it in runtime when we call kube-linter. |
Good question so I've asked chat gpt about it 😄
In my experience although they look like java jars or dynamic libs they are not close to this. Another thing is popularity, I've not seen many projects using it (although there are some. Even golangci/golangci-lint which could be perfect case for a plugin system is based on including a linters and compile them all into a single binary and other projects prefer subprocesses or some other solutions like traefik/yaegi or hashicorp/go-plugin Other sources: |
ChatGPT saver of all) Yeah it makes sense, subprocess approach would be much better, only thing that can go wrong is performance. |
I'm also looking to extend with fully custom checks but based on the examples it looks like only way is to fork and rebuild, is this correct? |
Description of the problem/feature request
Ability to write your own custom plugins for linter, using golang. Looks like golang supports plugin loading through https://pkg.go.dev/plugin.
Description of the existing behavior vs. expected behavior
Doesn`t exist yet.
Additional context
Example of such plugin would be knative, as it has its own set of abstractions that you work with and I for example would like to check that my ingress controllers uses real knative services and applies required annotations for dns resolution. I would like to work on PR if you will be interested.
The text was updated successfully, but these errors were encountered: