Last modified: 2013-06-28 19:52:29 UTC
Now that deployment is doing from Git/Gerrit please modify Logmsgbot/Morebots in a way that entries in the Server admin log with a reference to Gerrit like 'g4102' (see http://wikitech.wikimedia.org/index.php?title=Server_admin_log&action=historysubmit&diff=45476&oldid=45473 ) are linked to Gerrit.
Using {{gerrit|1234}} markup works, see http://wikitech.wikimedia.org/index.php?title=Server_admin_log&diff=53199&oldid=53198 . Not sure how common g1234 format is, at least I couldn't find a recent entry with this.
It should link Git hashes and gerrit change-id hashes, those are recommended form for identifiers of changes. During deployment a git hash is most precise (not ambiguous between different branches and usable in Git outside Gerrit context). Regexes for these can be found in the Gerrit configuration and Bugzilla configuration, which already have these. Should probably be added to the syntax morebots already has for expanding "bug 123" to {{bug|123}} which displays [[bugzilla:123|bug 123]]. I updated the {{Gerrit}} template on wikitech to also take git hashes and gerrit change-ids, so it should be easier to expand for more bots. It only has to expand the matched hashes to {{Gerrit|<match>}}.
Related URL: https://gerrit.wikimedia.org/r/68135 (Gerrit Change I0a7ca5450f9f74ebd36fb47f8e30d2fc8135df97)
Current status: 68135 was deployed, but I was dumb and screwed up the regexp. (It matches '2013', for example.) I worked around some of the bugs in https://wikitech.wikimedia.org/wiki/Module:Gerrit, but I should fix the regexp. I wonder if perhaps we should avoid doing *any* special processing of the message content in adminbot and instead just wrap the entire message in a template or module invocation, like: {{SAL|17:33 mutante: upgrading binutils and libldap on bast1001}} That way we do all the fancy string processing in Lua. We'd be able to tweak it / enhance it without relying on rebuilding the adminbot package.
