helm-classic icon indicating copy to clipboard operation
helm-classic copied to clipboard

Improve parsing errors for Kube manifest files

Open technosophos opened this issue 10 years ago • 3 comments

If a manifest file cannot be parsed, we should give helpful information about line number and all that.

technosophos avatar Oct 22 '15 17:10 technosophos

+1!!!!

michelleN avatar Oct 28 '15 02:10 michelleN

@technosophos do you have an example error?

adamreese avatar Dec 08 '15 18:12 adamreese

I think I just found an example of this by accident: Simons-Air:Ghost simon$ helm install bad ---> Running kubectl create -f ... error validating "STDIN": error validating data: Object 'apiVersion' is missing in '{}'; if you choose to ignore these errors, turn validation off with --validate=false

[ERROR] Failed to upload manifests: exit status 1 Simons-Air:Ghost simon$ kubectl create -f ~/.helm/workspace/charts/bad/manifests/bad-pvc.yaml persistentvolumeclaim "nfs" created Simons-Air:Ghost simon$ Simons-Air:Ghost simon$ cat ~/.helm/workspace/charts/bad/manifests/bad-pvc.yaml apiVersion: v1 kind: PersistentVolumeClaim metadata: name: nfs spec: accessModes: - ReadWriteMany resources: requests: storage: 1Mi Simons-Air:Ghost simon$ helm --version helm version 0.3.1+d4c0fa8

nomisbeme avatar Jan 27 '16 02:01 nomisbeme