pw_presubmit: Store buildbucket ids as strs

Bug: b/270154434 Change-Id: Ife8008ede754c39841cde1dfa889e91f8c0d3962 Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/130892 Reviewed-by: Ted Pudlik <[email protected]> Pigweed-Auto-Submit: Rob Mohr <[email protected]> Commit-Queue: Auto-Submit <[email protected]>
google · Feb 24, 2023 · d0e2943 · d0e2943
1 parent 8db026a
commit d0e2943
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/pw_presubmit/docs.rst b/pw_presubmit/docs.rst
@@ -142,7 +142,7 @@ following members:
 
 * ``round``: The zero-indexed round number.
 * ``builds_from_previous_iteration``: A list of the buildbucket ids from the
-  previous round, if any.
+  previous round, if any, encoded as strs.
 
 The ``triggers`` member is a sequence of ``LuciTrigger`` objects, which have the
 following members:

diff --git a/pw_presubmit/py/pw_presubmit/presubmit.py b/pw_presubmit/py/pw_presubmit/presubmit.py
@@ -217,7 +217,7 @@ def __len__(self) -> int:
 @dataclasses.dataclass
 class LuciPipeline:
     round: int
-    builds_from_previous_iteration: Sequence[int]
+    builds_from_previous_iteration: Sequence[str]
 
     @staticmethod
     def create(
@@ -239,9 +239,9 @@ def create(
 
         return LuciPipeline(
             round=int(pipeline_props['round']),
-            builds_from_previous_iteration=[
-                int(x) for x in pipeline_props['builds_from_previous_iteration']
-            ],
+            builds_from_previous_iteration=list(
+                pipeline_props['builds_from_previous_iteration']
+            ),
         )