911136 - Allow editing of relevant slavealloc fields directly from slave health

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Ed Morley [:emorley]]

It would be helpful if the slave health pages, eg:
https://secure.pub.build.mozilla.org/builddata/reports/slave_health/slave.html?name=talos-r4-lion-031

...linked to slavealloc for that slave, so sheriffs can quickly jump to slavealloc to disable the slave (and not have to load the main slavealloc UI, wait for the UI to stop hanging and copy/paste the slave name  etc).

Thank you! :-)

Comment 1

[Chris Cooper [:coop] (he/him)]

slavealloc doesn't make this easy, unfortunately. AFAICT, you can't provide a slavename or pattern and have the slavealloc display limited automatically.

I would much rather have the slavealloc information editable directly in slave health. Would that be acceptable?

Comment 2

[Ed Morley [:emorley]]

(In reply to Chris Cooper [:coop] from comment #1)
> I would much rather have the slavealloc information editable directly in
> slave health. Would that be acceptable?

Definitely :-)

Comment 3

[Chris Cooper [:coop] (he/him)]

(In reply to (Away 12th-20th July) [:edmorley] from comment #2)
> (In reply to Chris Cooper [:coop] from comment #1)
> > I would much rather have the slavealloc information editable directly in
> > slave health. Would that be acceptable?

Question about scope here:

How many fields in slavealloc do sheriffs actually use? I'm guessing Notes and Enabled? only, which would make this quicker to a useful first implementation.

Comment 4

[Ed Morley [:emorley]]

(In reply to Chris Cooper [:coop] from comment #3)
> How many fields in slavealloc do sheriffs actually use? I'm guessing Notes
> and Enabled? only, which would make this quicker to a useful first
> implementation.

Correct :-)

Comment 5

[Chris Cooper [:coop] (he/him)]

https://hg.mozilla.org/build/slave_health/rev/9f2e04b2f50f

Comment 6

[Chris Cooper [:coop] (he/him)]

This is live now. Please file follow-up bugs for issues.