Skip to content

Show Away Dead Down Workers in Status Monitor#19444

Open
L00196726 wants to merge 3 commits intoopenSUSE:masterfrom
L00196726:fix-monitor-worker-states
Open

Show Away Dead Down Workers in Status Monitor#19444
L00196726 wants to merge 3 commits intoopenSUSE:masterfrom
L00196726:fix-monitor-worker-states

Conversation

@L00196726
Copy link
Copy Markdown

@L00196726 L00196726 commented Mar 16, 2026

  • [src/api] Return explicit non-building worker states from BuildingInformationUpdater
  • [src/api] Render away/dead/down worker states in Status Monitor page

Hey friends,

This introduces labels for the away, dead and down worker states on the project monitor page.

To verify this feature

  • Enable the interconnect to build.opensuse.org
  • Create the project home:Admin
  • Add 'openSUSE Tumbleweed' as a repository to the project
  • Branch a couple of packages into the project:
for i in `osc -A http://0.0.0.0:3000 ls openSUSE.org:home:hennevogel`; do osc -A http://0.0.0.0:3000 copypac openSUSE.org:home:hennevogel "$I" home:Admin; done
  • Visit the monitor page and see the new labels according to the worker states

Here is a screen shot of how it looks:
image

…ormationUpdater

The previous code only return the state for idle. This change also return worker states
for away, dead and down workers.

These worker states are used to show these workers on the monitoring
page. Previously they were hidden.

Fixes openSUSE#2488
Added code in monitor.js to render workers in down/dead/away states.

This is to give visibility of hosts that are not available for building
on the Status Monitor page.

Fixes openSUSE#2488
@github-actions github-actions bot added the Frontend Things related to the OBS RoR app label Mar 16, 2026
@L00196726 L00196726 marked this pull request as ready for review March 16, 2026 22:33
@hennevogel
Copy link
Copy Markdown
Member

Show Away Dead Down Workers in Status Monitor

Why?

@L00196726
Copy link
Copy Markdown
Author

L00196726 commented Mar 17, 2026

Show Away Dead Down Workers in Status Monitor

Why?

Hi @hennevogel
I saw this issue #2488 and found it an easy one for a contribution.
The issue mention "It would be nice" to show those on this page, but it does not give more details. This PR was my understanding on the request of that issue. If you think I missunderstood the issue, I accept the possibility to be wrong. Or if you have a different understanding of the issue, I can also adjust and rework it to make it according.
Also, it might be the issue is no longer required as the issue is years old now. In that case, let me know and happy to close the PR.

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

Labels

Frontend Things related to the OBS RoR app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants