-
Notifications
You must be signed in to change notification settings - Fork 226
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
Add support for Istio resources #80
Comments
I'd be interested to work on this. |
There are some details in #47 as well. This should just require generating the JSON Schemas from the Istio OpenAPI docs (I use https://github.com/garethr/openapi2jsonschema) and then including them in the right location. If someone can point me at the openapi description for Istio I might be able to add better support for CRDs in kubeval. |
Is it possible to hit a dynamic resource for this? As I do more CRD work (and work with knative, etc.) running a 'kubectl proxy' and hitting the /api endpoints there would be pretty fantastic.. |
@tmc yes, at least theoretically. The problem there is you can only test against that specific version of the schema, and you need a server running to run the tests - which might be OK depending on your use case. |
@garethr Any update on this ? Can I help ? |
For people that will like Istio or other CRD like Knative I subject that skip validation for specific apiVersion that you pass to your script before running KubeVal. |
hi @garethr , in general it would be useful to have possibility to add additional schemas (alongside k8s one's) - how I can see it, it could be via config file, when location for additional schemas could be placed. What do you think of it ? My use case :
cheers |
So we can't run kubeval if there is a CRD in the |
I'm going to close this as:
|
Considering this resource:
I get:
This is the API reference: https://istio.io/docs/reference/config/istio.networking.v1alpha3/
The text was updated successfully, but these errors were encountered: