Closed
Bug 475003
Opened 16 years ago
Closed 16 years ago
Sort slaves by name on buildbot /buildslaves page
Categories
(Release Engineering :: General, defect, P2)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: catlee, Assigned: catlee)
Details
Attachments
(1 file)
1.35 KB,
patch
|
bhearsum
:
review+
catlee
:
checked-in+
|
Details | Diff | Splinter Review |
It can be hard to find the slave you want on the /buildslaves page since they're not sorted. This patch sorts them in "natural order", i.e. slave10 sorts after slave2, rather than before.
This is upstream ticket #416
Attachment #358414 -
Flags: review?(bhearsum)
Updated•16 years ago
|
Attachment #358414 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 1•16 years ago
|
||
Comment on attachment 358414 [details] [diff] [review]
sort slaves on /buildslaves page
Checking in buildbot/util.py;
/cvsroot/mozilla/tools/buildbot/buildbot/util.py,v <-- util.py
new revision: 1.2; previous revision: 1.1
done
Checking in buildbot/status/web/slaves.py;
/cvsroot/mozilla/tools/buildbot/buildbot/status/web/slaves.py,v <-- slaves.py
new revision: 1.3; previous revision: 1.2
done
Attachment #358414 -
Flags: checked‑in+ checked‑in+
Comment 2•16 years ago
|
||
Comment on attachment 358414 [details] [diff] [review]
sort slaves on /buildslaves page
This hasn't been checked into the Mercurial repository yet. This will happen when we have downtime to reconfig Buildbot for it.
Attachment #358414 -
Flags: checked‑in+ checked‑in+
Updated•16 years ago
|
Priority: -- → P2
Comment 3•16 years ago
|
||
Don't know whether you've seen it, but I also have the following in place on staging-1.9-master:
http://staging-1.9-master.build.mozilla.org:8810/buildslaves
Assignee | ||
Updated•16 years ago
|
Attachment #358414 -
Flags: checked‑in+ checked‑in+
Assignee | ||
Comment 4•16 years ago
|
||
Comment on attachment 358414 [details] [diff] [review]
sort slaves on /buildslaves page
changeset: 5:cfbd868afae1
Assignee | ||
Comment 5•16 years ago
|
||
Running in production now.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•