commit eda522b0f964583b55475e0eb94b60c36fee9cce
author Alex Klein <saklein@chromium.org> Tue Oct 18 09:11:27 2022 -0600
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Nov 04 20:15:07 2022 +0000
tree 0e528ad187b225ad89472e8999215f1609b94aa3
parent 49e02b9ee608d02d8981ee92e6921519ae39f35d

TODO updates.

Switch TODOs from an ldap to the bug they reference.

BUG=None
TEST=None

Change-Id: I2640970bb8ad1318d00c84047d9230374b26d2e3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4003937
Auto-Submit: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Sergey Frolov <sfrolov@google.com>

api/metrics.py

diff --git a/api/metrics.py b/api/metrics.py
index b51e5e1..97c88ac 100644
--- a/api/metrics.py
+++ b/api/metrics.py
@@ -44,7 +44,7 @@
                 input_event.timestamp_epoch_millis,
             )
         elif input_event.op == metrics_lib.OP_STOP_TIMER:
-            # TODO(wbbradley): Drop the None fallback https://crbug.com/1001909.
+            # TODO(b/187788898): Drop the None fallback.
             timer = timers.pop(input_event.arg, None)
             if timer is None:
                 logging.error(
@@ -99,6 +99,6 @@
         output_event.timestamp_milliseconds = counter_times[counter]
 
     # Check for any unhandled timers.
-    # TODO(wbbradley): Turn this back into an assert https://crbug.com/1001909.
+    # TODO(b/187788898): Turn this back into an assert.
     if timers:
         logging.error("excess timer metric data left over: %s", timers)