Page MenuHomePhabricator

Delete tool speed-patrolling
Closed, ResolvedPublic

Description

I created the speed-patrolling tool because the Toolforge API temporarily reported that speedpatrolling, the name I actually wanted, was unavailable; however, I was later able to create the speedpatrolling tool and migrate everything to it, so now speed-patrolling should be deleted. (Neither tool should have any users yet, I’m still in the process of cleaning this all up, so there should be no disruption.)

Event Timeline

Mentioned in SAL (#wikimedia-cloud) [2019-08-23T08:35:29Z] <wm-bot> <lucaswerkmeister> repurposing this defunct tool (T212968) for some ad-hoc CORS testing

@LucasWerkmeister have you re-abandoned this tool or should I avoid deleting it?

Mentioned in SAL (#wikimedia-cloud) [2021-11-16T21:10:11Z] <wm-bot> <lucaswerkmeister> stopped webservice, apparently I left it running but access.log has no requests since 2019-11-03 (CC T212968)

I don’t remember what those tests were but I’m pretty sure they don’t matter anymore. Feel free to delete the tool. (If I want to do some more ad-hoc testing in the future, would it be okay if I create a dedicated tool with a name like lucaswerkmeister-test, or would that be frowned upon?)

Andrew claimed this task.

You're welcome to create tools for testing purposes whenever you need one.

In the meantime, I've deleted speed-patrolling.

Hm, is it intentional that there’s still a trace of the tool left behind, so to speak?

lucaswerkmeister@tools-sgebastion-07:~$ printf '%s\n' $(groups) | grep speed
tools.speed-patrolling
tools.speedpatrolling
lucaswerkmeister@tools-sgebastion-07:~$ become speed-patrolling 
This tool is disabled.

Mentioned in SAL (#wikimedia-cloud) [2021-11-18T13:59:43Z] <lucaswerkmeister> added “owner of abandoned tools” to maintainers and removed self (T212968)

(It still shows up in my groups, but maybe that’s just because credentials are cached for a bit.)

Change 739913 had a related patch set uploaded (by Andrew Bogott; author: Andrew Bogott):

[cloud/toolforge/disable-tool@master] os.path.remove doesn't exist!

https://gerrit.wikimedia.org/r/739913

Change 739913 merged by Andrew Bogott:

[cloud/toolforge/disable-tool@master] os.path.remove doesn't exist!

https://gerrit.wikimedia.org/r/739913

You found a bug! It's fixed now :)

\o/

lucaswerkmeister@tools-sgebastion-07:~$ printf '%s\n' $(groups) | grep speed
tools.speedpatrolling
lucaswerkmeister@tools-sgebastion-07:~$ become speed-patrolling 
become: no such tool 'speed-patrolling'