osmith has submitted this change. (
https://gerrit.osmocom.org/c/osmo-ci/+/29700 )
Change subject: jenkins-gerrit: meaningful re_start_build error
......................................................................
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(-)
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
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
https://gerrit.osmocom.org/c/osmo-ci/+/29700
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ib29b7ce69a27453b460f946ad67bd12faffc2c89
Gerrit-Change-Number: 29700
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged