Comments (11)
So, looking further into this, based on my reading of the OpenAPI 2.0 specification, the Kubernetes spec might not actually be valid. Or the implementation has a bug. This is something of a rabbit hole.
Kubernetes introduces a format of int-or-string
to the String
type (which is part of the spec). The text in the spec states that format "can have any value to support documentation needs" but in this case Kubernetes will pass integers to the String type, relying on parsing rules in the format.
The spec specifies a mechanism for vendor extensions. I think rather than int-or-string
being a Format on the String type, this should actually be a type of x-int-or-string
.
I'll post something to API Machinery to see what folks think.
In the meantime I'll see if I can spot a local work around.
from kubeval.
@elblivion would you be able to provide a sample config file which throws these errors?
I'm pretty sure this is due to the addition of int-or-string
in OpenAPI which is not in JSON Schema.
I have a workaround for this but looking at the code now I think the fact this function https://github.com/garethr/kubeval/blob/master/cmd/root.go#L30 takes a string might be the issue. If you could provide a failing config I'll get this fixed and add that to the acceptance tests. Thanks for trying it out and reporting the issue.
from kubeval.
Sure, this one is from an official Kubernetes add-on and has this issue:
$ cat ./heapster/heapster-service.yaml
apiVersion: v1
kind: Service
metadata:
labels:
task: monitoring
# For use as a Cluster add-on (https://github.com/kubernetes/kubernetes/tree/master/cluster/addons)
# If you are NOT using this as an addon, you should comment out this line.
kubernetes.io/cluster-service: 'true'
kubernetes.io/name: Heapster
name: heapster
namespace: kube-system
spec:
ports:
- port: 80
targetPort: 8082
selector:
k8s-app: heapster
$ kubeval ./heapster/heapster-service.yaml
The document ./heapster/heapster-service.yaml is not a valid Service
--> spec.ports.0.targetPort: Invalid type. Expected: string, given: integer
from kubeval.
So, I think their is a workaround to this that can be implemented in kubeval
before any changes that might happen upstream.
For each error in Validate
If Type() is invalid_type and
Value() is an integer and
Field() has a format of int-or-string
then don't raise the error
The implementation needs to grab the Field() value from the error, then look in the raw schema to see if that field has a format of int-or-string.
The other option is to convert at the schema level. ie. make the files published in kubernetes-json-schema actually valid according to the spec.
I'm going to investigate both when I get a moment.
from kubeval.
@elblivion so, I have a blunt workaround in #2
If you have a moment to test that out I'd appreciate feedback. I've added the sample you provided to the acceptance tests and the done now passes. It should provide you with a warning about the field in question as the code doesn't yet determine with 100% certainty that it's correct. I'll be working on improving that when I get a moment.
from kubeval.
Hi @garethr, tested this and it does what it says on the tin. The thing is, there is at least one case where we absolutely do need an error to be raised - whenever any desc.Field()
containsenv.%d.value
is an integer, since this breaks horribly a create or apply operation. Sorry if I didn't make that clearer.
Here's a sample spec that should break with env
but not other integers:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
name: nginx
labels:
app: nginx
spec:
strategy:
type: RollingUpdate
rollingUpdate:
maxUnavailable: 0
maxSurge: 2
template:
metadata:
labels:
app: nginx
spec:
containers:
- image: nginx
name: nginx
env:
- name: INTEGER_VAR
value: 80
ports:
- containerPort: 80
name: http
protocol: UDP
Current kubeval master:
The document /tmp/yaml.yml is not a valid Deployment
--> spec.template.spec.containers.0.env.0.value: Invalid type. Expected: string, given: integer
--> spec.strategy.rollingUpdate.maxSurge: Invalid type. Expected: string, given: integer
--> spec.strategy.rollingUpdate.maxUnavailable: Invalid type. Expected: string, given: integer
kubeval from #2:
---> spec.template.spec.containers.0.env.0.value is an integer, but might be an int_or_string property
---> spec.strategy.rollingUpdate.maxSurge is an integer, but might be an int_or_string property
---> spec.strategy.rollingUpdate.maxUnavailable is an integer, but might be an int_or_string property
The document /tmp/yaml.yml is a valid Deployment
from kubeval.
I've just tried a much better approach in #3 which passes both of the two test cases you provided. ie. the first one passes as valid and the second one fails like so.
The document fixtures/int_or_string_false.yaml is not a valid Deployment
--> spec.template.spec.containers.0.env.0.value: Invalid type. Expected: string, given: integer
@elblivion If you could try that out it would be much appreciated, and if it works out I'll merge and release a new versions.
from kubeval.
Works beautifully :-) Much obliged!
The only catch for us is that the Go YAML library does not support document separators (multiple specs in the same file) but that's a different issue I can open if you wish - you might want a big warning about that.
I was planning on proposing sticking to one spec per file anyway for clarity.
from kubeval.
@elblivion excellent, thanks.
Yes, I'd like to support document separators. If you could open a new issue that would be great.
from kubeval.
@elblivion thanks for your help with this. I've merged #3 and released 0.1.1 with the fix. Much appreciated.
from kubeval.
👍
from kubeval.
Related Issues (20)
- kubeval release download not in gzip format? HOT 1
- Calling bottle :unneeded is deprecated warning with brew HOT 2
- SARIF output management ?
- Failed initializing schema v1beta3
- Validation of Kustomization files HOT 1
- Validation error: Additional property seccompProfile is not allowed HOT 4
- error when attempting to lint poddisruption budget HOT 1
- Error 404 on schema json file when validate HOT 1
- Errors ignore output string type HOT 1
- ingressclass-networking-v1.json: Could not read schema from HTTP, response status is 404 Not Found HOT 7
- Valid resource failed validation with 404 schema not found
- Kubernetes OpenAPI v3 HOT 5
- kubeval of "helm create test-helm" fails HOT 1
- Latter half of domain is cut off resource group
- [BUG] ConfigMap Validation should be ERROR but is WARN HOT 3
- Connection reset by peer HOT 1
- Missing 'kind' key && getting Failed to decode YAML Errors HOT 1
- Maintenance information HOT 3
- Kubeval config file to reuse custom settings
- Add Support for .kubevalignore file HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from kubeval.