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

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

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.

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.

bd808 moved this task from Done to Archive on the User-bd808 board.