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

Issue 786415 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Task

Blocking:
issue 786056



Sign in to add a comment

Migrate IsolateClientEvent from event_mon to new pipeline

Project Member Reported by katthomas@chromium.org, Nov 17 2017

Issue description

maruel@, yours to re-assign as desired

Please let me know how I can help!
 
Cc: tandrii@chromium.org
 Issue 765434  has been merged into this issue.
It seems like this event data has been useful for the analysis work that mcgreevy is currently doing. @maruel, do you think this is worth doing then? If so, when do you think you or someone else will be able to get to it? If not, we will need to take steps to remove the event_mon instrumentation after mcgreevy is done with his analysis. 
Components: -Infra>Monitoring Infra>Platform>Swarming
Cc: jparent@chromium.org
@maruel, thoughts on this? 

Comment 6 by mar...@chromium.org, Dec 20 2017

Once mcgreevy is done with the analysis, we can remove the event_mon code from the client.

That's still nice to have data for mass analysis later on, to do on BQ. I think we can live without time series.
Thanks for the update!

It's actually event data, and not time series data.

We can always add new instrumentation for BigQuery if we find we need it. Thanks to the new pipeline, this should be much easier than it used to be. 
Cc: mar...@chromium.org
Owner: katthomas@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 10 by bugdroid1@chromium.org, Jan 9 2018

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

commit 9ca18ff161cc51eb426fa26ce0fc10130c58faa7
Author: Katie Thomas <katthomas@google.com>
Date: Tue Jan 09 23:27:46 2018

Roll infra/go/src/go.chromium.org/luci/ d1be5461d..2c110abe9 (1 commit)

https://chromium.googlesource.com/infra/luci/luci-go/+log/d1be5461de9b..2c110abe941d

$ git log d1be5461d..2c110abe9 --date=short --no-merges --format='%ad %ae %s'
2017-12-21 katthomas [isolate] remove IsolateClientEvent sending

Created with:
  roll-dep infra/go/src/go.chromium.org/luci

Bug:786415
Change-Id: Icda6d18a83bb357e5e0bcad0d13c54a826baaa88
Reviewed-on: https://chromium-review.googlesource.com/857517
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Commit-Queue: Katie Thomas <katthomas@google.com>

[modify] https://crrev.com/9ca18ff161cc51eb426fa26ce0fc10130c58faa7/DEPS

Status: Fixed (was: Started)

Sign in to add a comment