-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Error messages are not human friendly #1304
Comments
agreed. might try a later release though. not sure where you are, but (some of) those strings aren't in head. |
I'm on kubectl |
kubectl 1.15 has kustomize 2.0.3 embedded. Kustomize just released 3.0.0. See https://github.com/kubernetes-sigs/kustomize/releases Not saying the error messages are vastly better, but many other things are improved. |
Cool, I'll have to check that out. Feel free to close this if its been improved |
Yeah, I'm sure these are better, though not perfect.
with v3 kustomize and v1.15 kubectl is the best option right now. |
Error message "already exists b the map" still exists in latest Kubectl. I'm not sure how to check the Kustomize version. I was following the instructions from https://kubernetes.io/docs/tutorials/stateful-application/mysql-wordpress-persistent-volume/
|
@johntellsall #1500 We have technical problems to update the kustomize version in kubectl. But we are working on it. |
Example:
error: MergeResourcesWithErrorOnIdCollision: id '"~G_v1_Namespace|~X|~P|istio-system|~S"' already used
I have no clue what all of these symbols mean which makes this hard to debug. I think this means I have a duplicate namespace defined?
Another example:
error: rawResources failed to read Resources: GroupVersionKindName: resid.ResId{gvKind:gvk.Gvk{Group:"config.istio.io", Version:"v1alpha2", Kind:"attributemanifest"}, name:"istioproxy", prefix:"", suffix:"", namespace:"istio-system"} already exists b the map
This one is even harder to understand, what does "already exists b the map" mean?
It would be great if the errors could be more humanly consumable for a better UX
The text was updated successfully, but these errors were encountered: