osmith submitted this change.

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, but someone else must approve
  msuraev: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved

jenkins-gerrit: meaningful re_start_build error

Related: OS#2385
Change-Id: Ib29b7ce69a27453b460f946ad67bd12faffc2c89
---
M scripts/jenkins-gerrit/pipeline_summary.py
1 file changed, 4 insertions(+), 0 deletions(-)

diff --git a/scripts/jenkins-gerrit/pipeline_summary.py b/scripts/jenkins-gerrit/pipeline_summary.py
index c12ad64..85b2393 100755
--- a/scripts/jenkins-gerrit/pipeline_summary.py
+++ b/scripts/jenkins-gerrit/pipeline_summary.py
@@ -65,6 +65,10 @@
             # Parse result lines
             if re_result.match(line):
                 stage = line.split("_")[1].lower()
+                assert stage in ret, f"found result for stage {stage}, but" \
+                        " didn't find where it was started. The" \
+                        " re_start_build regex probably needs to be adjusted" \
+                        " to match the related gerrit-*-build job."
                 passed = line.split("=")[1].rstrip() == "1"
                 ret[stage]["passed"] = passed

To view, visit change 29700. To unsubscribe, or for help writing mail filters, visit settings.