osmith submitted this change.

View Change

Approvals: fixeria: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
jenkins-gerrit: print the URLs this script reads

Make errors easier to understand by printing all URLs that the script is
reading from.

Change-Id: Ic0e7fc0e5a82927fc9ef6ec02c374b1ec7dd73e5
---
M scripts/jenkins-gerrit/comment_generate.py
1 file changed, 3 insertions(+), 0 deletions(-)

diff --git a/scripts/jenkins-gerrit/comment_generate.py b/scripts/jenkins-gerrit/comment_generate.py
index a2aec0e..c81011f 100755
--- a/scripts/jenkins-gerrit/comment_generate.py
+++ b/scripts/jenkins-gerrit/comment_generate.py
@@ -41,6 +41,7 @@
global re_distro

url = f"{job_url}/consoleText"
+ print(f"Reading {url}")
with urllib.request.urlopen(url) as response:
content = response.read().decode("utf-8")
match = re_distro.search(content)
@@ -77,6 +78,7 @@
ret = {}

url = f"{build_url}/consoleText"
+ print(f"Reading {url}")
with urllib.request.urlopen(url) as response:
for line in io.TextIOWrapper(response, encoding='utf-8'):
# Parse start build lines
@@ -123,6 +125,7 @@

ret = []
url = f"{job['url']}/consoleFull"
+ print(f"Reading {url}")
with urllib.request.urlopen(url) as response:
for line in io.TextIOWrapper(response, encoding='utf-8'):
if " completed with result " in line:

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

Gerrit-MessageType: merged
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ic0e7fc0e5a82927fc9ef6ec02c374b1ec7dd73e5
Gerrit-Change-Number: 40911
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>