Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#743 closed enhancement (fixed)

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 (3)

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

Download all attachments as: .zip

Change History (9)

Changed 10 years ago by jhford

simple slave list.diff

Changed 10 years ago by jhford

template

Changed 10 years ago by jhford

this patch replaces the other two already attached.

comment:1 Changed 10 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 10 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 10 years ago by dustin

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

comment:4 Changed 10 years ago by Karl Norby

Make the buildslaves display optional in the /slaves display

Refs #743; still needs docs

Changeset: 6f05b86870a791877ae1ee70c2aa8691971478c9

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

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

document no_builders (fixes #743)

Changeset: 6dd94bad8ba4bab07e225cb68ad8a0d8f98b8945

comment:6 Changed 9 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.