Skip to content
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

MetalK8s UI freezes to list the all pods when there is a very high number of evicted pods #3416

Open
ChengYanJin opened this issue Jun 8, 2021 · 0 comments
Labels
complexity:medium Something that requires one or few days to fix kind:bug Something isn't working topic:ui UI-related issues

Comments

@ChengYanJin
Copy link
Contributor

Component: UI

What happened:
image

When there is a very high number of evicted pods, the UI freezes when listing all the Pods on MetalK8s UI.

status:
  message: 'Pod The node had condition: [DiskPressure]. '
  phase: Failed
  reason: Evicted
  startTime: "2021-05-05T12:22:20Z"
[root@artesca centos]# k get pods -n xxx-cert
NAME                                       READY   STATUS    RESTARTS   AGE
cert-manager-76fc969599-n2xlv              0/1     Evicted   0          33d
cert-manager-76fc969599-z9jkh              1/1     Running   2          4d8h
cert-manager-cainjector-55f959b6cc-5b4dx   0/1     Evicted   0          4d3h
cert-manager-cainjector-55f959b6cc-6l7b9   0/1     Evicted   0          11d
cert-manager-cainjector-55f959b6cc-6p65b   0/1     Evicted   0          4d10h
cert-manager-cainjector-55f959b6cc-7f42c   0/1     Evicted   0          3d20h
cert-manager-cainjector-55f959b6cc-8lb7h   0/1     Evicted   0          23d
cert-manager-cainjector-55f959b6cc-96582   0/1     Evicted   0          4d1h
cert-manager-cainjector-55f959b6cc-bbb56   1/1     Running   6          3d19h
cert-manager-cainjector-55f959b6cc-dgqks   0/1     Evicted   0          4d4h
cert-manager-cainjector-55f959b6cc-dxm2c   0/1     Evicted   0          4d9h
cert-manager-cainjector-55f959b6cc-f7ccs   0/1     Evicted   0          4d1h
cert-manager-cainjector-55f959b6cc-f7q7g   0/1     Evicted   0          4d4h
cert-manager-cainjector-55f959b6cc-hsf7j   0/1     Evicted   0          4d8h
cert-manager-cainjector-55f959b6cc-j97qj   0/1     Evicted   0          4d16h
cert-manager-cainjector-55f959b6cc-kdggf   0/1     Evicted   0          33d
cert-manager-cainjector-55f959b6cc-kn7j2   0/1     Evicted   0          4d6h
cert-manager-cainjector-55f959b6cc-l4lrg   0/1     Evicted   0          4d8h
cert-manager-cainjector-55f959b6cc-ljtf2   0/1     Evicted   0          4d7h
cert-manager-cainjector-55f959b6cc-lz6gg   0/1     Evicted   0          4d12h
cert-manager-cainjector-55f959b6cc-ml8gz   0/1     Evicted   0          4d11h

What was expected:
The UI should not be frozen, and we should add a STATUS selector outside the pod table in order to be able to filter the pods base on the STATUS.
Will require a design from @Cuervino

Steps to reproduce

Resolution proposal (optional):

  1. Will implement virtualized table for all the table (thanks to the coming new Table compoent)
  2. Add a STATUS selector next to the table
  3. For the evicted pods we should display Evicted instead of Failed to be more specific same as kubectl does.
@ChengYanJin ChengYanJin added topic:ui UI-related issues complexity:medium Something that requires one or few days to fix kind:bug Something isn't working labels Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity:medium Something that requires one or few days to fix kind:bug Something isn't working topic:ui UI-related issues
Projects
None yet
Development

No branches or pull requests

1 participant