kn service list is not sorted

[steren]

$ kn version
Version:      v20190731-71c39e8
Build Date:   2019-07-31 10:04:14
Git Revision: 71c39e8
Dependencies:
- serving:    v0.6.0

Expected Behavior

$ kn service list lists services sorted by alphabetical order.

Actual Behavior

$ kn service list does not seem to have any particular sorting.

[navidshaikh]

I've built the client from current master and I see that service list is printed in sorted order

15:38 ➜  client git:(master)  kn service create z --image gcr.io/knative-samples/helloworld-go --async
Service 'z' successfully created in namespace 'default'.

15:39 ➜  client git:(master)  kn service create s --image gcr.io/knative-samples/helloworld-go --async
Service 's' successfully created in namespace 'default'.

15:39 ➜  client git:(master)  kn service create b --image gcr.io/knative-samples/helloworld-go --async
Service 'b' successfully created in namespace 'default'.

15:39 ➜  client git:(master)  kn service create a --image gcr.io/knative-samples/helloworld-go --async
Service 'a' successfully created in namespace 'default'.

15:39 ➜  client git:(master)  kn service create az --image gcr.io/knative-samples/helloworld-go --async
Service 'az' successfully created in namespace 'default'.

15:39 ➜  client git:(master)  kn service create ab --image gcr.io/knative-samples/helloworld-go --async
Service 'ab' successfully created in namespace 'default'.

15:39 ➜  client git:(master)  kn service list
NAME   URL   GENERATION   AGE   CONDITIONS   READY     REASON
a            1            13s   0 OK / 3     Unknown   OutOfDate : The Route is still working to reflect the latest desired specification.
ab           1            4s    0 OK / 3     Unknown   OutOfDate : The Route is still working to reflect the latest desired specification.
az           1            8s    0 OK / 3     Unknown   OutOfDate : The Route is still working to reflect the latest desired specification.
b            1            21s   0 OK / 3     Unknown   OutOfDate : The Route is still working to reflect the latest desired specification.
s            1            27s   0 OK / 3     Unknown   OutOfDate : The Route is still working to reflect the latest desired specification.
z            1            43s   0 OK / 3     Unknown   OutOfDate : The Route is still working to reflect the latest desired specification.

@steren could you please re-check and probably add a sample output ?

/cc @daisy-ycguo

[rhuss]

@navidshaikh I don't think it would harm if we had a client-side sorting, too. E.g. it could be that one Knative implementation (e.g. in Kubernetes) does a server-side sorting and another (e.g. Cloud Run on Appengine) does not (i guess its the latter for @steren ).

To be safe we have to sort client-side (as a sort order is not part of the Knative serving spec anyway).

[navidshaikh]

@rhuss ah! right. Makes sense. 👍
/cc @daisy-ycguo