Report buildbucket ID for all builds

infra/tools/send_monitoring_event/send_event.py

Index: infra/tools/send_monitoring_event/send_event.py
diff --git a/infra/tools/send_monitoring_event/send_event.py b/infra/tools/send_monitoring_event/send_event.py
index 50a3debd5c713a5fdca4d27aefec6862a146daff..ba5d2d02ab6eb0bb06970caac4f4d94a67a676a9 100644
--- a/infra/tools/send_monitoring_event/send_event.py
+++ b/infra/tools/send_monitoring_event/send_event.py
@@ -126,6 +126,8 @@ def get_arguments(argv):
                            'Each one must be less than 20 characters long.')
   build_group.add_argument('--build-event-patch-url',
                            help='URL of the patchset that triggered build')
+  build_group.add_argument('--build-event-bbucket-id',

[pgervais, 2016/06/09 16:38:49]

After-the fact drive-by: I think it's always better to avoid abbreviation. 

+                           help='Buildbucket ID for this build')
 
   build_group.add_argument('--build-event-goma-stats-path',
                            metavar='FILENAME',
@@ -331,6 +333,7 @@ def send_build_event(args):
     result=args.build_event_result,
     extra_result_code=args.build_event_extra_result_code,
     patch_url=args.build_event_patch_url,
+    bbucket_id=args.build_event_bbucket_id,
     timestamp_kind=args.event_mon_timestamp_kind,
     event_timestamp=args.event_mon_event_timestamp,
     goma_stats=goma_stats,
@@ -418,7 +421,8 @@ def read_events_from_file(filename):
             timestamp_kind=args.get('event-mon-timestamp-kind'),
             event_timestamp=args.get('event-mon-event-timestamp'),
             service_name=args.get('event-mon-service-name'),
-            patch_url=args.get('build-event-patch-url')))
+            patch_url=args.get('build-event-patch-url'),
+            bbucket_id=args.get('build-event-bbucket-id')))
       else:
         LOGGER.warning('build-event-type field not found, skipping line '
                        '%d in %s', lineno, filename)