-
Notifications
You must be signed in to change notification settings - Fork 885
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
kubectl get pods --sort-by=status
makes kubectl crash (error not handled)
#993
Comments
@cig0: This issue is currently awaiting triage. SIG CLI takes a lead on issue triage for this repo, but any Kubernetes member can accept issues by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Looks like a long time regression and still on master. In branch 1.18 it seems works in a "better" way. $ git branch
$ git branch
|
looks like this is caused by the changes of klog kubernetes/klog#79 It might be better to check if objects is sortable in advance rather than during sorting. |
@cig0 Is there somewhere in the docs saying that
I'm trying to understand the bug here, or are you saying that the error should be handled better? |
Hi @brianpursley I see, I confused it's use and hence the trace I got -- thanks. Anyway, yeah, maybe the error could be shown without the trace for a cleaner output, since it doesn't add any useful information. Thanks everyone. |
What happened:
Kubectl crashes with
F1215 09:51:35.632353 79162 sorter.go:360] Field {.status} in [][][]reflect.Value is an unsortable type: interface, err: unsortable type: map[string]interface {}
What you expected to happen:
Get a list of pods ordered by STATUS (I now know there're other ways to do this)
How to reproduce it (as minimally and precisely as possible):
Run the command in the title of this issue.
Anything else we need to know?:
Yes, following is the crash output messages:
Environment:
kubectl version
):cat /etc/os-release
): macOS and GNU+LinuxThe text was updated successfully, but these errors were encountered: