New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 790559 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

events.alerts isn't getting populated in prod

Project Member Reported by seanmccullough@chromium.org, Nov 30 2017

Issue description

Logs for /_cron/analyze/foo all have this error:
eventupload: Uploader.Put failed :: {"error":"1 row insertion failed"}
error sending alerts to bigquery: 1 row insertion failed


 
Also ran bqshcmaupdater to check if I missed that step and it didn't have and updates for the alerts table. Should probably add some extra logging to see if it's a row size limit issue.
Owner: seanmccullough@chromium.org
Status: Started (was: Available)
Cc: katthomas@chromium.org
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/3e9fee3ce91ca7415203713941426d5ace7a7fae

commit 3e9fee3ce91ca7415203713941426d5ace7a7fae
Author: Sean McCullough <seanmccullough@chromium.org>
Date: Fri Dec 01 02:02:12 2017

[som] Send a *pointer* to a proto struct to eventuploader

Bug:  790559 
Change-Id: Ice577212c8bcf6d12aebbf649f90f6a23d54cb2c
Reviewed-on: https://chromium-review.googlesource.com/801353
Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org>
Commit-Queue: Sean McCullough <seanmccullough@chromium.org>

[modify] https://crrev.com/3e9fee3ce91ca7415203713941426d5ace7a7fae/go/src/infra/appengine/sheriff-o-matic/som/handler/analyze.go

Agh, I'm sorry. I swore when I made that change I also verified that it was what all clients were doing already... Made a bug for eventuploader to be more helpful in this case: https://bugs.chromium.org/p/chromium/issues/detail?id=791032
Status: Fixed (was: Started)

Sign in to add a comment