add pod count onto Node view

[thllxb]

Please add pod count onto Node view to show the number of pods running on each node

Is your feature request related to a problem? Please describe.
Nope

Describe the solution you'd like
Add a pod count column onto Node view

Describe alternatives you've considered
now I have to do it through kubectl and wc -l

Additional context

[derailed]

@thllxb I think you can press enter for a given node and it should show all the pods (with count) running on that node. Does that help?

[thllxb]

@derailed Thx for the reply. I know I can see the pod count when I press enter on the node, but I was looking for an overview of pod counts on all node instead of a single node view.
I do admit this is a very minor feature and not important at all. I'm happy with the current design already. You may close this issue if you think the same.

[derailed]

@thllxb Thank you! Very kind of you to say. I'll need to noodle on this a bit but I think it will slow things down loading the view as these aren't (currently) precomputed tallies. What would showing the #pods per nodes indicate for you?

[thllxb]

Azure AKS has the limit of 30 pods per node. So, for me, it is something I would have to check every now and then. When we have 50+ nodes, it is a lot of enter to press.

[derailed]

@thllxb Ah! indeed excellent point! Thank you!! I'll take a peek...

[thllxb]

But, again, I have kubectl and wc -l at hand, so it is not a big deal to not have this feature in k9s.

[derailed]

@thllxb Totally! I do see your point and think this could be useful to other K9ers as well. Would you be ok if the pod counts are eventually consistent ie depending on your cluster size the numbers would eventually report the actual tally of pods? If so I think we can make this happen...

[thllxb]

@derailed Sorry, I don't think I understand your question. What is pod counts are eventually consistent?

[derailed]

@thllxb yes sorry... by that I mean that on first load the counts might be a tad off based on whether the pod cache is up to date or not. Then the view will zero in on the actual counts once the k9s cache is updated. Does this make more sense?

[derailed]

@thllxb Let's try this out in the next drop (0.21.4). If problematic will pull it out. Lmk what u think and if it fits the bill...
Tx!!

[thllxb]

@derailed
I tried 0.21.4, and it is great! Now I understand what you meant but it is completely fine for me.
Thanks a lot for the improvement!

[derailed]

@thllxb My pleasure! You've been kind enough to make the effort to sponsor the K9s project and in turn I'd like to make the effort to help better your k8s experience. It makes me happy to hear, this is what you were looking for. Thank you!

[Xyaren]

I think this feature causes me problems:
It is no longer possible to list nodes without havin permissions on pods on a cluster level :/

[thllxb]

I think this feature causes me problems:
It is no longer possible to list nodes without havin permissions on pods on a cluster level :/

@derailed If you want to fix this problem by removing the feature, I'm ok with it.

[Xyaren]

If don't think thats neccecary. It just should not break the node view. You could maybe display a "?" or remove the column entirely when the get-pods request failes due to rbac :)