Remove unused files under treeherder/workers/

RESOLVED FIXED

Status

Tree Management
Treeherder
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Most of the files under https://github.com/mozilla/treeherder/tree/master/treeherder/workers are blank/default templates.

The only file of value is:
https://github.com/mozilla/treeherder/blob/master/treeherder/workers/management/commands/shutdown_workers.py

...however I've never used it and don't know if we even need it any more.

Cameron, thoughts? :-)
(Assignee)

Comment 1

2 years ago
(In reply to Ed Morley [:emorley] from comment #0)
> ...however I've never used it and don't know if we even need it any more.
> 
> Cameron, thoughts? :-)
Flags: needinfo?(cdawson)
I'm pretty sure those are no longer used or useful.  That warm shutdown for the workers was most likely prior to the current setup with :fubar.  I think we can safely delete them.  :)

That being said, Mauro wrote that command.  Perhaps he has more info?
Flags: needinfo?(cdawson) → needinfo?(mdoglio)
Yeah those ones were an attempt to fix the issue with stuck workers we had at some point. I don't think we need it anymore
Flags: needinfo?(mdoglio)
Created attachment 8712626 [details] [review]
[treeherder] mozilla:rm-workers-app > mozilla:master
(Assignee)

Updated

2 years ago
Assignee: nobody → emorley
Status: NEW → ASSIGNED
(Assignee)

Updated

2 years ago
Attachment #8712626 - Flags: review?(mdoglio)
Attachment #8712626 - Flags: review?(mdoglio) → review+
(Assignee)

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.