Closed Bug 1215450 Opened 6 years ago Closed 6 years ago

Treeherder single result set view is stuck "waiting for a push with revision xxx"


(Tree Management :: Treeherder, defect, P1)



(Not tracked)



(Reporter: azasypkin, Assigned: camd)



(1 file)

47 bytes, text/x-github-pull-request
: review+
Details | Review
Please, see [1], Treeherder doesn't show any results and just always displays "waiting for a push with revision xxx", it happens for other PR pushes as well. Close-open PR didn't help.

At the same time, [2] works fine.


Hey Kendall,

We were advised that you can help to diagnose this :)

Flags: needinfo?(klibby)
weird. I don't actually know what's up there. NI on Cam, maybe he knows what's up?  (sadly, he's PST, and Ed's out today)
Flags: needinfo?(klibby) → needinfo?(cdawson)
I see what's going on here.  I think a recent change caused a regression here.  Working on a fix now.  I'll try to get it pushed out as soon as possible.  But may only make it to stage today, tbh.
Assignee: nobody → cdawson
Flags: needinfo?(cdawson)
This will happen with any revision that is full 40 chars in github repos.  The full 40 has been stored, but we are only searching for the 12 char, so it can never find it.  We need to search for both short and long in that field.
Priority: -- → P1
As a sort-of work around, you can change the URL param from "revision" to "tochange" which will show your revision at the top of the list.
Attached file PR
Attachment #8675064 - Flags: review?(mdoglio)
Attachment #8675064 - Flags: review?(mdoglio) → review+
Commit pushed to master at
Bug 1215450 - single resultset view stuck when using long revision

Some repos have been storing 40 char revisions, though most only store
But a recent change to search by only 12 chars, even if 40 passed in
the ability to search when the stored length was 40.  This change will
search for both length revisions, if the 40 char is passed in.
this is now fixed and pushed to master.  It should go to stage and production servers soon.
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.