Ticket #743 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

add a /slaves status page that only shows slave status (comes with patch)

Reported by: jhford Owned by:
Priority: major Milestone: 0.8.1
Version: 0.7.12 Keywords:
Cc:

Description

I have 130 slaves that each have tons of builders associated with them. These slaves are also very commonly offline (they are phones) which means that nearly every time I load /buildslaves it is only to see if the slave is building, idle or disconnected. This patch is to create a new page /slaves that only lists the slave, admin, last heard from time and status. While /buildslaves does give me this information as well, it is slow to generate for me and requires a full screen of text for one slave which makes the information very difficult to work with.

I have sample output from my (0.7.12) master at  http://pastebin.mozilla.org/708252 . That is one slave's builder list.

Attachments

slaves-page.diff Download (4.1 KB) - added by jhford 3 years ago.
simple slave list.diff
template.diff Download (1.3 KB) - added by jhford 3 years ago.
template
ticket743-full.diff Download (5.4 KB) - added by jhford 3 years ago.
this patch replaces the other two already attached.

Change History

Changed 3 years ago by jhford

simple slave list.diff

Changed 3 years ago by jhford

template

Changed 3 years ago by jhford

this patch replaces the other two already attached.

comment:1 Changed 3 years ago by jhford

sorry about the multiple patches. The first two patches attached to this bug can be fully ignored. It turns out that git diff is a little different to what i am used to doing (hg diff).

comment:2 Changed 3 years ago by dustin

As discussed in IRC, this should really be a URL parameter for the regular /buildslaves view, maybe /buildslaves?no_builders=1

comment:3 Changed 3 years ago by dustin

  • Version set to 0.7.12
  • Milestone changed from undecided to 0.8.1

comment:4 Changed 3 years ago by Karl Norby

Make the buildslaves display optional in the /slaves display

Refs #743; still needs docs

Changeset: 6f05b86870a791877ae1ee70c2aa8691971478c9

comment:5 Changed 3 years ago by Dustin J. Mitchell

  • Status changed from new to closed
  • Resolution set to fixed

document no_builders (fixes #743)

Changeset: 6dd94bad8ba4bab07e225cb68ad8a0d8f98b8945

comment:6 Changed 3 years ago by Karl Norby

Make the buildslaves display optional in the /slaves display

Refs #743; still needs docs

Changeset: 6f05b86870a791877ae1ee70c2aa8691971478c9

Note: See TracTickets for help on using tickets.