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.

Sign up for GitHub

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

Jump to bottom

Monitoring Table can't display all the Alerts #2495

Closed
ChengYanJin opened this issue Apr 28, 2020 · 2 comments
Closed

Monitoring Table can't display all the Alerts #2495

ChengYanJin opened this issue Apr 28, 2020 · 2 comments
Assignees
@ChengYanJin
Labels
complexity:easy Something that requires less than a day to fix kind:bug Something isn't working topic:monitoring Everything related to monitoring of services in a running cluster topic:ui UI-related issues
Milestone
MetalK8s 2.5.1

Comments

@ChengYanJin
Copy link
Contributor

Component: ui

What happened:
When the number of alerts is more than the capacity of the table, we can't scroll down to check the last few alerts.
image

What was expected:
We can scroll down until the end of the table.

Steps to reproduce
I was trying with dev/2.6, but we have the same issue in 2.5 branch
There are many ways to create many alerts. What I did is to create many new Nodes but without deploying them and wait 15mins.

Resolution proposal (optional):
The height problem of the table.
@ChengYanJin ChengYanJin added kind:bug Something isn't working topic:ui UI-related issues complexity:easy Something that requires less than a day to fix topic:monitoring Everything related to monitoring of services in a running cluster labels Apr 28, 2020
@ChengYanJin ChengYanJin added this to the MetalK8s 2.5.1 milestone Apr 28, 2020
@ChengYanJin ChengYanJin self-assigned this Apr 29, 2020
ChengYanJin added a commit that referenced this issue Apr 29, 2020
@ChengYanJin
ui/table: Set the height to 80%
1c77209 
For some reason, if we set the height of the table to 100% we can't see the last few rows.

Refs: #2495
ChengYanJin added a commit that referenced this issue Apr 29, 2020
@ChengYanJin
ui/lint: Auto formatting
94aff46 
Refs: #2495
@ChengYanJin ChengYanJin mentioned this issue Apr 30, 2020
Merged
@gdemonet
Copy link
Contributor

gdemonet commented May 4, 2020

@ChengYanJin this can be closed I guess?

@ChengYanJin
Copy link
Contributor Author

Merged in PR #2503

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChengYanJin ChengYanJin

Labels
complexity:easy Something that requires less than a day to fix kind:bug Something isn't working topic:monitoring Everything related to monitoring of services in a running cluster topic:ui UI-related issues
Projects
None yet
Milestone
MetalK8s 2.5.1
Development

No branches or pull requests
2 participants
@ChengYanJin @gdemonet