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

monitor page does not strip columns with hidden status #8470

Open
olafhering opened this issue Sep 30, 2019 · 5 comments
Open

monitor page does not strip columns with hidden status #8470

olafhering opened this issue Sep 30, 2019 · 5 comments
Labels
Bug Frontend Things related to the OBS RoR app

Comments

@olafhering
Copy link
Contributor

On a /project/monitor/$prj URL, certain status values can be enabled and disabled. Depending on this list, cells for a pkg/repository might be empty.

On build.opensuse.org, entirely empty columns for a repository, or entirely empty rows for a pkg used to be removed from the page. Such small monitor page is not provided anymore on build.opensuse.org, all empty cells are always shown. As a result, one has to scroll around to see the actual data. Other build service instances do not have this problem.

@hennevogel
Copy link
Member

entirely empty columns for a repository, or entirely empty rows for a pkg used to be removed from the page.

Everything is as before. If you have one cell in the row or column they are shown. If not, then not. There are two things you have to take into account.

  1. The "Show [25] entries" selector which paginates the results. You might have the cell that makes the row/column display on page 7 while viewing page 1
  2. The horizontal scroll bar at the bottom of the table. You might have the cell that makes the row/column display on the far right.

And of course the combination of 1 & 2...

@coolo
Copy link
Member

coolo commented Sep 30, 2019

https://build.opensuse.org/project/monitor/devel:openQA - check openSUSE_Tumbleweed/i586. No pagination

@olafhering
Copy link
Contributor Author

Well, compare devel:languages:ocaml, home:olh:obs:devel:languages:ocaml:405, and Essentials (with arm64 disabled)
Disable 'succeeded', the former is broken, the others are as compact as expected. "All" or not makes no difference.

@hennevogel
Copy link
Member

check openSUSE_Tumbleweed/i586

Okay, repos with multiple archs where one is empty might be a problem. But if so, then this isn't new.

@hennevogel hennevogel reopened this Sep 30, 2019
@coolo
Copy link
Member

coolo commented Oct 1, 2019

https://build.opensuse.org/project/monitor/openSUSE:Leap:15.2:Staging:N - bootstrap_copy is disabled/excluded for all archs and it's still shown. I claim your "this isn't new" claim is wrong.

@eduardoj eduardoj added Bug Frontend Things related to the OBS RoR app labels Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Frontend Things related to the OBS RoR app
Projects
None yet
Development

No branches or pull requests

4 participants