Page MenuHomePhabricator

ToolsAdmin: Error 500 for ZppixBot repo listing
Closed, ResolvedPublic

Description

It returns a 500 error when clicking on the repo listed under the tool info provided within tools admin website, it was recently connected to its repo on gerrit by @Krenair (T151279)
Heres the Request ID: f9272ceb6d5844e4a8bda2991a3bee99

Screenshot of user-side error page: https://goo.gl/Mdr9La

Event Timeline

Zppix created this task.Nov 22 2016, 10:10 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 22 2016, 10:10 PM

django.core.urlresolvers.NoReverseMatch: Reverse for 'repo_view' with arguments '()' and keyword arguments '{'tool': 'zppixbot', 'repo': 'labs-tools-ZppixBot'}' not found. 1 pattern(s) tried: ['tools/id/(?P<tool>[_a-z][-0-9_a-z]*)/repos/id/(?P<repo>[_a-z][-0-9_a-z]*)$']

Change 323088 had a related patch set uploaded (by BryanDavis):
Allow uppercase in diffusion repo names

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

bd808 claimed this task.Nov 22 2016, 11:56 PM

I have stopped the 500 error temporarily by editing the record created for T151279 to use all lower case. This is still broken however because the name is used to lookup data in diffusion via the api and that is not case-insensitive.

Restricted Application added a project: User-bd808. · View Herald TranscriptNov 22 2016, 11:56 PM
bd808 triaged this task as High priority.Nov 22 2016, 11:57 PM

Change 323088 merged by jenkins-bot:
Allow uppercase in diffusion repo names

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

@bd808 Anything i need to do to fix that?

@bd808 Anything i need to do to fix that?

I've got a fix on Striker's side that removes the bad assumption. I'll try to get it deployed tomorrow.

Mentioned in SAL (#wikimedia-operations) [2016-11-23T01:13:25Z] <bd808> Updated striker to c546f4c (T151409)

bd808 moved this task from To Do to Done on the User-bd808 board.Jan 7 2017, 12:19 AM
bd808 moved this task from Done to Archive on the User-bd808 board.
bd808 moved this task from Doing to Done on the Striker board.Mar 4 2017, 3:54 AM