client/third_party/infra_libs/event_mon/monitoring.py - Issue 1894593004: luci-py: roll infra_libs / gae_ts_mon to 4d74a66

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: client/third_party/infra_libs/event_mon/monitoring.py

Issue 1894593004: luci-py: roll infra_libs / gae_ts_mon to 4d74a66 (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-py@master

Patch Set: Created 4 years, 8 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « client/third_party/infra_libs/README.swarming ('k') | client/third_party/infra_libs/event_mon/protos/chrome_infra_log_pb2.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: client/third_party/infra_libs/event_mon/monitoring.py

diff --git a/client/third_party/infra_libs/event_mon/monitoring.py b/client/third_party/infra_libs/event_mon/monitoring.py

index 2965c86c3de2d263207a8b900e6818b315d101a7..c3d692348e40c9f096406f00d98375e4eb6d9a6d 100644

--- a/client/third_party/infra_libs/event_mon/monitoring.py

+++ b/client/third_party/infra_libs/event_mon/monitoring.py

@@ -261,6 +261,7 @@ def get_build_event(event_type,

build_number=None,

build_scheduling_time=None,

step_name=None,

+ step_text=None,

step_number=None,

result=None,

extra_result_code=None,

@@ -330,6 +331,8 @@ def get_build_event(event_type,

if step_name:

event.build_event.step_name = step_name

+ if step_text:

+ event.build_event.step_text = step_text

if step_number is not None:

event.build_event.step_number = step_number

if patch_url is not None:

@@ -419,6 +422,7 @@ def send_build_event(event_type,

build_number=None,

build_scheduling_time=None,

step_name=None,

+ step_text=None,

step_number=None,

result=None,

extra_result_code=None,

@@ -443,6 +447,7 @@ def send_build_event(event_type,

scheduled. This is required when build_number is provided to make it

possibly to distinguish two builds with the same build number.

step_name (str): name of the step.

+ step_text (str): text of the step.

step_number (int): rank of the step in the build. This is mandatory

if step_name is provided, because step_name is not enough to tell the

order.

@@ -464,6 +469,7 @@ def send_build_event(event_type,

build_number=build_number,

build_scheduling_time=build_scheduling_time,

step_name=step_name,

+ step_text=step_text,

step_number=step_number,

result=result,

extra_result_code=extra_result_code,