New issue
Advanced search Search tips

Issue 674624 link

Starred by 3 users

Issue metadata

Status: Duplicate
Merged: issue 775266
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

404 errors when trying to snooze items

Project Member Reported by twelling...@chromium.org, Dec 15 2016

Issue description

I'm getting 404 errors when trying to snooze alerts on sheriff-o-matic-staging.appspot.com.

Console output attached.
 
sheriff-o-matic-errors.png
126 KB View Download
The testText thing is a red herring.

Also why are you using sheriff-o-matic-staging? Android sheriffs should be able to use sheriff-o-matic.appspot.com. Does some documentation have that link?
Status: Available (was: Untriaged)
I think it's 404-ing because you're trying to snooze something where the key has a '/' in it, which is confusing. Should probably urlencode the key...
My bookmarks have that link; I'll update it to point at the non-staging version.

I was trying to snooze an alert for a stable master chromium-fyi bot (which probably shouldn't be on the dashboard at all).
Huh, that's strange. The key is currently being run through encodeURIComponent(key) in sendAnnotation. But yeah, it does look like it's not getting encoded here. 
*stale master chromium-fyi bot
Cc: jbudorick@chromium.org
https://cs.chromium.org/chromium/build/scripts/slave/gatekeeper_trees.json?q=gatekeeper_trees&sq=package:chromium&l=55 is the line that adds a dependency on chromium.fyi to android.

John, we still care about that builder?
Yes, as it triggers five of the fyi bots, but I'm not sure about having sheriffs be responsible for anything on fyi.
Labels: -Infra-DX Milestone-UX
 Issue 678589  has been merged into this issue.
 Issue 698758  has been merged into this issue.
Owner: zhangtiff@chromium.org
Status: Assigned (was: Available)
Mergedinto: 775266
Status: Duplicate (was: Assigned)

Sign in to add a comment