osmith submitted this change.

View Change


Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
scripts/jenkins-gerrit: make assert more verbose

Change-Id: I0ea40663220f7eb8ee72e43f3f6b776be5835db2
---
M scripts/jenkins-gerrit/comment_generate.py
1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/scripts/jenkins-gerrit/comment_generate.py b/scripts/jenkins-gerrit/comment_generate.py
index e96893c..147faac 100755
--- a/scripts/jenkins-gerrit/comment_generate.py
+++ b/scripts/jenkins-gerrit/comment_generate.py
@@ -97,7 +97,8 @@
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."
+ " to match the related gerrit-*-build job.\n\n" \
+ f"ret: {ret}"
ret[stage]["passed"] = (match.group(2) == "SUCCESS")

return ret

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

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I0ea40663220f7eb8ee72e43f3f6b776be5835db2
Gerrit-Change-Number: 33991
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged