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

Issue 672542 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Add event file upload to cbuildbot

Project Member Reported by chingcodes@chromium.org, Dec 8 2016

Issue description

Add uploading of the generated events file from the build stage.

File to upload: buildroot/events/build-events.json

Related bug Chromium:660050
 

Comment 1 by autumn@chromium.org, Dec 13 2016

Labels: -current-issue
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/e654b974bffbd6a986b6c3eabeda06246f238193

commit e654b974bffbd6a986b6c3eabeda06246f238193
Author: Chris Ching <chingcodes@chromium.org>
Date: Fri Dec 09 23:19:04 2016

build_stages: add events file upload

build_packages will write events to build_root/events/build-events.json.
This file will be uploaded after the build command is completed.

BUG= chromium:672542 
TEST=unittests

Change-Id: I2a23f55b09ab0365b55640fb5261e6ae6218f0e3
Reviewed-on: https://chromium-review.googlesource.com/417993
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Chris Ching <chingcodes@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Chris Ching <chingcodes@chromium.org>

[modify] https://crrev.com/e654b974bffbd6a986b6c3eabeda06246f238193/cbuildbot/stages/build_stages.py
[modify] https://crrev.com/e654b974bffbd6a986b6c3eabeda06246f238193/cbuildbot/commands.py
[modify] https://crrev.com/e654b974bffbd6a986b6c3eabeda06246f238193/cbuildbot/commands_unittest.py

Project Member

Comment 3 by bugdroid1@chromium.org, Jan 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/12fa1c641fc37e559026ef3137d593281842a8aa

commit 12fa1c641fc37e559026ef3137d593281842a8aa
Author: Chris Ching <chingcodes@chromium.org>
Date: Wed Jan 18 18:33:28 2017

build_stages: add build-events file to archive

Also add more logging messages

BUG= chromium:672542 
TEST=trybot

Change-Id: I803a25bdf61dcaebc890214b220a89867d01eac2
Reviewed-on: https://chromium-review.googlesource.com/430092
Commit-Ready: Chris Ching <chingcodes@chromium.org>
Tested-by: Chris Ching <chingcodes@chromium.org>
Reviewed-by: Chris Ching <chingcodes@chromium.org>

[modify] https://crrev.com/12fa1c641fc37e559026ef3137d593281842a8aa/cbuildbot/stages/build_stages.py

Project Member

Comment 4 by bugdroid1@chromium.org, Jan 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/e0e01cacf8710d122b534daccd0ca168368055f1

commit e0e01cacf8710d122b534daccd0ca168368055f1
Author: Chris Ching <chingcodes@chromium.org>
Date: Tue Jan 24 23:53:21 2017

cros_event: Changed Event ID gen for gCloud integration

EventLogger now creates events that is a list, and not an int.
The new list is [event_kind, event_id]. With the default kind
being 'root'.

The term kind come's from gCloud's datastore concept of entities
 - https://cloud.google.com/datastore/docs/concepts/entities

BUG= chromium:672542 
TEST=unittest

Change-Id: Ia3db50a918f1054e0710cee5e0db543539f1bd4f
Reviewed-on: https://chromium-review.googlesource.com/433084
Commit-Ready: Chris Ching <chingcodes@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Chris Ching <chingcodes@chromium.org>

[modify] https://crrev.com/e0e01cacf8710d122b534daccd0ca168368055f1/lib/cros_event.py
[modify] https://crrev.com/e0e01cacf8710d122b534daccd0ca168368055f1/lib/cros_event_unittest.py

Project Member

Comment 5 by bugdroid1@chromium.org, Jan 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/151617c3ede5a8bb3c6f6fe8aa7c45ae8e3344a8

commit 151617c3ede5a8bb3c6f6fe8aa7c45ae8e3344a8
Author: Chris Ching <chingcodes@chromium.org>
Date: Wed Jan 25 23:57:06 2017

parallel_emerge: Set Events' 'kind'

Changed Events to use the new 'kind' keyword so that events are
correctly identified in gCloud datastore.

BUG= chromium:672542 
TEST=build_packages

Change-Id: Icd7329fd6cbe326dad0920408a6baa83a62ae87d
Reviewed-on: https://chromium-review.googlesource.com/433183
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Chris Ching <chingcodes@chromium.org>

[modify] https://crrev.com/151617c3ede5a8bb3c6f6fe8aa7c45ae8e3344a8/scripts/parallel_emerge.py

Status: Fixed (was: Started)

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment